Skip to content

modesetting: Fix build with DebugPresent() enabled

Olivier Fourdan requested to merge ofourdan/xserver:fix-modesetting-DebugPresent into master

By default, the macro DebugPresent() is a no-op but it can be enabled at build time for debugging purpose.

However, doing so prevents the code to build because one debug statement tries to make use of a non-existent variable:

  present.c: In function ‘ms_present_queue_vblank’:
  present.c:147:18: error: ‘vbl’ undeclared (first use in this function)
    147 |                  vbl.request.sequence));
        |                  ^~~
  present.c:49:32: note: in definition of macro ‘DebugPresent’
    49 | #define DebugPresent(x) ErrorF x
       |                                ^

Fix the build with DebugPresent() by removing the vbl variable from the debug message.

Signed-off-by: Olivier Fourdan ofourdan@redhat.com

Edited by Olivier Fourdan

Merge request reports