config(),
layouts(),
layers(),
- pending(),
name_mapping(),
id_alloc{},
last_active() {
this->display->flush();
// execute pending tasks, that is layout changes etc.
- this->execute_pending();
+ //this->execute_pending();
return 0;
}
return nullptr;
}
-void App::add_task(char const *name, std::function<void()> &&f) {
- this->pending.emplace_back(std::make_pair(name, f));
-}
-
-void App::execute_pending() {
- if (!this->pending.empty()) {
- for (auto &t : this->pending) {
- logdebug("executing task '%s'", t.first);
- t.second();
- }
- this->pending.clear();
- this->controller->commit_changes();
- this->display->flush();
- }
-}
-
// _ _ _____ _
// _ __ _ __ _____ _(_) ___ __| | | ____|_ _____ _ __ | |_ ___
// | '_ \| '__/ _ \ \/ / |/ _ \/ _` | | _| \ \ / / _ \ '_ \| __/ __|
logdebug("surface_id is %u", surface_id);
- // We need to execute the surface setup after its creation.
- this->add_task("surface_set_layout",
- [surface_id, this] { this->surface_set_layout(surface_id); });
+ this->surface_set_layout(surface_id);
}
void App::surface_removed(uint32_t surface_id) {
logdebug("surface_id is %u", surface_id);
- this->add_task("remove surface ID",
- [surface_id, this] {
- this->id_alloc.remove_id(surface_id);
+ this->id_alloc.remove_id(surface_id);
- // Also remove from last_active, if found
- auto i = std::find(this->last_active.begin(),
- this->last_active.end(), surface_id);
- if (i != this->last_active.end()) {
- this->last_active.erase(i);
- }
- });
+ // Also remove from last_active, if found
+ auto i = std::find(this->last_active.begin(),
+ this->last_active.end(), surface_id);
+ if (i != this->last_active.end()) {
+ this->last_active.erase(i);
+ }
}
void App::emit_activated(char const *label) {
this->app->surface_removed(surface_id);
}
-void controller_hooks::add_task(char const *name, std::function<void()> &&f) {
- this->app->add_task(name, std::move(f));
-}
-
} // namespace wm
layer_map layers;
typedef std::pair<char const *, std::function<void()>> name_task_pair;
- std::vector<name_task_pair> pending;
typedef std::map<std::string, int> drawing_name_map;
drawing_name_map name_mapping;
char const *activate_surface(char const *drawing_name);
char const *deactivate_surface(char const *drawing_name);
- // add tasks, executed after dispatch_events()
- void add_task(char const *name, std::function<void()> &&f);
- void execute_pending();
-
// Events from the compositor we are interested in
void surface_created(uint32_t surface_id);
void surface_removed(uint32_t surface_id);
void controller::layer_destroyed(struct layer *l) {
logdebug("genivi::layer %s @ %p", __func__, this->proxy.get());
- this->chooks->add_task("remove layer", [l, this] {
- this->lprops.erase(l->id);
- this->layers.erase(l->id);
- });
+ this->lprops.erase(l->id);
+ this->layers.erase(l->id);
}
// __
if (content_state == IVI_CONTROLLER_SURFACE_CONTENT_STATE_CONTENT_REMOVED) {
// XXX is this the right thing to do?
this->chooks->surface_removed(s->id);
- this->chooks->add_task("remove surface", [this, s] {
- this->sprops.erase(s->id);
- this->surfaces.erase(s->id);
- });
+ this->sprops.erase(s->id);
+ this->surfaces.erase(s->id);
}
}