kolan
/
CMake
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

ccmake: Pass format string to 'printw' (#15738) 

printw takes a format string as first argument, so don't pass variable
strings to it directly.
This commit is contained in:
Joerg Sonnenberger 2015-09-12 16:35:36 +02:00 committed by Brad King
parent 6dad4c25b0
commit 6c442e5a89
3 changed files with 20 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Source/CursesDialog/cmCursesLongMessageForm.cxx
View File

@ -80,12 +80,13 @@ void cmCursesLongMessageForm::UpdateStatusBar()
sprintf(version+sideSpace, "%s", vertmp);
version[width] = '\0';
char fmt_s[] = "%s";
curses_move(y-4,0);
attron(A_STANDOUT);
printw(bar);
printw(fmt_s, bar);
attroff(A_STANDOUT);
curses_move(y-3,0);
printw(version);
printw(fmt_s, version);
pos_form_cursor(this->Form);
}
@ -101,8 +102,9 @@ void cmCursesLongMessageForm::PrintKeys()
char firstLine[512];
sprintf(firstLine, "Press [e] to exit help");
char fmt_s[] = "%s";
curses_move(y-2,0);
printw(firstLine);
printw(fmt_s, firstLine);
pos_form_cursor(this->Form);
}

17
Source/CursesDialog/cmCursesMainForm.cxx
View File

@ -451,24 +451,25 @@ void cmCursesMainForm::PrintKeys(int process /* = 0 */)
}
curses_move(y-4,0);
char fmt_s[] = "%s";
char fmt[512] = "Press [enter] to edit option";
if ( process )
{
strcpy(fmt, " ");
}
printw(fmt);
printw(fmt_s, fmt);
curses_move(y-3,0);
printw(firstLine);
printw(fmt_s, firstLine);
curses_move(y-2,0);
printw(secondLine);
printw(fmt_s, secondLine);
curses_move(y-1,0);
printw(thirdLine);
printw(fmt_s, thirdLine);
if (cw)
{
sprintf(firstLine, "Page %d of %d", cw->GetPage(), this->NumberOfPages);
curses_move(0,65-static_cast<unsigned int>(strlen(firstLine))-1);
printw(firstLine);
printw(fmt_s, firstLine);
}
// }
@ -612,13 +613,13 @@ void cmCursesMainForm::UpdateStatusBar(const char* message)
version[width] = '\0';
// Now print both lines
char fmt_s[] = "%s";
curses_move(y-5,0);
attron(A_STANDOUT);
char format[] = "%s";
printw(format, bar);
printw(fmt_s, bar);
attroff(A_STANDOUT);
curses_move(y-4,0);
printw(version);
printw(fmt_s, version);
pos_form_cursor(this->Form);
}

12
Source/CursesDialog/cmCursesStringWidget.cxx
View File

@ -221,6 +221,7 @@ bool cmCursesStringWidget::PrintKeys()
}
if (this->InEdit)
{
char fmt_s[] = "%s";
char firstLine[512];
// Clean the toolbar
for(int i=0; i<512; i++)
@ -229,17 +230,16 @@ bool cmCursesStringWidget::PrintKeys()
}
firstLine[511] = '\0';
curses_move(y-4,0);
printw(firstLine);
printw(fmt_s, firstLine);
curses_move(y-3,0);
printw(firstLine);
printw(fmt_s, firstLine);
curses_move(y-2,0);
printw(firstLine);
printw(fmt_s, firstLine);
curses_move(y-1,0);
printw(firstLine);
printw(fmt_s, firstLine);
sprintf(firstLine, "Editing option, press [enter] to leave edit.");
curses_move(y-3,0);
printw(firstLine);
printw(fmt_s, "Editing option, press [enter] to leave edit.");
return true;
}
else