Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

re: CVS commit: src/sys/dev/pci



> On Sat, Sep 01, 2012 at 02:08:28AM +0000, Matt Thomas wrote:
> > Module Name:        src
> > Committed By:       matt
> > Date:               Sat Sep  1 02:08:28 UTC 2012
> > 
> > Modified Files:
> >     src/sys/dev/pci: if_wm.c
> > 
> > Log Message:
> > Shut up gcc about some uninitialized variables.
> 
> 
> Hum, gcc is wrong here, cmdlen and fields are only used if do_csum == true,
> and do_csum cannot be changed between setting it to false and testing it.
> 
> Is it dependant on compiler flags ?

FWIW, usually these sorts of problems only appear at -O3.  it's the
main reason i stopped using -O3 in general.


.mrg.


Home | Main Index | Thread Index | Old Index