comparison arogue5/main.c @ 112:ee250e3646fd

Don't truncate player name in savefile name or log message. The player name is stored in whoami[], which is length 80 in most games (1024 in rogue5). Only the first 10 chars were used to create file_name, because that buffer is the same length. Increasing the size of file_name to 256 permits using all of whoami. The name is also no longer truncated to 20 chars when writing the log. All games should now be able to handle 79-character names without collisions. Anything more would break save compatibility.
author John "Elwin" Edwards
date Sun, 23 Mar 2014 21:27:14 -0700
parents f2951c4e28d9
children 7faf4568c295
comparison
equal deleted inserted replaced
111:7f8f43943b1f 112:ee250e3646fd
62 #ifdef SAVEDIR 62 #ifdef SAVEDIR
63 if (argc >= 3 && !strcmp(argv[1], "-n")) { 63 if (argc >= 3 && !strcmp(argv[1], "-n")) {
64 use_savedir = TRUE; 64 use_savedir = TRUE;
65 strncpy(whoami, argv[2], LINELEN); 65 strncpy(whoami, argv[2], LINELEN);
66 whoami[LINELEN - 1] = '\0'; 66 whoami[LINELEN - 1] = '\0';
67 if (snprintf(file_name, LINELEN, "%s/%d-%.10s.ar5sav", SAVEDIR, 67 if (snprintf(file_name, 256, "%s/%d-%s.ar5sav", SAVEDIR,
68 md_getuid(), whoami) >= LINELEN) 68 md_getuid(), whoami) >= 256)
69 { 69 {
70 /* The name is too long */ 70 /* The name is too long */
71 use_savedir = FALSE; 71 use_savedir = FALSE;
72 } 72 }
73 } 73 }