Changeset 84e74ea in mainline for uspace/srv/hid/display/window.c

Timestamp:

2020-06-08T18:30:52Z (4 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

bc492d5

Parents:

a65b0c8

Message:

Remove XXX comment from each call to display_get_gc()

These were added with the intent that each piece of code that does rendering
in the display server is augmented with a loop over all display devices.
The better solution seems to be to introduce a virtualization layer in the
form of a cloning GC that clones the rendering operations to each display
device. It makes the business code simpler and separates concerns better.
It could also allow having more types of output GCs, than just display
devices (such as an 'observer' for RFB).

File:

• uspace/srv/hid/display/window.c (modified) (3 diffs)

uspace/srv/hid/display/window.c

@@ -84 +84 @@
         bparams.rect = params->rect;
 
-        dgc = ds_display_get_gc(wnd->display); // XXX
+        dgc = ds_display_get_gc(wnd->display);
         if (dgc != NULL) {
                 rc = gfx_bitmap_create(dgc, &bparams, NULL, &wnd->bitmap);
@@ -268 +268 @@
                 return rc;
 
-        gc = ds_display_get_gc(wnd->display); // XXX
+        gc = ds_display_get_gc(wnd->display);
         if (gc != NULL) {
                 gfx_set_color(gc, color);
@@ -695 +695 @@
         errno_t rc;
 
-        dgc = ds_display_get_gc(wnd->display); // XXX
+        dgc = ds_display_get_gc(wnd->display);
         if (dgc != NULL) {
                 gfx_bitmap_params_init(&bparams);