As pointed by gcc, the height is never used and the code make use of the SCROLLER_WIDTH constant anyway, so this patch updates the code to directly use that constant instead of intermediate values that just adds noise for code maintainability.
Signed-off-by: Christophe CURIS <christophe.cu...@free.fr> --- WINGs/wscroller.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/WINGs/wscroller.c b/WINGs/wscroller.c index 4a5ae8c..f1daa3a 100644 --- a/WINGs/wscroller.c +++ b/WINGs/wscroller.c @@ -64,8 +64,6 @@ typedef struct W_Scroller { } flags; } Scroller; -#define DEFAULT_HEIGHT 60 -#define DEFAULT_WIDTH SCROLLER_WIDTH #define DEFAULT_ARROWS_POSITION WSAMinEnd #define BUTTON_SIZE ((SCROLLER_WIDTH) - 4) @@ -108,7 +106,7 @@ WMScroller *WMCreateScroller(WMWidget * parent) WMCreateEventHandler(sPtr->view, ExposureMask | StructureNotifyMask | ClientMessageMask, handleEvents, sPtr); - W_ResizeView(sPtr->view, DEFAULT_WIDTH, DEFAULT_WIDTH); + W_ResizeView(sPtr->view, SCROLLER_WIDTH, SCROLLER_WIDTH); sPtr->flags.arrowsPosition = DEFAULT_ARROWS_POSITION; WMCreateEventHandler(sPtr->view, ButtonPressMask | ButtonReleaseMask -- 2.1.3 -- To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.