voxglitch icon indicating copy to clipboard operation
voxglitch copied to clipboard

Just FYI: Some warnings building mac arm with latest clang

Open baconpaul opened this issue 2 years ago • 1 comments

src/GrooveBox/widgets/LCDTrackDisplay.hpp:123:13: warning: logical not is only applied to the left hand side of this comparison [-Wlogical-not-parentheses]
        if (!module->lcd_screen_mode == module->TRACK)
            ^                        ~~
src/GrooveBox/widgets/LCDTrackDisplay.hpp:123:13: note: add parentheses after the '!' to evaluate the comparison first
        if (!module->lcd_screen_mode == module->TRACK)
            ^
             (                                       )
src/GrooveBox/widgets/LCDTrackDisplay.hpp:123:13: note: add parentheses around left hand side expression to silence this warning
        if (!module->lcd_screen_mode == module->TRACK)
            ^
            (                       )
src/GrooveBox/widgets/LCDTrackDisplay.hpp:400:13: warning: logical not is only applied to the left hand side of this comparison [-Wlogical-not-parentheses]
        if (!module->lcd_screen_mode == module->TRACK)
            ^                        ~~
src/GrooveBox/widgets/LCDTrackDisplay.hpp:400:13: note: add parentheses after the '!' to evaluate the comparison first
        if (!module->lcd_screen_mode == module->TRACK)
            ^
             (                                       )
src/GrooveBox/widgets/LCDTrackDisplay.hpp:400:13: note: add parentheses around left hand side expression to silence this warning
        if (!module->lcd_screen_mode == module->TRACK)
            ^
            (                       )

I am pretty sure you mean module->lcd_screen_mode != module->TRACK but you might actually mean not-the-mode but since TRACK is 0 it has the same effect I think.

No biggie just saw it doing an ARM build and figured I would drop a note

baconpaul avatar Jan 26 '23 00:01 baconpaul

Ah, thanks! I'll dig into that soon!

clone45 avatar Jan 26 '23 00:01 clone45