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.

Reply via email to