clarify some things

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

diff --git a/src/app.cpp b/src/app.cpp
index f3c9234..2fd11dc 100644 (file)
--- a/src/app.cpp
+++ b/src/app.cpp
@@ -306,11 +306,12 @@ void App::surface_set_layout(uint32_t surface_id) {
 
    // configure surface to wxh dimensions
    s->set_configuration(w, h);
+   // set source reactangle, even if we should not need to set it.
+   s->set_source_rectangle(0, 0, w, h);
    // set destination to the display rectangle
    s->set_destination_rectangle(x, y, w, h);
 
-   // XXX: visibility should be determined independently of our
-   //      layer + geometry setup.
+   // XXX: The main_surface will be visible regardless
    s->set_visibility(
       surface_id == static_cast<unsigned>(this->layers.main_surface) ? 1 : 0);
    this->controller->layers[layer_id]->add_surface(s.get());
@@ -393,11 +394,6 @@ result<int> App::request_surface(char const *drawing_name) {
    auto rname = this->id_alloc[drawing_name];
    if (!rname) {
       // name does not exist yet, allocate surface id...
-      // XXX: how to allocate surface IDs?
-      // * allocate by running a counter for each layer?
-      // * allocate IDs globally, i.e. do not have layers contain
-      //   ID ranges (only define the surfaces on the layer by
-      //   role?)
       auto id = int(this->id_alloc(drawing_name));
       this->layers.add_surface(id, lid.value());