Fix compiler warnings in release builds
Signed-off-by: Yuxuan Shui <yshuiv7@gmail.com>
This commit is contained in:
parent
065f9ffd4d
commit
583cf35bed
2
src/c2.c
2
src/c2.c
|
@ -249,7 +249,7 @@ static inline int strcmp_wd(const char *needle, const char *src) {
|
||||||
/**
|
/**
|
||||||
* Return whether a c2_ptr_t is empty.
|
* Return whether a c2_ptr_t is empty.
|
||||||
*/
|
*/
|
||||||
static inline bool c2_ptr_isempty(const c2_ptr_t p) {
|
static inline attr_unused bool c2_ptr_isempty(const c2_ptr_t p) {
|
||||||
return !(p.isbranch ? (bool)p.b : (bool)p.l);
|
return !(p.isbranch ? (bool)p.b : (bool)p.l);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -134,7 +134,7 @@ static inline bool ensure_glx_context(session_t *ps) {
|
||||||
/**
|
/**
|
||||||
* Free a GLX texture.
|
* Free a GLX texture.
|
||||||
*/
|
*/
|
||||||
static inline void free_texture_r(session_t *ps, GLuint *ptexture) {
|
static inline void free_texture_r(session_t *ps attr_unused, GLuint *ptexture) {
|
||||||
if (*ptexture) {
|
if (*ptexture) {
|
||||||
assert(glx_has_context(ps));
|
assert(glx_has_context(ps));
|
||||||
glDeleteTextures(1, ptexture);
|
glDeleteTextures(1, ptexture);
|
||||||
|
|
|
@ -57,7 +57,7 @@ safe_isnan(double a) {
|
||||||
/// being always true or false.
|
/// being always true or false.
|
||||||
#define ASSERT_IN_RANGE(var, lower, upper) \
|
#define ASSERT_IN_RANGE(var, lower, upper) \
|
||||||
do { \
|
do { \
|
||||||
auto __tmp = (var); \
|
auto __tmp attr_unused = (var); \
|
||||||
_Pragma("GCC diagnostic push"); \
|
_Pragma("GCC diagnostic push"); \
|
||||||
_Pragma("GCC diagnostic ignored \"-Wtype-limits\""); \
|
_Pragma("GCC diagnostic ignored \"-Wtype-limits\""); \
|
||||||
assert(__tmp >= lower); \
|
assert(__tmp >= lower); \
|
||||||
|
@ -69,7 +69,7 @@ safe_isnan(double a) {
|
||||||
/// being always true or false.
|
/// being always true or false.
|
||||||
#define ASSERT_GEQ(var, lower) \
|
#define ASSERT_GEQ(var, lower) \
|
||||||
do { \
|
do { \
|
||||||
auto __tmp = (var); \
|
auto __tmp attr_unused = (var); \
|
||||||
_Pragma("GCC diagnostic push"); \
|
_Pragma("GCC diagnostic push"); \
|
||||||
_Pragma("GCC diagnostic ignored \"-Wtype-limits\""); \
|
_Pragma("GCC diagnostic ignored \"-Wtype-limits\""); \
|
||||||
assert(__tmp >= lower); \
|
assert(__tmp >= lower); \
|
||||||
|
@ -111,7 +111,7 @@ safe_isnan(double a) {
|
||||||
#define to_u32_checked(val) \
|
#define to_u32_checked(val) \
|
||||||
({ \
|
({ \
|
||||||
auto tmp = (val); \
|
auto tmp = (val); \
|
||||||
int64_t max = UINT32_MAX; /* silence clang tautological \
|
int64_t max attr_unused = UINT32_MAX; /* silence clang tautological \
|
||||||
comparison warning*/ \
|
comparison warning*/ \
|
||||||
ASSERT_IN_RANGE(tmp, 0, max); \
|
ASSERT_IN_RANGE(tmp, 0, max); \
|
||||||
(uint32_t) tmp; \
|
(uint32_t) tmp; \
|
||||||
|
|
|
@ -329,8 +329,7 @@ void win_release_images(struct backend_base *backend, struct managed_win *w) {
|
||||||
void win_process_flags(session_t *ps, struct managed_win *w) {
|
void win_process_flags(session_t *ps, struct managed_win *w) {
|
||||||
// Make sure all pending window updates are processed before this. Making this
|
// Make sure all pending window updates are processed before this. Making this
|
||||||
// assumption simplifies some checks (e.g. whether window is mapped)
|
// assumption simplifies some checks (e.g. whether window is mapped)
|
||||||
auto iw = (struct managed_win_internal *)w;
|
assert(((struct managed_win_internal *)w)->pending_updates == 0);
|
||||||
assert(iw->pending_updates == 0);
|
|
||||||
|
|
||||||
if (!w->flags || (w->flags & WIN_FLAGS_IMAGE_ERROR) != 0) {
|
if (!w->flags || (w->flags & WIN_FLAGS_IMAGE_ERROR) != 0) {
|
||||||
return;
|
return;
|
||||||
|
@ -2250,6 +2249,7 @@ win_is_fullscreen_xcb(xcb_connection_t *c, const struct atom *a, const xcb_windo
|
||||||
void win_queue_update(struct managed_win *_w, enum win_update update) {
|
void win_queue_update(struct managed_win *_w, enum win_update update) {
|
||||||
auto w = (struct managed_win_internal *)_w;
|
auto w = (struct managed_win_internal *)_w;
|
||||||
assert(popcount(update) == 1);
|
assert(popcount(update) == 1);
|
||||||
|
assert(update == WIN_UPDATE_MAP); // Currently the only supported update
|
||||||
|
|
||||||
if (unlikely(_w->state == WSTATE_DESTROYING)) {
|
if (unlikely(_w->state == WSTATE_DESTROYING)) {
|
||||||
log_error("Updates queued on a destroyed window %#010x (%s)", _w->base.id,
|
log_error("Updates queued on a destroyed window %#010x (%s)", _w->base.id,
|
||||||
|
@ -2257,7 +2257,7 @@ void win_queue_update(struct managed_win *_w, enum win_update update) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
w->pending_updates |= WIN_UPDATE_MAP;
|
w->pending_updates |= update;
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Process pending updates on a window. Has to be called in X critical section
|
/// Process pending updates on a window. Has to be called in X critical section
|
||||||
|
|
Loading…
Reference in New Issue