<p dir="ltr"><br>
Em 08/05/2014 17:03, "Keith Packard" <<a href="mailto:keithp@keithp.com">keithp@keithp.com</a>> escreveu:<br>
><br>
> Peter Hutterer <<a href="mailto:peter.hutterer@who-t.net">peter.hutterer@who-t.net</a>> writes:<br>
><br>
> > The last two aren't rev-by. If you want to add the tag, feel free to add the<br>
> > tag after merging.<br>
><br>
> I've reviewed both of these; feel free to add as you please, or send<br>
> along and I'll add that when I merge. However, I'm getting a bunch of<br>
> new warnings now and those will need to be fixed before I can push out<br>
> the merge...<br>
><br>
> > Laércio de Sousa (2):<br>
> >       xfree86: allow fallback to PCI bus probe for graphics devices on non-seat0 X servers (#66851)<br>
> >       xfree86: add short description about MatchSeat key in xorg.conf man page<br>
> ><br>
> > Oleg Samarin (1):<br>
> >       xfree86: add new key MatchSeat to xorg.conf sections "Device", "Screen", and "ServerLayout"<br>
><br>
> I think it's this patch which generates several new warnings:<br>
><br>
> xf86Config.c: In function 'configLayout':<br>
> xf86Config.c:1596:29: warning: declaration of 'l' shadows a previous local [-Wshadow]<br>
>          FIND_SUITABLE (XF86ConfScreenPtr, xf86configptr->conf_screen_lst, screen);<br>
>                              ^<br>
> xf86Config.c:1484:23: warning: shadowed declaration is here [-Wshadow]<br>
>      XF86ConfLayoutPtr l;<br>
>                        ^<br>
> xf86Config.c:1596:172: warning: suggest parentheses around '&&' within '||' [-Wparentheses]<br>
>          FIND_SUITABLE (XF86ConfScreenPtr, xf86configptr->conf_screen_lst, screen);<br>
>                                                                                                                                                                             ^<br>
> xf86Config.c: In function 'configScreen':<br>
> xf86Config.c:1838:172: warning: suggest parentheses around '&&' within '||' [-Wparentheses]<br>
>          FIND_SUITABLE (XF86ConfDevicePtr, xf86configptr->conf_device_lst, conf_screen->scrn_device);<br>
>                                                                                                                                                                             ^<br>
> xf86Config.c: In function 'xf86HandleConfigFile':<br>
> xf86Config.c:2446:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]<br>
>      XF86ConfLayoutPtr layout;<br>
>      ^<br>
> xf86Config.c:2448:168: warning: suggest parentheses around '&&' within '||' [-Wparentheses]<br>
>      FIND_SUITABLE(XF86ConfLayoutPtr, xf86configptr->conf_layout_lst, layout);<br>
>                                                                                                                                                                         ^<br>
> xf86Config.c:2456:172: warning: suggest parentheses around '&&' within '||' [-Wparentheses]<br>
>          FIND_SUITABLE (XF86ConfScreenPtr, xf86configptr->conf_screen_lst, screen);<br>
>                                                                                                                                                                             ^</p>
<p dir="ltr">These warnings are all related to a malformed FIND_SUITABLE macro expansion. I can fix it tomorrow.</p>
<p dir="ltr">> > Peter Hutterer (3):<br>
> >       systemd-logind: let the logind code decided whether to close an fd<br>
> >       mi: start calculating the sprite trace from the first child of root<br>
> >       Xi: don't copy a DeviceEvent into an InternalEvent<br>
><br>
> --<br>
> <a href="mailto:keith.packard@intel.com">keith.packard@intel.com</a><br>
><br>
> _______________________________________________<br>
> <a href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development<br>
> Archives: <a href="http://lists.x.org/archives/xorg-devel">http://lists.x.org/archives/xorg-devel</a><br>
> Info: <a href="http://lists.x.org/mailman/listinfo/xorg-devel">http://lists.x.org/mailman/listinfo/xorg-devel</a><br>
</p>