Re: [LTP] [PATCH] Annotate tst_*() helpers with __attribute__ ((format (printf, M, N)))

Mike Frysinger
2009-08-28T14:44:17Z

[-- Attachment #1.1: Type: Text/Plain, Size: 1130 bytes --]

On Friday 28 August 2009 06:11:02 Subrata Modak wrote:
> On Fri, 2009-08-28 at 05:14 -0400, Mike Frysinger wrote:
> > On Friday 28 August 2009 04:38:17 Subrata Modak wrote:
> > > Sorry. I need to revert this as it generates a huge set of warning for
> > > all the tests compiled. It should not only get fixed at test.h, but
> > > also in all tests that uses the tst_* family of functions, which
> > > obviously is too big to fix.
> >
> > nothing is too big. the proposed patch is a great idea and anything
> > preventing it from being merged should be resolved so it can go back in.
>
> Great Mike. I am already seeing you doing some changes to the headers in
> CVS. Can you also push this patch once you are done with all the
> changes. Please notify me once all are done. I plan to checkout a fresh
> one for the upcoming release ;-)

ive fixed a good chunk of the warnings now, but there's still a good number
left. maybe someone else feels like taking a crack at them, but i'm bored
now. i'll leave it up to you to apply said patch now or wait a few days so it
goes in after the next release.
-mike

[-- Attachment #1.2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 355 bytes --]

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
trial. Simplify your report design, integration and deployment - and focus on
what you do best, core application coding. Discover what's new with
Crystal Reports now. http://p.sf.net/sfu/bobj-july

[-- Attachment #3: Type: text/plain, Size: 155 bytes --]