This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][BZ #14254] Remove wur on scanf et al.


On Tuesday 08 October 2013 11:53:31 OndÅej BÃlka wrote:
> This is another bug with simple patch. Accept it or close but we should
> not keep these in indefinite state.
> 
> https://sourceware.org/bugzilla/show_bug.cgi?id=14254

the proposed idiom is useful, but so is the warning.  it is common place for 
people to use scanf funcs w/out checking the return value which means wur is 
useful to the wider ecosystem.  imo, if you really really don't need to check 
the return value (because you're using a trick like %n), then it's easy enough 
to do:
	if (scanf(....)) {/* we use %n for checking */}
which means i'm in favor of closing the bug as WONTFIX.  

we could add a note to the documentation on the topic in the scanf section, 
although i'm not sure how useful that'll be in practice.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]