On Fri, Jun 05, 2015 at 03:30:21PM +1000, Peter Hutterer wrote: > > Sparked by Jon's patchset, this tidies up the doxygen bits a little bit. > Turns out doxygen will simply take the last option, so rather than hiding > the few manual configurations in the massive file we can accumulate them in > one spot at the end of the file. Which makes it a lot clearer what we want > to have set. > > The second patch simply removes the default configs, relying on doxygen to > know what the default is. In theory this could trip us up if future doxygen > versions change defaults significantly, in practice I doubt we'll care (and > if we do we can just set that option manually). > > Feel free to squash the two together if the end result is what we want.
Yeah, was thinking similarly myself when looking at the doxygen setup. For both patches: Reviewed-by: Bryce Harrington <br...@osg.samsung.com> > Cheers, > Peter > _______________________________________________ > wayland-devel mailing list > wayland-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel