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

Re: [Xen-devel] [PATCH] Add -Wcast-qual flag

On 21/12/06 12:51, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:

> While I'd really like to see all Xen code become const-correct (because I do
> believe this helps with code quality, like does marking read only data read
> only in the page tables), I don't think adding the warning here is
> appropriate - after all, in C this is one of the purposes of adding explicit
> casts (although I agree it's questionable to use casts for this purpose).
> And no, I don't think that many people screwing up const placement is a
> proper argument for not using const where possible.

Well, I'm inclined to take the clean-up parts of the patches but leave out
__UNCONST() and -Wcast-qual. I really dislike the compiler thinking it knows
better than an explicit cast, and casting via unsigned long is just a

 -- Keir

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.