Improve debug

diff --git a/modules/video_output/xcb/events.c b/modules/video_output/xcb/events.c
index 0f09885ef287d85d663d8721bd7b368a0e85f2ee..908f8116c5ca9f01656a23aa28b06b30df7ca000 100644 (file)
--- a/modules/video_output/xcb/events.c
+++ b/modules/video_output/xcb/events.c
@@ -110,7 +110,7 @@ int ProcessEvent (vout_thread_t *vout, xcb_generic_event_t *ev)
             break;
 
         default:
-            msg_Dbg (vout, "unhandled event %02x", (unsigned)ev->response_type);
+            msg_Dbg (vout, "unhandled event %"PRIu8, ev->response_type);
     }
 
     free (ev);

diff --git a/modules/video_output/xcb/window.c b/modules/video_output/xcb/window.c
index f31690a0227f5d7466c2280c8104e3f3dda2b081..6271316cb7ee16d525a0e6a0059bb1bdb22d95f9 100644 (file)
--- a/modules/video_output/xcb/window.c
+++ b/modules/video_output/xcb/window.c
@@ -183,6 +183,7 @@ static void *Thread (void *data)
         {
             if (ProcessKeyEvent (p_sys->keys, ev) == 0)
                 continue;
+            msg_Dbg (wnd, "unhandled event: %"PRIu8, ev->response_type);
             free (ev);
         }
         vlc_restorecancel (canc);

diff --git a/modules/video_output/xcb/xcb.c b/modules/video_output/xcb/xcb.c
index 8d96b1bd81d4613de4e5f5a28acc0c018a14bbd1..0f1eafeb3b9da8b5e75ef203f0b38d7a2afe79e6 100644 (file)
--- a/modules/video_output/xcb/xcb.c
+++ b/modules/video_output/xcb/xcb.c
@@ -417,12 +417,13 @@ static int Init (vout_thread_t *vout)
                                             &width, &height);
         if (p_sys->embed == NULL)
         {
-            msg_Err (vout, "cannot get window");
+            msg_Err (vout, "cannot get parent window");
             return VLC_EGENERIC;
         }
         p_sys->parent = p_sys->embed->handle.xid;
     }
 
+    /* FIXME: incorrect placement if resize now */
     vout_PlacePicture (vout, width, height, &x, &y, &width, &height);
 
     /* FIXME: I don't get the subtlety between output and fmt_out here */
@@ -446,7 +447,7 @@ static int Init (vout_thread_t *vout)
     /* Create window */
     const uint32_t mask = XCB_CW_BACK_PIXEL | XCB_CW_EVENT_MASK
                         | XCB_CW_COLORMAP;
-    uint32_t values[] = {
+    const uint32_t values[] = {
         /* XCB_CW_BACK_PIXEL */
         screen->black_pixel,
         /* XCB_CW_EVENT_MASK */