[Lldb-commits] [PATCH] Fix asserts in debugserver with side effects

[Charles Davis]

Thanks! r151013.

Chip

On Feb 20, 2012, at 5:53 PM, Greg Clayton wrote:

> Yep, looks good. Commmit anytime.
> 
> On Feb 20, 2012, at 4:39 PM, Charles Davis wrote:
> 
>> Hi,
>> 
>> I'm working on trying to get debugserver built from the Makefile system, and I've run into a bug.
>> 
>> If NDEBUG is defined, debugserver will actually break. Since there are no #ifdef NDEBUGs anywhere in the source, this indicates to me that an assert(3) expression somewhere contains a side effect. This patch should fix all the assertions in debugserver not to have side effects, and get it working in a Release build.
>> 
>> OK to commit? (Note that I have commit privileges to all LLVM repositories, including LLDB.)
>> 
>> Chip
>> <dnb-assert-fix.patch>_______________________________________________
>> lldb-commits mailing list
>> lldb-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
>