Source-Changes-D archive

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

Re: CVS commit: src/usr.sbin/wlanctl

On Tue, Aug 16, 2011 at 04:33:46AM -0400, Christos Zoulas wrote:
> Module Name:  src
> Committed By: christos
> Date:         Tue Aug 16 08:33:46 UTC 2011
> Modified Files:
>       src/usr.sbin/wlanctl: wlanctl.c
> Log Message:
> avoid non-literal format strings

What's the purpose of this change, to quiet compiler warnings?  The old
code looked correct to me.  Wasn't it?  I had purposefully not repeated
the printf()s because I thought that it showed the intention of the code

In principle, at least, the compiler should have been able to tell that
the old code was safe.  Can't it?


David Young             OJC Technologies      Urbana, IL * (217) 344-0444 x24

Home | Main Index | Thread Index | Old Index