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

SVN_REVNUM_T_FMT anyone?, WAS: RE: [PATCH]: fix #661 (Bad user supplied rev # produces scary error)

From: Sander Striker <striker_at_apache.org>
Date: 2002-05-01 08:53:12 CEST

> From: David Kimdon [mailto:dwhedon@dualmedia.fr]On Behalf Of David
> Kimdon
> Sent: 01 May 2002 07:58

> Hi,
> This patch fixes the spooky error message of issue #661.

> + return
> + svn_error_createf
> + (SVN_ERR_FS_NO_SUCH_REVISION, 0, 0, fs->pool,
> + "reference to non-existent revision `%lu' in filesystem `%s'",
> + (unsigned long) rev, fs->path);

I've seen this a lot in the codebase. What I would really like
is an SVN_REVNUM_T_FMT define, which we can use similar like the
APR_xxx_T_FMT defines (example: printf("%" APR_OFF_T_FMT, some_apr_off_t_var); ).
We can loose the cast with this and changing the type (lets not do that!),
will only be a one place modification.


PS. This has nothing to do with the presented patch.

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed May 1 08:46:37 2002

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.