warden/early-roguelike
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

srogue: remove unnecessary pointer cast.

Browse source 
Casting char* to int before a comparison doesn't alter the result, and
isn't safe if sizeof(char *) != sizeof(int).
This commit is contained in:
John "Elwin" Edwards 2014-03-27 11:09:30 -07:00
parent 67ec840c3c
commit dff96acf36
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
srogue/options.c
View file

@ -124,7 +124,7 @@ WINDOW *awin;
*/ */
for (sp = buf; (c=readchar(awin)) != '\n' && c != '\r' && c != ESCAPE; for (sp = buf; (c=readchar(awin)) != '\n' && c != '\r' && c != ESCAPE;
wclrtoeol(awin), draw(awin)) { wclrtoeol(awin), draw(awin)) {
if (( (int)sp - (int)buf ) >= 50) { if (sp - buf >= 50) {
*sp = '\0'; /* line was too long */ *sp = '\0'; /* line was too long */
strucpy(opt,buf,strlen(buf)); strucpy(opt,buf,strlen(buf));
mvwaddstr(awin, 0, 0, "Name was truncated --More--"); mvwaddstr(awin, 0, 0, "Name was truncated --More--");