[lttng-dev] [PATCH lttng-ust] Unnecessary bit shit
Jonathan Rajotte
jonathan.rajotte-julien at efficios.com
Mon Sep 21 14:31:33 EDT 2015
Reported by cppcheck [1].
[1]
https://ci.lttng.org/view/Code%20quality/job/lttng-ust_master_cppcheck/5/cppcheckResult/source.10/
Proposed-by: Michael Jeanson <mjeanson at efficios.com>
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien at efficios.com>
---
libringbuffer/backend_internal.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libringbuffer/backend_internal.h b/libringbuffer/backend_internal.h
index 344784f..79f8bde 100644
--- a/libringbuffer/backend_internal.h
+++ b/libringbuffer/backend_internal.h
@@ -485,7 +485,7 @@ static inline int lttng_ust_fls(unsigned int x)
r -= 2;
}
if (!(x & 0x80000000U)) {
- x <<= 1;
+ /* No need to bit shift on last operation */
r -= 1;
}
return r;
--
2.1.4
More information about the lttng-dev
mailing list