xf86-video-mga: Changes to 'master'

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Sat Nov 16 21:29:04 UTC 2024


 src/mga_dac3026.c |   10 --------
 src/mga_dacG.c    |   28 +++--------------------
 src/mga_driver.c  |   65 ++++++++++++++++++++++++------------------------------
 src/mga_macros.h  |    2 -
 src/mga_merge.c   |    2 -
 5 files changed, 35 insertions(+), 72 deletions(-)

New commits:
commit 62f9d8de067945431d4ba3726954c0a4b9f6450b
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Nov 10 10:42:17 2024 -0800

    Remove MGA_NOT_HAL() macro left behind from HAL removal
    
    When commit 94bbeb132c7eda removed the HAL support, it removed the
    MGA_HAL() macro to wrap code only used with HAL, but left behind
    the equivalent MGA_NO_HAL() macro for the non-HAL code paths.
    Since it just expanded to the code it contained, it did nothing
    but upset clang and mess up the code formatting.
    
    This fixes 6 clang warnings of the form:
    
    mga_dacG.c:1670:2: warning: embedding a directive within macro arguments
     has undefined behavior [-Wembedded-directive]
    #ifdef XSERVER_LIBPCIACCESS
     ^
    
    and 23 clang warnings of the form:
    
    mga_dacG.c:1351:3: warning: empty expression statement has no effect;
     remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
            );      /* MGA_NOT_HAL */
             ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-mga/-/merge_requests/14>



More information about the xorg-commit mailing list