mirror of
https://github.com/gryf/wmaker.git
synced 2025-12-22 22:28:02 +01:00
WINGs: Value stored to 'scroll' is never read
it is assigned after doing switch() {}
so values assigned in switch() are dropped anyway
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>
This commit is contained in:
committed by
Carlos R. Mafra
parent
b857bafc88
commit
f41880cf05
@@ -1253,7 +1253,6 @@ static void scrollersCallBack(WMWidget * w, void *self)
|
|||||||
tPtr->vpos -= 16;
|
tPtr->vpos -= 16;
|
||||||
else
|
else
|
||||||
tPtr->vpos = 0;
|
tPtr->vpos = 0;
|
||||||
scroll = True;
|
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
|
|
||||||
@@ -1264,7 +1263,6 @@ static void scrollersCallBack(WMWidget * w, void *self)
|
|||||||
tPtr->vpos += 16;
|
tPtr->vpos += 16;
|
||||||
else
|
else
|
||||||
tPtr->vpos = limit;
|
tPtr->vpos = limit;
|
||||||
scroll = True;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
@@ -1274,21 +1272,17 @@ static void scrollersCallBack(WMWidget * w, void *self)
|
|||||||
tPtr->vpos -= height;
|
tPtr->vpos -= height;
|
||||||
else
|
else
|
||||||
tPtr->vpos = 0;
|
tPtr->vpos = 0;
|
||||||
|
|
||||||
scroll = True;
|
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case WSIncrementPage:
|
case WSIncrementPage:
|
||||||
tPtr->vpos += height;
|
tPtr->vpos += height;
|
||||||
if (tPtr->vpos > (tPtr->docHeight - height))
|
if (tPtr->vpos > (tPtr->docHeight - height))
|
||||||
tPtr->vpos = tPtr->docHeight - height;
|
tPtr->vpos = tPtr->docHeight - height;
|
||||||
scroll = True;
|
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case WSKnob:
|
case WSKnob:
|
||||||
tPtr->vpos = WMGetScrollerValue(tPtr->vS)
|
tPtr->vpos = WMGetScrollerValue(tPtr->vS)
|
||||||
* (float)(tPtr->docHeight - height);
|
* (float)(tPtr->docHeight - height);
|
||||||
scroll = True;
|
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case WSKnobSlot:
|
case WSKnobSlot:
|
||||||
@@ -1311,7 +1305,6 @@ static void scrollersCallBack(WMWidget * w, void *self)
|
|||||||
tPtr->hpos -= 16;
|
tPtr->hpos -= 16;
|
||||||
else
|
else
|
||||||
tPtr->hpos = 0;
|
tPtr->hpos = 0;
|
||||||
scroll = True;
|
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
|
|
||||||
@@ -1322,7 +1315,6 @@ static void scrollersCallBack(WMWidget * w, void *self)
|
|||||||
tPtr->hpos += 16;
|
tPtr->hpos += 16;
|
||||||
else
|
else
|
||||||
tPtr->hpos = limit;
|
tPtr->hpos = limit;
|
||||||
scroll = True;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
@@ -1332,21 +1324,17 @@ static void scrollersCallBack(WMWidget * w, void *self)
|
|||||||
tPtr->hpos -= width;
|
tPtr->hpos -= width;
|
||||||
else
|
else
|
||||||
tPtr->hpos = 0;
|
tPtr->hpos = 0;
|
||||||
|
|
||||||
scroll = True;
|
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case WSIncrementPage:
|
case WSIncrementPage:
|
||||||
tPtr->hpos += width;
|
tPtr->hpos += width;
|
||||||
if (tPtr->hpos > (tPtr->docWidth - width))
|
if (tPtr->hpos > (tPtr->docWidth - width))
|
||||||
tPtr->hpos = tPtr->docWidth - width;
|
tPtr->hpos = tPtr->docWidth - width;
|
||||||
scroll = True;
|
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case WSKnob:
|
case WSKnob:
|
||||||
tPtr->hpos = WMGetScrollerValue(tPtr->hS)
|
tPtr->hpos = WMGetScrollerValue(tPtr->hS)
|
||||||
* (float)(tPtr->docWidth - width);
|
* (float)(tPtr->docWidth - width);
|
||||||
scroll = True;
|
|
||||||
break;
|
break;
|
||||||
|
|
||||||
case WSKnobSlot:
|
case WSKnobSlot:
|
||||||
|
|||||||
Reference in New Issue
Block a user