On Fri, 2014-02-21 at 16:12 +0100, Petr Spacek wrote:
> Hello,
> 
> Add function attributes warn_unused_result and nonnull
> where appropriate and add missing CHECK()s to string operations.
> 
> Lukas, thanks for catching the missing CHECK() around str_new().
> 
> As a reward, you can review attached patches.
> 
> Have fun! :-)

NACK

Adding attributes to a function definition is a no-op and can create
potential confusion should the definitions and declarations differ.

I would strongly prefer that they are only used in function
declarations.

Nathaniel

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to