Dave Airlie <[email protected]> writes: > +int > +AddGPUScreen(Bool (*pfnInit) (ScreenPtr /*pScreen */ , > + int /*argc */ , > + char ** /*argv */ > + ), > + int argc, char **argv) > +{ > + int i; > + ScreenPtr pScreen; > + Bool ret; > + > + i = screenInfo.numGPUScreens; > + if (i == MAXSCREENS) > + return -1; > + > + pScreen = (ScreenPtr) calloc(1, sizeof(ScreenRec)); > + if (!pScreen) > + return -1; > + > + ret = init_screen(pScreen, i, TRUE); > + if (ret) { > + free(pScreen); > + return ret; > + }
(same complaint as last patch -- init_screen returns int, not Bool) > ScreenPtr screens[MAXSCREENS]; > + int numGPUScreens; > + ScreenPtr gpuscreens[MAXSCREENS]; These should probably be different constants. -- [email protected]
pgp6LNTjNjRtR.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
