gryf/wmaker
1
0
Fork 0
mirror of https://github.com/gryf/wmaker.git synced 2025-12-19 12:28:22 +01:00
Code Issues Projects Releases Wiki Activity

Remove warning in wmagnify.c (correctly)

Browse Source 
The problem with f65b99e615 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.
This commit is contained in:
Brad Jorsch
2011-08-09 16:13:12 -04:00
committed by Carlos R. Mafra
parent e01d14abe4
commit d9296ff9a4
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
util/wmagnify.c
View File

@@ -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);