[Bug 51356] Fix C warnings in the Telepathy tests library

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Jul 30 10:57:08 CEST 2012


https://bugs.freedesktop.org/show_bug.cgi?id=51356

Xavier Claessens <xclaesse at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |xclaesse at gmail.com

--- Comment #2 from Xavier Claessens <xclaesse at gmail.com> 2012-07-30 08:57:08 UTC ---
IMO fixing warnings without adding the corresponding -Wfoo is useless because
it will regress tomorrow. I don't think any C code ever considered "foo" a
const thing. That flag will just never work in the real world (but I agree it
should be considered const).

So IMO this just makes part of our code base looks uncommon. I would agree if
we decide to respect that across the whole project and force the gcc flags.

Unless tp-glib maintainer has another opinion? jonny?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
You are the assignee for the bug.



More information about the telepathy-bugs mailing list