On Thu, Nov 24, 2011 at 10:49:43AM +0100, Holger Hans Peter Freyther wrote:
On 11/24/2011 10:36 AM, Alexander Huemer wrote:
Steve and Holger raised some reasonable concerns
on some of the other
patches, it seems I was quite unconcentrated and they are mostly crap.
crap? no, not at all. I agree that we should get the number of warnings down,
sometimes a warning just needs something else than the straight forward fix.
Compiler warnings are of course controversial.
I personally think that osmocom-bb and friends are fine pieces of code
and deserve a kind of "polished" look when compiling.
Some warnings are also quite strong indications of bugs like "... makes
pointer from integer without a cast" and things like that.
I myself don't know how to handle e.g. "discards qualifiers from pointer
target type" when the constness of lvalue and rvalue don't match.
Maybe it doesn't actually matter so much and I am just pedantic.