[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