xfree86: fix warnings after MatchSeat patch

This patch fixes some compile warnings that arise after
commit 7070ebeeba
(xfree86: add new key MatchSeat to xorg.conf sections "Device", "Screen", and "ServerLayout")
available at git repository
git://people.freedesktop.org/~whot/xserver for-keith

Signed-off-by: Keith Packard <keithp@keithp.com>
This commit is contained in:
Laércio de Sousa 2014-05-09 09:09:16 -03:00 committed by Keith Packard
parent bbfed454b2
commit fb24ac0a2c

View File

@ -232,16 +232,17 @@ xf86ValidateFontPath(char *path)
return tmp_path; return tmp_path;
} }
#define FIND_SUITABLE(pointertype, listhead, ptr) \ #define FIND_SUITABLE(pointertype, listhead, ptr) \
{ \ do { \
pointertype l, p; \ pointertype _l, _p; \
\ \
for (l = listhead, p = NULL; !p && l; l = (pointertype) l->list.next) { \ for (_l = (listhead), _p = NULL; !_p && _l; _l = (pointertype)_l->list.next) { \
if (! l->match_seat || SeatId && xf86nameCompare(l->match_seat, SeatId) == 0) \ if (!_l->match_seat || (SeatId && xf86nameCompare(_l->match_seat, SeatId) == 0)) \
p = l; \ _p = _l; \
} \ } \
ptr = p; \ \
} (ptr) = _p; \
} while(0)
/* /*
* use the datastructure that the parser provides and pick out the parts * use the datastructure that the parser provides and pick out the parts
@ -2368,6 +2369,7 @@ xf86HandleConfigFile(Bool autoconfig)
const char *scanptr; const char *scanptr;
Bool singlecard = 0; Bool singlecard = 0;
Bool implicit_layout = FALSE; Bool implicit_layout = FALSE;
XF86ConfLayoutPtr layout;
if (!autoconfig) { if (!autoconfig) {
char *filename, *dirname, *sysdirname; char *filename, *dirname, *sysdirname;
@ -2443,8 +2445,6 @@ xf86HandleConfigFile(Bool autoconfig)
*/ */
/* First check if a layout section is present, and if it is valid. */ /* First check if a layout section is present, and if it is valid. */
XF86ConfLayoutPtr layout;
FIND_SUITABLE(XF86ConfLayoutPtr, xf86configptr->conf_layout_lst, layout); FIND_SUITABLE(XF86ConfLayoutPtr, xf86configptr->conf_layout_lst, layout);
if (layout == NULL || xf86ScreenName != NULL) { if (layout == NULL || xf86ScreenName != NULL) {
XF86ConfScreenPtr screen; XF86ConfScreenPtr screen;