On Thu, Feb 26, 2015 at 1:55 PM, Michał Górny <[email protected]> wrote: > Support reading multiple 'Files' sections in configuration, > concatenating the resulting paths. This makes it possible to add > ModulePaths and FontPaths within xorg.conf.d/ files without interfering > with user-provided xorg.conf. Gentoo needs this to support using > replacement xorg modules provided by proprietary video drivers. > > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=88003 > Signed-off-by: Michał Górny <[email protected]> > --- > [not subscribed, please CC me on replies] > > hw/xfree86/parser/Files.c | 8 ++++++-- > hw/xfree86/parser/configProcs.h | 2 +- > hw/xfree86/parser/read.c | 2 +- > 3 files changed, 8 insertions(+), 4 deletions(-) > > diff --git a/hw/xfree86/parser/Files.c b/hw/xfree86/parser/Files.c > index 849bf92..5cc3ec7 100644 > --- a/hw/xfree86/parser/Files.c > +++ b/hw/xfree86/parser/Files.c > @@ -76,14 +76,18 @@ static xf86ConfigSymTabRec FilesTab[] = { > #define CLEANUP xf86freeFiles > > XF86ConfFilesPtr > -xf86parseFilesSection(void) > +xf86parseFilesSection(XF86ConfFilesPtr ptr) > { > int i, j; > int k, l; > char *str; > int token; > > - parsePrologue(XF86ConfFilesPtr, XF86ConfFilesRec) > + if (!ptr) { > + if( (ptr=calloc(1,sizeof(XF86ConfFilesRec))) == NULL ) {
Style issues: - space between if and ( - no spaces between parentheses and the conditional expression - spaces around = - space after comma On IRC, Keith says the patch has a memory leak, but that there are some existing ones in this code already. I'm not sure if he wants you to fix the one you're adding or not worry about it. _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
