clang-tidy

[staging/windowmanager.git] / src / app.cpp

diff --git a/src/app.cpp b/src/app.cpp
index c38a7f4..f3c9234 100644 (file)
--- a/src/app.cpp
+++ b/src/app.cpp
@@ -28,10 +28,10 @@
 
 #include <json-c/json.h>
 
+#include <algorithm>
 #include <bits/signum.h>
 #include <csignal>
 #include <fstream>
-#include <algorithm>
 #include <json.hpp>
 
 namespace wm {
@@ -129,8 +129,7 @@ App::App(wl::display *d)
      layers(),
      pending(),
      name_mapping(),
-     id_alloc{}
-{
+     id_alloc{} {
    assert(g_app == nullptr);
    g_app = this;
 
@@ -284,7 +283,8 @@ void App::surface_set_layout(uint32_t surface_id) {
    }
 
    uint32_t layer_id = o_layer_id.value();
-   logdebug("surface_set_layout for surface %u on layer %u", surface_id, layer_id);
+   logdebug("surface_set_layout for surface %u on layer %u", surface_id,
+            layer_id);
 
    auto const &layer = this->layers.get_layer(layer_id);
    auto rect = layer.value().rect;
@@ -311,7 +311,8 @@ void App::surface_set_layout(uint32_t surface_id) {
 
    // XXX: visibility should be determined independently of our
    //      layer + geometry setup.
-   s->set_visibility(surface_id == (unsigned)this->layers.main_surface ? 1 : 0);
+   s->set_visibility(
+      surface_id == static_cast<unsigned>(this->layers.main_surface) ? 1 : 0);
    this->controller->layers[layer_id]->add_surface(s.get());
 
    logdebug("Surface %u now on layer %u with rect { %d, %d, %d, %d }",
@@ -390,7 +391,7 @@ result<int> App::request_surface(char const *drawing_name) {
    }
 
    auto rname = this->id_alloc[drawing_name];
-   if (! rname) {
+   if (!rname) {
       // name does not exist yet, allocate surface id...
       // XXX: how to allocate surface IDs?
       // * allocate by running a counter for each layer?
@@ -400,10 +401,9 @@ result<int> App::request_surface(char const *drawing_name) {
       auto id = int(this->id_alloc(drawing_name));
       this->layers.add_surface(id, lid.value());
 
-      // XXX: setup the main_surface id if we registered HomeScreen
       // XXX: you should fix this!
       if (!this->layers.main_surface_name.empty() &&
-           this->layers.main_surface_name == drawing_name) {
+          this->layers.main_surface_name == drawing_name) {
          this->layers.main_surface = id;
          this->activate_surface(id);
          logdebug("Set main_surface id to %u", id);
@@ -416,11 +416,12 @@ result<int> App::request_surface(char const *drawing_name) {
    return Err<int>("Surface already present");
 }
 
-char const* App::activate_surface(char const *drawing_name) {
+char const *App::activate_surface(char const *drawing_name) {
    auto osid = this->id_alloc[drawing_name];
 
    if (osid) {
-      logdebug("ativate surface with name %s and id %u", drawing_name, osid.value());
+      logdebug("ativate surface with name %s and id %u", drawing_name,
+               osid.value());
       this->activate_surface(osid.value());
       return nullptr;
    }
@@ -439,7 +440,7 @@ binding_api::result_type binding_api::request_surface(
    char const *drawing_name) {
    auto r = this->app->request_surface(drawing_name);
    if (r.is_err()) {
-      return Err<json_object*>(r.unwrap_err());
+      return Err<json_object *>(r.unwrap_err());
    }
    return Ok(json_object_new_int(r.unwrap()));
 }
@@ -448,7 +449,7 @@ binding_api::result_type binding_api::activate_surface(
    char const *drawing_name) {
    logdebug("%s drawing_name %s", __func__, drawing_name);
    auto r = this->app->activate_surface(drawing_name);
-   if (r) {
+   if (r != nullptr) {
       return Err<json_object *>(r);
    }
    return Ok(json_object_new_object());
@@ -488,7 +489,7 @@ binding_api::result_type binding_api::debug_terminate() {
 binding_api::result_type binding_api::demo_activate_surface(
    uint32_t surfaceid) {
    char const *e = this->app->activate_surface(surfaceid);
-   if (e) {
+   if (e != nullptr) {
       return Err<json_object *>(e);
    }
    return Ok(json_object_new_object());