On Sun, 17 Apr 2016 17:14:17 +0300 Giulio Camuffo <[email protected]> wrote:
> Reviewed-by: Giulio Camuffo <[email protected]> > > 2016-04-16 6:28 GMT+03:00 Bryce Harrington <[email protected]>: > > From: Benoit Gschwind <[email protected]> > > > > refactor configuration API of headless-backend > > > > Signed-off-by: Bryce Harrington <[email protected]> > > Reviewed-by: Pekka Paalanen <[email protected]> > > > > v6: > > - Define version number in the header > > - Don't use leading underscores in header guards > > - Add stub config_init_to_defaults() > > - Allocate config on stack > > - Drop unused display_name parameter > > - Add error message when config is invalid > > - Install compositor-headless.h and list it in headless-backend sources > > v5: > > - Update to current trunk > > - Fixed typo 'struct weston_wayland_backend_config' > > - Dropped unused variables > > - Dropped weston_headless_backend_config_create() in favor of > > directly zalloc'ing the object > > - Dropped weston_headless_backend_load() in favor of the more > > generalized load_backend_new(). > > - Dropped typedef from header > > - Restored use of 'backend_init' entry point > > - Backend_init() takes a base weston_backend_config object > > - Renamed 'param' to 'config' in a few places for consistency > > - Renamed 'headless_options' variable to 'options for consistency > > - Version the base struct > > - Free config on error > > - Don't free config during backend_init normal operations > > - Adjust header ordering > > - Make header guard naming consistent with other headers > > - Light reformatting for code style and consistency with other > > backend config patches > > > > Signed-off-by: Bryce Harrington <[email protected]> > > --- > > Makefile.am | 3 ++ > > src/compositor-headless.c | 74 > > +++++++++++++++++++++-------------------------- > > src/compositor-headless.h | 53 +++++++++++++++++++++++++++++++++ > > src/main.c | 45 ++++++++++++++++++++++++++-- > > 4 files changed, 132 insertions(+), 43 deletions(-) > > create mode 100644 src/compositor-headless.h > > Hi, since this patch was ready by both Giulio's and my opinion, I decided to rebase it to apply upstream before the drm or x11 backend conversions. I also took the headless part of the patch "[PATCH weston v6 11/12] Enforce destruction of all backend config objects after initialization" and squashed it into this patch. Pushed: 20b66c3..3c53094 master -> master Thanks, pq
pgparL4isIJB6.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
