applications/i3
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Implement double-buffering to get rid of flickering

Browse Source
This commit is contained in:
Axel Wagner 2010-08-21 13:09:34 +02:00
parent 60da522e13
commit fe24eb748c
2 changed files with 113 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
i3bar/include/outputs.h
View File

@ -43,6 +43,7 @@ struct i3_output {
rect rect; /* The rect (relative to the root-win) */ rect rect; /* The rect (relative to the root-win) */
xcb_window_t bar; /* The id of the bar of the output */ xcb_window_t bar; /* The id of the bar of the output */
xcb_pixmap_t buffer; /* An extra pixmap for double-buffering */
xcb_gcontext_t bargc; /* The graphical context of the bar */ xcb_gcontext_t bargc; /* The graphical context of the bar */
struct ws_head *workspaces; /* The workspaces on this output */ struct ws_head *workspaces; /* The workspaces on this output */

150
i3bar/src/xcb.c
View File

@ -221,10 +221,11 @@ void init_xcb(char *fontname) {
/* We load and allocate the font */ /* We load and allocate the font */
xcb_font = xcb_generate_id(xcb_connection); xcb_font = xcb_generate_id(xcb_connection);
xcb_open_font(xcb_connection, xcb_void_cookie_t open_font_cookie;
xcb_font, open_font_cookie = xcb_open_font_checked(xcb_connection,
strlen(fontname), xcb_font,
fontname); strlen(fontname),
fontname);
/* We also need the fontheight to configure our bars accordingly */ /* We also need the fontheight to configure our bars accordingly */
xcb_list_fonts_with_info_cookie_t cookie; xcb_list_fonts_with_info_cookie_t cookie;
@ -249,6 +250,14 @@ void init_xcb(char *fontname) {
/* Now we get the atoms and save them in a nice data-structure */ /* Now we get the atoms and save them in a nice data-structure */
get_atoms(); get_atoms();
xcb_generic_error_t *err = xcb_request_check(xcb_connection,
open_font_cookie);
if (err != NULL) {
printf("ERROR: Could not open font! XCB-Error-Code: %d\n", err->error_code);
exit(EXIT_FAILURE);
}
/* Now we calculate the font-height */ /* Now we calculate the font-height */
xcb_list_fonts_with_info_reply_t *reply; xcb_list_fonts_with_info_reply_t *reply;
reply = xcb_list_fonts_with_info_reply(xcb_connection, reply = xcb_list_fonts_with_info_reply(xcb_connection,
@ -333,46 +342,81 @@ void reconfig_windows() {
printf("Creating Window for output %s\n", walk->name); printf("Creating Window for output %s\n", walk->name);
walk->bar = xcb_generate_id(xcb_connection); walk->bar = xcb_generate_id(xcb_connection);
walk->buffer = xcb_generate_id(xcb_connection);
mask = XCB_CW_BACK_PIXEL | XCB_CW_EVENT_MASK; mask = XCB_CW_BACK_PIXEL | XCB_CW_EVENT_MASK;
/* Black background */ /* Black background */
values[0] = xcb_screens->black_pixel; values[0] = xcb_screens->black_pixel;
/* The events we want to receive */ /* The events we want to receive */
values[1] = XCB_EVENT_MASK_EXPOSURE | values[1] = XCB_EVENT_MASK_EXPOSURE |
XCB_EVENT_MASK_BUTTON_PRESS; XCB_EVENT_MASK_BUTTON_PRESS;
xcb_create_window(xcb_connection,
xcb_screens->root_depth, xcb_void_cookie_t win_cookie = xcb_create_window_checked(xcb_connection,
walk->bar, xcb_screens->root_depth,
xcb_root, walk->bar,
walk->rect.x, walk->rect.y, xcb_root,
walk->rect.w, font_height + 6, walk->rect.x, walk->rect.y,
1, walk->rect.w, font_height + 6,
XCB_WINDOW_CLASS_INPUT_OUTPUT, 1,
xcb_screens->root_visual, XCB_WINDOW_CLASS_INPUT_OUTPUT,
mask, xcb_screens->root_visual,
values); mask,
values);
xcb_void_cookie_t pm_cookie = xcb_create_pixmap_checked(xcb_connection,
xcb_screens->root_depth,
walk->buffer,
walk->bar,
walk->rect.w,
walk->rect.h);
/* We want dock-windows (for now) */ /* We want dock-windows (for now) */
xcb_change_property(xcb_connection, xcb_void_cookie_t prop_cookie = xcb_change_property(xcb_connection,
XCB_PROP_MODE_REPLACE, XCB_PROP_MODE_REPLACE,
walk->bar, walk->bar,
atoms[_NET_WM_WINDOW_TYPE], atoms[_NET_WM_WINDOW_TYPE],
atoms[ATOM], atoms[ATOM],
32, 32,
1, 1,
(unsigned char*) &atoms[_NET_WM_WINDOW_TYPE_DOCK]); (unsigned char*) &atoms[_NET_WM_WINDOW_TYPE_DOCK]);
/* We also want a graphics-context (the "canvas" on which we draw) */ /* We also want a graphics-context (the "canvas" on which we draw) */
walk->bargc = xcb_generate_id(xcb_connection); walk->bargc = xcb_generate_id(xcb_connection);
mask = XCB_GC_FONT; mask = XCB_GC_FONT;
values[0] = xcb_font; values[0] = xcb_font;
xcb_create_gc(xcb_connection,
walk->bargc, xcb_void_cookie_t gc_cookie = xcb_create_gc_checked(xcb_connection,
walk->bar, walk->bargc,
mask, walk->bar,
values); mask,
values);
/* We finally map the bar (display it on screen) */ /* We finally map the bar (display it on screen) */
xcb_map_window(xcb_connection, walk->bar); xcb_void_cookie_t map_cookie = xcb_map_window_checked(xcb_connection, walk->bar);
if ((err = xcb_request_check(xcb_connection, win_cookie)) != NULL) {
printf("ERROR: Could not create Window. XCB-errorcode: %d\n", err->error_code);
exit(EXIT_FAILURE);
}
if ((err = xcb_request_check(xcb_connection, pm_cookie)) != NULL) {
printf("ERROR: Could not create Pixmap. XCB-errorcode: %d\n", err->error_code);
exit(EXIT_FAILURE);
}
if ((err = xcb_request_check(xcb_connection, prop_cookie)) != NULL) {
printf("ERROR: Could not set dock mode. XCB-errorcode: %d\n", err->error_code);
exit(EXIT_FAILURE);
}
if ((err = xcb_request_check(xcb_connection, gc_cookie)) != NULL) {
printf("ERROR: Could not create graphical context. XCB-errorcode: %d\n", err->error_code);
exit(EXIT_FAILURE);
}
if ((err = xcb_request_check(xcb_connection, map_cookie)) != NULL) {
printf("ERROR: Could not map window. XCB-errorcode: %d\n", err->error_code);
exit(EXIT_FAILURE);
}
} else { } else {
/* We already have a bar, so we just reconfigure it */ /* We already have a bar, so we just reconfigure it */
mask = XCB_CONFIG_WINDOW_X | mask = XCB_CONFIG_WINDOW_X |
@ -384,11 +428,32 @@ void reconfig_windows() {
values[2] = walk->rect.w; values[2] = walk->rect.w;
values[3] = font_height + 6; values[3] = font_height + 6;
printf("Reconfiguring Window for output %s to %d,%d\n", walk->name, values[0], values[1]); printf("Reconfiguring Window for output %s to %d,%d\n", walk->name, values[0], values[1]);
xcb_configure_window(xcb_connection,
walk->bar, xcb_void_cookie_t cfg_cookie = xcb_configure_window_checked(xcb_connection,
mask, walk->bar,
values); mask,
} values);
xcb_free_pixmap(xcb_connection, walk->buffer);
walk->buffer = xcb_generate_id(xcb_connection);
xcb_void_cookie_t pm_cookie = xcb_create_pixmap_checked(xcb_connection,
xcb_screens->root_depth,
walk->buffer,
walk->bar,
walk->rect.w,
walk->rect.h);
if ((err = xcb_request_check(xcb_connection, cfg_cookie)) != NULL) {
printf("ERROR: Could not reconfigure window. XCB-errorcode: %d\n", err->error_code);
exit(EXIT_FAILURE);
}
if ((err = xcb_request_check(xcb_connection, pm_cookie)) != NULL) {
printf("ERROR: Could not create Pixmap. XCB-errorcode: %d\n", err->error_code);
exit(EXIT_FAILURE);
}
}
} }
} }
@ -415,7 +480,7 @@ void draw_bars() {
&color); &color);
xcb_rectangle_t rect = { 0, 0, outputs_walk->rect.w, font_height + 6 }; xcb_rectangle_t rect = { 0, 0, outputs_walk->rect.w, font_height + 6 };
xcb_poly_fill_rectangle(xcb_connection, xcb_poly_fill_rectangle(xcb_connection,
outputs_walk->bar, outputs_walk->buffer,
outputs_walk->bargc, outputs_walk->bargc,
1, 1,
&rect); &rect);
@ -437,7 +502,7 @@ void draw_bars() {
xcb_void_cookie_t cookie; xcb_void_cookie_t cookie;
cookie = xcb_image_text_16(xcb_connection, cookie = xcb_image_text_16(xcb_connection,
glyph_count, glyph_count,
outputs_walk->bar, outputs_walk->buffer,
outputs_walk->bargc, outputs_walk->bargc,
outputs_walk->rect.w - get_string_width(text, glyph_count) - 4, outputs_walk->rect.w - get_string_width(text, glyph_count) - 4,
font_height + 1, font_height + 1,
@ -470,7 +535,7 @@ void draw_bars() {
&color); &color);
xcb_rectangle_t rect = { i + 1, 1, ws_walk->name_width + 8, font_height + 4 }; xcb_rectangle_t rect = { i + 1, 1, ws_walk->name_width + 8, font_height + 4 };
xcb_poly_fill_rectangle(xcb_connection, xcb_poly_fill_rectangle(xcb_connection,
outputs_walk->bar, outputs_walk->buffer,
outputs_walk->bargc, outputs_walk->bargc,
1, 1,
&rect); &rect);
@ -481,13 +546,22 @@ void draw_bars() {
&color); &color);
xcb_image_text_16(xcb_connection, xcb_image_text_16(xcb_connection,
ws_walk->name_glyphs, ws_walk->name_glyphs,
outputs_walk->bar, outputs_walk->buffer,
outputs_walk->bargc, outputs_walk->bargc,
i + 5, font_height + 1, i + 5, font_height + 1,
ws_walk->ucs2_name); ws_walk->ucs2_name);
i += 10 + ws_walk->name_width; i += 10 + ws_walk->name_width;
} }
xcb_copy_area(xcb_connection,
outputs_walk->buffer,
outputs_walk->bar,
outputs_walk->bargc,
0, 0,
0, 0,
outputs_walk->rect.w,
outputs_walk->rect.h);
i = 0; i = 0;
} }
} }