You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As of version 4.10.6, compiling nuklear.h with #define NK_INCLUDE_STANDARD_BOOL yields the following warning: enum constant in boolean context [-Wint-in-bool-context]
The issue stems from NK_API nk_bool nk_group_scrolled_offset_begin():
if (f & NK_WINDOW_CLOSED)
return NK_WINDOW_CLOSED;
if (f & NK_WINDOW_MINIMIZED)
return NK_WINDOW_MINIMIZED;
Given that the function is expected to return "true(1) if visible and fillable with widgets or false(0) otherwise", I'm guessing the two return should be merged into a single return false instead? Currently the conversion from the nk_window_flags enum to bool would return true.
The text was updated successfully, but these errors were encountered:
As of version 4.10.6, compiling
nuklear.h
with#define NK_INCLUDE_STANDARD_BOOL
yields the following warning:enum constant in boolean context [-Wint-in-bool-context]
The issue stems from
NK_API nk_bool nk_group_scrolled_offset_begin()
:Given that the function is expected to return "
true(1)
if visible and fillable with widgets orfalse(0)
otherwise", I'm guessing the tworeturn
should be merged into a single returnfalse
instead? Currently the conversion from thenk_window_flags
enum tobool
would returntrue
.The text was updated successfully, but these errors were encountered: