[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r22678 - trunk/subversion/libsvn_client

From: Daniel Rall <dlr_at_collab.net>
Date: 2006-12-13 04:49:02 CET

On Tue, 12 Dec 2006, David Glasser wrote:

> On 12/12/06, dlr@tigris.org <dlr@tigris.org> wrote:
>
> >
> >+/* CTX is of type "svn_client_ctx_t *". */
> >+#define SVN_CLIENT__HAS_LOG_MSG_FUNC(ctx) \
> >+ (ctx->log_msg_func3 || ctx->log_msg_func2 || ctx->log_msg_func)
> >+
>
> Shouldn't the ctx's in the expansion be parenthesized to avoid
> precedence mixups if the argument is weird?

I'll buy that for a dollar! r22680, thanks Dave.

- Dan

  • application/pgp-signature attachment: stored
Received on Wed Dec 13 04:42:49 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.