Re: More error leaks

On Mon, Jan 23, 2006 at 07:53:20PM +0000, Philip Martin wrote:
> > So do we add a macro that expands to '__attribute__(...) svn_error_t *'
> > and adjust all the existing function declarations/definitions to use that?
>
> No need for a macro, simply add __attribute__(...) to the function
> prototype. It should work since we already do that for other
> attributes.
>

I was thinking of a macro in case we later needed to change the expansion
(for example, if gcc changed the attribute we needed to use).

Do we need to adjust both the declaration and definition, or is one
enough?

Regards,
Malcolm

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jan 23 21:16:27 2006