[pvrusb2] New driver snapshot: pvrusb2-mci-20080210
roger
roger at eskimo.com
Mon Feb 11 01:18:01 CST 2008
On Sun, 2008-02-10 at 23:15 -0600, Mike Isely wrote:
> On Sun, 10 Feb 2008, Mark Goldberg wrote:
>
> > On Feb 10, 2008 9:49 PM, Mike Isely <isely at isely.net> wrote:
> > > On Sun, 10 Feb 2008, Mike Isely wrote:
> > > I just looked closer at the line in question. I thought initially this
> > > was a prototype issue, but it's complaining about a printf format.
> > > That suggests a possible compiler difference. What gcc version are you
> > > using? I'll bet you should be getting this warning with older snapshots
> > > as well.
> >
> > gcc (GCC) 4.1.2 20070925 (Red Hat 4.1.2-33)
>
> And what I have here is:
>
> gcc version 4.1.2 20061115 (prerelease) (Debian 4.1.1-21)
>
> The warning is happening because I'm passing a size_t to * width
> parameter (e.g. "%*s" where "*" indicates the width is passed as an
> argument to the call). This is really a pretty harmless thing, but a
> cast to int there is probably the right answer. I will put that into
> the source tree now. I have *no* idea why I'm not seeing this warning
> here.
>
> -Mike
For what it's worth, I remember this size_t cast to int issue quite well
with LinuxBios. About a year ago, they started converting their size_t
to int after the compiler started complaining.
For kicks, I'm using Gentoo gcc-4.1.2 now.
My install history states I compiled this gcc version:
Fri May 18 19:53:55 2007 >>> sys-devel/gcc-4.1.2
Prior, I think I was on gcc-4.1.1 and guessing no prerelease either.
Guess this explains why some devs on LinuxBios were still actively using
size_t during this time period!
--
Roger
http://www.eskimo.com/~roger/index.html
Key fingerprint = 8977 A252 2623 F567 70CD 1261 640F C963 1005 1D61
Sun Feb 10 22:12:47 AKST 2008
More information about the pvrusb2
mailing list