[asterisk-dev] russell: trunk r118556 - /trunk/include/asterisk/compat.h

[Russell Bryant]

On May 27, 2008, at 4:22 PM, Tony Mountifield wrote:
> It's also permissible to have the __attribute__ after the function
> declaration instead, which IMHO is slightly clearer to read because
> the function name is not then buried in the middle of the line:
>
> int vasprintf(char **strp, const char *fmt, va_list ap)  
> __attribute__ ((format (printf, 2, 0)));
>
> Only a minor point, but I thought it worth mentioning.


Thanks for pointing that out.  I have always put it in the middle and  
it has become habit.  For some reason, I thought that putting it  
afterwords was not valid.  Thank you for letting me know, as I do  
agree that it is more readable.

--
Russell Bryant
Senior Software Engineer
Open Source Team Lead
Digium, Inc.