[mythtv] 0.27 code cleanliness

Jean-Yves Avenard jean-yves at avenard.org
Tue Oct 9 18:45:33 UTC 2012


Hi

On 9 October 2012 03:45, Gary Buhrmaster <gary.buhrmaster at gmail.com> wrote:
> Devs,
>
> It is my understand that one of the goals for 0.27 is general
> code "cleanliness", with the result that there are fewer warnings
> (with -Wall) so that any new ones might suggest something to
> look further at (right now the posture child is all the overloaded
> virtual function hidden warnings).
>
> I know that clang is not an officially supported compiler, but
> when compiling mythtv under clang, a number of warnings are
> produced of the form:
>
>
>   RTjpegN.cpp:3098:22: warning: use of GNU old-style field designator
> extension [-Wgnu-designator]
>    static mmx_t neg= { uq: (unsigned long long)0xffffffffffffffffULL };
>                        ^~~
>                        .uq =
>
> As part of general code cleanup, I can contribute patches to
> replace the var: syntax with .var syntax.  If I do so, is it likely
> that such patches will be accepted.

Be careful... I use clang, not gnu.

That's a gnutism.


More information about the mythtv-dev mailing list