Re: [Synfig-devl] [PATCH] fix sequence point warning in state_brush.cpp

2015-07-08 Thread Konstantin Dmitriev
2015-06-22 23:37 GMT+06:00 Olaf Hering : > > gcc 4.8 in openSUSE 13.1 warns about this code: > states/state_brush.cpp:824:44: warning: operation on 'hue' may be undefined > [-Wsequence-point] > : hue = 60.0 * (((r - g)/(diff))+4.0); > > Remove early assignment and adjust indention to clear

[Synfig-devl] [PATCH] fix sequence point warning in state_brush.cpp

2015-06-22 Thread Olaf Hering
gcc 4.8 in openSUSE 13.1 warns about this code: states/state_brush.cpp:824:44: warning: operation on 'hue' may be undefined [-Wsequence-point] : hue = 60.0 * (((r - g)/(diff))+4.0); Remove early assignment and adjust indention to clearify logical code flow. Signed-off-by: Olaf Hering -