From 7678bfc942bd501f8e121ff153d1852d73578257 Mon Sep 17 00:00:00 2001
From: Marcus Fritzsch <marcus_fritzsch@mentor.com>
Date: Wed, 12 Jul 2017 16:40:29 +0200
Subject: main: move most of main to app, separate ownership

* Move everything apart from the display connection itself to App.
* Everything depending on the display however is owned by App.
* Dispatch events for the application in App::dispatch_events()
That is, display_dispatch and local stuff e.g. execute_pending().

Signed-off-by: Marcus Fritzsch <marcus_fritzsch@mentor.com>
---
 src/main.cpp | 108 +++--------------------------------------------------------
 1 file changed, 5 insertions(+), 103 deletions(-)

(limited to 'src/main.cpp')

diff --git a/src/main.cpp b/src/main.cpp
index 23b1b4d..6de8597 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -14,11 +14,9 @@ extern "C" {
 namespace {
 struct afb_instance {
    std::unique_ptr<wl::display> display;
-   std::unique_ptr<genivi::controller> controller;
-   std::vector<std::unique_ptr<wl::output>> outputs;
    wm::App app;
 
-   afb_instance() : display{new wl::display}, controller{nullptr}, outputs{}, app{} {}
+   afb_instance() : display{new wl::display}, app{this->display.get()} {}
 
    int init();
 };
@@ -30,90 +28,7 @@ int afb_instance::init() {
       return -1;
    }
 
-   this->display->r.add_global_handler("wl_output", [](wl_registry *r,
-                                                       uint32_t name,
-                                                       uint32_t v) {
-      g_afb_instance->outputs.emplace_back(std::make_unique<wl::output>(r, name, v));
-   });
-
-   this->display->r.add_global_handler(
-      "ivi_controller", [](wl_registry *r, uint32_t name, uint32_t v) {
-         g_afb_instance->controller =
-            std::make_unique<genivi::controller>(r, name, v);
-
-         // XXX: This protocol needs the output, so lets just add our mapping
-         // here...
-         g_afb_instance->controller->add_proxy_to_id_mapping(
-            g_afb_instance->outputs.back()->proxy.get(),
-            wl_proxy_get_id(reinterpret_cast<struct wl_proxy *>(
-               g_afb_instance->outputs.back()->proxy.get())));
-      });
-
-   // First level objects
-   this->display->roundtrip();
-   // Second level objects
-   this->display->roundtrip();
-   // Third level objects
-   this->display->roundtrip();
-
-   // Init the app's members
-   this->app.display = this->display.get();
-   this->app.controller = this->controller.get();
-
-   return 0;
-}
-
-//  _       _ _       _                         _    ____
-// (_)_ __ (_) |_    | | __ _ _   _  ___  _   _| |_ / /\ \
-// | | '_ \| | __|   | |/ _` | | | |/ _ \| | | | __| |  | |
-// | | | | | | |_    | | (_| | |_| | (_) | |_| | |_| |  | |
-// |_|_| |_|_|\__|___|_|\__,_|\__, |\___/ \__,_|\__| |  | |
-//              |_____|       |___/                 \_\/_/
-char const *init_layout() {
-   if (!g_afb_instance->controller) {
-      return "ivi_controller global not available";
-   }
-
-   if (g_afb_instance->outputs.empty()) {
-      return "no output was set up!";
-   }
-
-   auto &c = g_afb_instance->controller;
-
-   auto &o = g_afb_instance->outputs.front();
-   auto &s = c->screens.begin()->second;
-   auto &layers = c->layers;
-
-   // XXX: Write output dimensions to ivi controller...
-   c->output_size = genivi::size{uint32_t(o->width), uint32_t(o->height)};
-
-   // Clear scene
-   layers.clear();
-
-   // Clear screen
-   s->clear();
-
-   // Setup our dummy scene...
-   c->layer_create(100, 0, 0);   // bottom layer, anything else
-   c->layer_create(1000, 0, 0);  // top layer, mandelbrot
-
-   auto &l100 = c->layers[100];
-   auto &l1k = c->layers[1000];
-
-   // Set layers fullscreen
-   l100->set_destination_rectangle(0, 0, o->width, o->height);
-   l1k->set_destination_rectangle(0, 0, o->width, o->height);
-   l100->set_visibility(1);
-   l1k->set_visibility(1);
-
-   // Add layers to screen
-   s->set_render_order({100, 1000});
-
-   c->commit_changes();
-
-   g_afb_instance->display->flush();
-
-   return nullptr;
+   return this->app.init();
 }
 
 int display_event_callback(sd_event_source *evs, int fd, uint32_t events,
@@ -126,17 +41,9 @@ int display_event_callback(sd_event_source *evs, int fd, uint32_t events,
    }
 
    if (events & EPOLLIN) {
-      int ret = g_afb_instance->display->dispatch();
-      if (ret == -1) {
-         logerror("wl_display_dipatch() returned error %d",
-                   g_afb_instance->display->get_error());
-         goto error;
-      }
-      g_afb_instance->display->flush();
-
-      // execute pending tasks, that is layout changes etc.
-      g_afb_instance->controller->execute_pending();
-      g_afb_instance->display->roundtrip();
+       if (g_afb_instance->app.dispatch_events() == -1) {
+          goto error;
+       }
    }
 
    return 0;
@@ -171,11 +78,6 @@ int binding_init_() {
       goto error;
    }
 
-   if (char const *e = init_layout()) {
-      logerror("Could not init layout: %s", e);
-      goto error;
-   }
-
    {
       int ret = sd_event_add_io(afb_daemon_get_event_loop(), nullptr,
                                 g_afb_instance->display->get_fd(), EPOLLIN,
-- 
cgit 1.2.3-korg