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

[David Young <dyoung%pobox.com@localhost>]

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
better.

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

Dave

-- 
David Young             OJC Technologies
dyoung%ojctech.com@localhost      Urbana, IL * (217) 344-0444 x24