Re: [ros-dev] [ros-diffs] [hbelusca] 58902: [FREELDR] - Fix some function parameter types, so to avoid unuseful casts (from pointers to const strings to pointers to strings in particular). - The big thing: make FreeLdr load...

[Sven Barth]

Am 02.05.2013 00:30, schrieb Timo Kreuzer:
>  
>  I have an actual pragmatic argument against one liners:
>  When you debug code (and I mean with a real debugger aka WinDbg and 
>  not kdbg) and you step through the source, whenever there is a one 
>  liner, you just don't see which branch it takes and whether it 
>  executed the statement or not. So you have to check other things. Look 
>  at the variables. And when there is stuff like
>  "if (FOO_MACRO(Value)) GlobalVariable++;" you are simply f***ed.
>  You'll have to add a watch for GlobalVariable and check the value 
>  before and after.
>  I really prefer to see what path the code takes, when stepping over it.
>
>  So it's not a question of style or beauty, but a question of 
>  convenient debugging. 
This is also the reason why I don't like to put a
statement in the same 
line as a if/while-condition or for-loop-header no matter what 
programming language I'm using.

Regards,
Sven