Source-Changes-D archive

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

Re: CVS commit: src/external/bsd/mdocml



On Fri, Aug 19, 2011 at 07:54:28PM +0200, Joerg Sonnenberger wrote:
> When have you last checked the number of -Wno-* instances in base?
> Sorry, but pessimizing code to work around clearly bogus GCC warnings is
> not helpful. It doesn't make code easier to read, it doesn't improve
> code quality. It's the same kind of behavior that results in
> isalpha((int)ch) code.

Since we're still in the thread started by the mdocml change, I'll
point to the URL that Kristaps found which convinced him to accept the
diff.

http://c-faq.com/ansi/constmismatch.html

 Thomas


Home | Main Index | Thread Index | Old Index