[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