From a17e8bda4d54ffb127099fa11cf0f38e4d461364 Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Tue, 8 Apr 2025 17:24:03 +0200 Subject: [PATCH] dix: make CountBits() inline The function is small enough for easily being inlined. Signed-off-by: Enrico Weigelt, metux IT consult --- dix/input_priv.h | 10 +++++++++- dix/inpututils.c | 13 ------------- 2 files changed, 9 insertions(+), 14 deletions(-) diff --git a/dix/input_priv.h b/dix/input_priv.h index 9828fea64..2025bfab6 100644 --- a/dix/input_priv.h +++ b/dix/input_priv.h @@ -53,6 +53,7 @@ SOFTWARE. #include "include/cursor.h" #include "include/input.h" +#include "include/inputstr.h" void InitCoreDevices(void); void InitXTestDevices(void); @@ -516,6 +517,13 @@ void LastEventTimeToggleResetAll(Bool state); * @param len size of bitmask in bits (may span multiple bytes) * @return number of bits set in the given bitmask */ -int CountBits(const uint8_t * mask, int len); +static inline int CountBits(const uint8_t * mask, int len) +{ + int ret = 0; + for (int i = 0; i < len; i++) + if (BitIsOn(mask, i)) + ret++; + return ret; +} #endif /* _XSERVER_INPUT_PRIV_H */ diff --git a/dix/inpututils.c b/dix/inpututils.c index df7411527..25a309067 100644 --- a/dix/inpututils.c +++ b/dix/inpututils.c @@ -690,19 +690,6 @@ valuator_mask_fetch_unaccelerated(const ValuatorMask *mask, return FALSE; } -int -CountBits(const uint8_t * mask, int len) -{ - int i; - int ret = 0; - - for (i = 0; i < len; i++) - if (BitIsOn(mask, i)) - ret++; - - return ret; -} - /** * Verifies sanity of the event. If the event is not an internal event, * memdumps the first 32 bytes of event to the log, a backtrace, then kill