[Olsr-dev] (format)

[Sven-Ola Tuecke]

Hi Bernd,

no. I've removed those braces and all was fine. As I wrote - outdated 
compiler, outdated kernel includes etc. So simply forget that - there are 
more important things than that :)

BTW: The process_routes.c rewrite seems to work. Less cluttered 
logfiles(e.g. no more "cannot remove route" / "add route already exists" on 
multihomed hosts). So I switch status from "do not use" to "try it on your 
own risk". These two patches should apply fine on top of olsrd-current - in 
that order:

http://olsrexperiment.de/download/sven-ola/nylon/packages/olsrd/files/110-policy-routing.patch
http://olsrexperiment.de/download/sven-ola/nylon/packages/olsrd/files/130-routing-cleanup.patch

// Sven-Ola

"Bernd Petrovitsch" <(spam-protected)> schrieb im Newsbeitrag 
news:(spam-protected)
> On Wed, 2007-05-23 at 18:57 +0200, Sven-Ola Tuecke wrote:
> [...]
>> investigated a bit. That thing is really old (claims 2.9x from 2001). 
>> Look
>> for example in src/build_msg.c:818.
>>
>> OLSR_PRINTF(1, "TC: Sending empty package\n");
>>
>> which expands to
>>
>> do { if((olsr_cnf->debug_level >= ( 1 )) && debug_handle)
>> fprintf(debug_handle, (  "TC: Sending empty package\n"  ); } while (0) ;
>>
>> Looks like the empty ellipse (...args) is handled differently. As long as
>> other compilers (windoof, osx, bsd) do not complain -> simply ignore my
>> post.
>
> Another thing came to my mind: Does it help if you put some additional
> spaces before and after the "args..." and/or "##args" in that macro?
>
> Bernd
> -- 
> Firmix Software GmbH                   http://www.firmix.at/
> mobil: +43 664 4416156                 fax: +43 1 7890849-55
>          Embedded Linux Development and Services
>
>


--------------------------------------------------------------------------------


> -- 
> Olsr-dev mailing list
> (spam-protected)
> https://lists.olsr.org/mailman/listinfo/olsr-dev
>