[PATCH] Override modules list and don't always load desktop-shell.so
Bill Spitzak
spitzak at gmail.com
Mon Apr 1 10:10:25 PDT 2013
Emilio Pozuelo Monfort wrote:
> I would rather do something like this to simplify the code (at least it looks
> clearer to me but YMMV):
>
> if (load_modules(ec, option_modules != NULL ? option_modules : modules,
> &argc, argv, config_file) < 0)
> goto out;
That's better than my suggestion.
I'm not thrilled with using "x != NULL" rather than just "x" as a
conditional, to me I find it hard to read as I almost always read it
inverted (ie "x == NULL") unless I look carefully. However that may be
me and it may not match wayland coding standards?
More information about the wayland-devel
mailing list