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

AW: svn commit: r1100704 - /subversion/trunk/subversion/libsvn_wc/wc_db.c

From: Markus Schaber <m.schaber_at_3s-software.com>
Date: Tue, 10 May 2011 09:51:16 +0200

Hi, Philip,

> Von: Philip Martin [mailto:philip.martin_at_wandisco.com]
> > To do that, I would need to introduce an additional type letter into
> > the "bindf" format, because some parameters will still need to be
> > int64_t
>
> I think that is a bad idea unless there is some way to automatically
catch
> a mismatch between an integer argument and multiple integer format
> letters. We regularly made mistakes in svn_error_createf before we
added
> the gcc stuff to do printf checking.
>
> I don't know if we can use that for bindf as I'm not sure if it would
> handle our "t" that consumes two arguments, but until we have some
sort of
> automatic checking the fewer format letters the better. While we rely
on
> manual checking the rule "all bound integers are 64bit" is easier to
> verify.

Maybe
http://gcc.gnu.org/onlinedocs/gcc/Function-Attributes.html#Function-Attr
ibutes helps (scroll down to the "format" section).
It seems that allows you to apply custom printf like checking.

Best regards

Markus Schaber

___________________________
We software Automation.

3S-Smart Software Solutions GmbH
Markus Schaber | Developer
Memminger Str. 151 | 87439 Kempten | Germany | Tel. +49-831-54031-0 |
Fax +49-831-54031-50

Email: m.schaber@3s-software.com | Web: http://www.3s-software.com
CoDeSys internet forum: http://forum.3s-software.com
Download CoDeSys sample projects:
http://www.3s-software.com/index.shtml?sample_projects

Managing Directors: Dipl.Inf. Dieter Hess, Dipl.Inf. Manfred Werner |
Trade register: Kempten HRB 6186 | Tax ID No.: DE 167014915
Received on 2011-05-10 09:51:56 CEST

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.