The problem with f65b99e615847 was that it removed the call to
newWindow() along with the unused variable, without considering that
newWindow() has side effects that need to happen even if the return
value is ignored. So as long as we keep the newWindow() call, we can
safely remove the (unused) variable.
---
util/wmagnify.c | 3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/util/wmagnify.c b/util/wmagnify.c
index d65d540..3bb7a37 100644
--- a/util/wmagnify.c
+++ b/util/wmagnify.c
@@ -397,7 +397,6 @@ static BufferData *newWindow(int magfactor)
int main(int argc, char **argv)
{
- BufferData *data;
int i;
char *display = "";
char *vdisplay = NULL;
@@ -481,7 +480,7 @@ int main(int argc, char **argv)
cursorColor1 = WMCreateNamedColor(scr, "#ff0000", False);
cursorColor2 = WMCreateNamedColor(scr, "#00ff00", False);
- data = newWindow(magfactor);
+ newWindow(magfactor);
WMScreenMainLoop(scr);
--
1.7.5.4
--
To unsubscribe, send mail to [email protected].
