]> git.xonotic.org Git - xonotic/darkplaces.git/blobdiff - console.c
two int casts that were missing, reported by Willis
[xonotic/darkplaces.git] / console.c
index 03b917c1d1460836384d81b26355191c73ebcdb7..7c1977155096752b6d2bb05b0412f6ca9cd8ba7a 100644 (file)
--- a/console.c
+++ b/console.c
@@ -41,9 +41,9 @@ int con_current;
 int con_x;
 char con_text[CON_TEXTSIZE];
 
-//seconds
-cvar_t con_notifytime = {CVAR_SAVE, "con_notifytime","3"};
-cvar_t con_notify = {CVAR_SAVE, "con_notify","4"};
+cvar_t con_notifytime = {CVAR_SAVE, "con_notifytime","3", "how long notify lines last, in seconds"};
+cvar_t con_notify = {CVAR_SAVE, "con_notify","4", "how many notify lines to show (0-32)"};
+cvar_t con_textsize = {CVAR_SAVE, "con_textsize","8", "console text size in virtual 2D pixels"};       //[515]: console text size in pixels
 
 #define MAX_NOTIFYLINES 32
 // cl.time time the line was generated for transparent notify lines
@@ -62,7 +62,7 @@ LOGGING
 ==============================================================================
 */
 
-cvar_t log_file = {0, "log_file",""};
+cvar_t log_file = {0, "log_file","", "filename to log messages to"};
 char crt_log_file [MAX_OSPATH] = "";
 qfile_t* logfile = NULL;
 
@@ -316,9 +316,14 @@ If the line width has changed, reformat the buffer.
 void Con_CheckResize (void)
 {
        int i, j, width, oldwidth, oldtotallines, numlines, numchars;
+       float f;
        char tbuf[CON_TEXTSIZE];
 
-       width = (vid_conwidth.integer >> 3);
+       f = bound(1, con_textsize.value, 128);
+       if(f != con_textsize.value)
+               Cvar_SetValueQuick(&con_textsize, f);
+       width = (int)floor(vid_conwidth.value / con_textsize.value);
+       width = bound(1, width, CON_TEXTSIZE/4);
 
        if (width == con_linewidth)
                return;
@@ -400,13 +405,14 @@ void Con_Init_Commands (void)
        // register our cvars
        Cvar_RegisterVariable (&con_notifytime);
        Cvar_RegisterVariable (&con_notify);
+       Cvar_RegisterVariable (&con_textsize);
 
        // register our commands
-       Cmd_AddCommand ("toggleconsole", Con_ToggleConsole_f);
-       Cmd_AddCommand ("messagemode", Con_MessageMode_f);
-       Cmd_AddCommand ("messagemode2", Con_MessageMode2_f);
-       Cmd_AddCommand ("clear", Con_Clear_f);
-       Cmd_AddCommand ("maps", Con_Maps_f);                                                    // By [515]
+       Cmd_AddCommand ("toggleconsole", Con_ToggleConsole_f, "opens or closes the console");
+       Cmd_AddCommand ("messagemode", Con_MessageMode_f, "input a chat message to say to everyone");
+       Cmd_AddCommand ("messagemode2", Con_MessageMode2_f, "input a chat message to say to only your team");
+       Cmd_AddCommand ("clear", Con_Clear_f, "clear console history");
+       Cmd_AddCommand ("maps", Con_Maps_f, "list information about available maps");   // By [515]
 
        con_initialized = true;
        Con_Print("Console initialized.\n");
@@ -723,7 +729,7 @@ void Con_DrawInput (void)
                text += 1 + key_linepos - con_linewidth;
 
        // draw it
-       DrawQ_ColoredString(0, con_vislines - 16, text, con_linewidth, 8, 8, 1.0, 1.0, 1.0, 1.0, 0, NULL );
+       DrawQ_ColoredString(0, con_vislines - con_textsize.value*2, text, con_linewidth, con_textsize.value, con_textsize.value, 1.0, 1.0, 1.0, 1.0, 0, NULL );
 
        // remove cursor
 //     key_lines[edit_line][key_linepos] = 0;
@@ -739,7 +745,7 @@ Draws the last few lines of output transparently over the game top
 */
 void Con_DrawNotify (void)
 {
-       int             x, v;
+       float   x, v;
        char    *text;
        int             i;
        float   time;
@@ -771,13 +777,13 @@ void Con_DrawNotify (void)
                        int linewidth;
 
                        for (linewidth = con_linewidth; linewidth && text[linewidth-1] == ' '; linewidth--);
-                       x = (vid_conwidth.integer - linewidth * 8) / 2;
+                       x = (vid_conwidth.integer - linewidth * con_textsize.value) * 0.5;
                } else
                        x = 0;
 
-               DrawQ_ColoredString( x, v, text, con_linewidth, 8, 8, 1.0, 1.0, 1.0, 1.0, 0, &colorindex );
+               DrawQ_ColoredString( x, v, text, con_linewidth, con_textsize.value, con_textsize.value, 1.0, 1.0, 1.0, 1.0, 0, &colorindex );
 
-               v += 8;
+               v += con_textsize.value;
        }
 
 
@@ -794,14 +800,14 @@ void Con_DrawNotify (void)
                        sprintf(temptext, "say:%s%c", chat_buffer, (int) 10+((int)(realtime*con_cursorspeed)&1));
                while ((int)strlen(temptext) >= con_linewidth)
                {
-                       DrawQ_ColoredString( 0, v, temptext, con_linewidth, 8, 8, 1.0, 1.0, 1.0, 1.0, 0, &colorindex );
+                       DrawQ_ColoredString( 0, v, temptext, con_linewidth, con_textsize.value, con_textsize.value, 1.0, 1.0, 1.0, 1.0, 0, &colorindex );
                        strcpy(temptext, &temptext[con_linewidth]);
-                       v += 8;
+                       v += con_textsize.value;
                }
                if (strlen(temptext) > 0)
                {
-                       DrawQ_ColoredString( 0, v, temptext, 0, 8, 8, 1.0, 1.0, 1.0, 1.0, 0, &colorindex );
-                       v += 8;
+                       DrawQ_ColoredString( 0, v, temptext, 0, con_textsize.value, con_textsize.value, 1.0, 1.0, 1.0, 1.0, 0, &colorindex );
+                       v += con_textsize.value;
                }
        }
 }
@@ -816,7 +822,8 @@ The typing input line at the bottom should only be drawn if typing is allowed
 */
 void Con_DrawConsole (int lines)
 {
-       int i, y, rows, j;
+       int i, rows, j;
+       float y;
        char *text;
        int colorindex = -1;
 
@@ -828,20 +835,20 @@ void Con_DrawConsole (int lines)
                DrawQ_Pic(0, lines - vid_conheight.integer, "gfx/conback", vid_conwidth.integer, vid_conheight.integer, scr_conbrightness.value, scr_conbrightness.value, scr_conbrightness.value, scr_conalpha.value, 0);
        else
                DrawQ_Fill(0, lines - vid_conheight.integer, vid_conwidth.integer, vid_conheight.integer, 0, 0, 0, scr_conalpha.value, 0);
-       DrawQ_String(vid_conwidth.integer - strlen(engineversion) * 8 - 8, lines - 8, engineversion, 0, 8, 8, 1, 0, 0, 1, 0);
+       DrawQ_String(vid_conwidth.integer - strlen(engineversion) * con_textsize.value - con_textsize.value, lines - con_textsize.value, engineversion, 0, con_textsize.value, con_textsize.value, 1, 0, 0, 1, 0);
 
 // draw the text
        con_vislines = lines;
 
-       rows = (lines-16)>>3;           // rows of text to draw
-       y = lines - 16 - (rows<<3);     // may start slightly negative
+       rows = (int)ceil((lines/con_textsize.value)-2);         // rows of text to draw
+       y = lines - (rows+2)*con_textsize.value;        // may start slightly negative
 
-       for (i = con_current - rows + 1;i <= con_current;i++, y += 8)
+       for (i = con_current - rows + 1;i <= con_current;i++, y += con_textsize.value)
        {
                j = max(i - con_backscroll, 0);
                text = con_text + (j % con_totallines)*con_linewidth;
 
-               DrawQ_ColoredString( 0, y, text, con_linewidth, 8, 8, 1.0, 1.0, 1.0, 1.0, 0, &colorindex );
+               DrawQ_ColoredString( 0, y, text, con_linewidth, con_textsize.value, con_textsize.value, 1.0, 1.0, 1.0, 1.0, 0, &colorindex );
        }
 
 // draw the input prompt, user text, and cursor if desired
@@ -878,7 +885,7 @@ qboolean GetMapList (const char *s, char *completedname, int completednamebuffer
        min = 666;
        for(max=i=0;i<t->numfilenames;i++)
        {
-               k = strlen(t->filenames[i]);
+               k = (int)strlen(t->filenames[i]);
                k -= 9;
                if(max < k)
                        max = k;
@@ -887,7 +894,7 @@ qboolean GetMapList (const char *s, char *completedname, int completednamebuffer
                        min = k;
                len[i] = k;
        }
-       o = strlen(s);
+       o = (int)strlen(s);
        for(i=0;i<t->numfilenames;i++)
        {
                int lumpofs = 0, lumplen = 0;
@@ -896,6 +903,7 @@ qboolean GetMapList (const char *s, char *completedname, int completednamebuffer
                char keyname[64];
                char entfilename[MAX_QPATH];
                strcpy(message, "^1**ERROR**^7");
+               p = 0;
                f = FS_Open(t->filenames[i], "rb", true, false);
                if(f)
                {
@@ -997,6 +1005,8 @@ qboolean GetMapList (const char *s, char *completedname, int completednamebuffer
        for(p=o;p<min;p++)
        {
                k = *(t->filenames[0]+5+p);
+               if(k == 0)
+                       goto endcomplete;
                for(i=1;i<t->numfilenames;i++)
                        if(*(t->filenames[i]+5+p) != k)
                                goto endcomplete;
@@ -1005,7 +1015,7 @@ endcomplete:
        if(p > o)
        {
                memset(completedname, 0, completednamebufferlength);
-               memcpy(completedname, (t->filenames[0]+5), p);
+               memcpy(completedname, (t->filenames[0]+5), min(p, completednamebufferlength - 1));
        }
        Z_Free(len);
        FS_FreeSearch(t);
@@ -1064,7 +1074,8 @@ void Con_DisplayList(const char **list)
 */
 void Con_CompleteCommandLine (void)
 {
-       const char *cmd = "", *s;
+       const char *cmd = "";
+       char *s;
        const char **list[3] = {0, 0, 0};
        char s2[512];
        int c, v, a, i, cmd_len, pos, k;
@@ -1095,20 +1106,41 @@ void Con_CompleteCommandLine (void)
                                char t[MAX_QPATH];
                                if (GetMapList(s, t, sizeof(t)))
                                {
-                                       i = strlen(t) - strlen(s);
-                                       strcpy((char*)s, t);
-                                       if(s2[0])       //add back chars after cursor
-                                               strcpy(&key_lines[edit_line][key_linepos], s2);
-                                       key_linepos += i;
+                                       // first move the cursor
+                                       key_linepos += (int)strlen(t) - (int)strlen(s);
+
+                                       // and now do the actual work
+                                       *s = 0;
+                                       strlcat(key_lines[edit_line], t, MAX_INPUTLINE);
+                                       strlcat(key_lines[edit_line], s2, MAX_INPUTLINE); //add back chars after cursor
+
+                                       // and fix the cursor
+                                       if(key_linepos > (int) strlen(key_lines[edit_line]))
+                                               key_linepos = (int) strlen(key_lines[edit_line]);
                                }
                                return;
                        }
                }
 
-       // Count number of possible matches
+       // Count number of possible matches and print them
        c = Cmd_CompleteCountPossible(s);
+       if (c)
+       {
+               Con_Printf("\n%i possible command%s\n", c, (c > 1) ? "s: " : ":");
+               Cmd_CompleteCommandPrint(s);
+       }
        v = Cvar_CompleteCountPossible(s);
+       if (v)
+       {
+               Con_Printf("\n%i possible variable%s\n", v, (v > 1) ? "s: " : ":");
+               Cvar_CompleteCvarPrint(s);
+       }
        a = Cmd_CompleteAliasCountPossible(s);
+       if (a)
+       {
+               Con_Printf("\n%i possible aliases%s\n", a, (a > 1) ? "s: " : ":");
+               Cmd_CompleteAliasPrint(s);
+       }
 
        if (!(c + v + a))       // No possible matches
        {
@@ -1117,76 +1149,48 @@ void Con_CompleteCommandLine (void)
                return;
        }
 
-       if (c + v + a == 1) {
-               if (c)
-                       list[0] = Cmd_CompleteBuildList(s);
-               else if (v)
-                       list[0] = Cvar_CompleteBuildList(s);
-               else
-                       list[0] = Cmd_CompleteAliasBuildList(s);
-               cmd = *list[0];
-               cmd_len = (int)strlen (cmd);
-       } else {
-               if (c)
-                       cmd = *(list[0] = Cmd_CompleteBuildList(s));
-               if (v)
-                       cmd = *(list[1] = Cvar_CompleteBuildList(s));
-               if (a)
-                       cmd = *(list[2] = Cmd_CompleteAliasBuildList(s));
-
-               cmd_len = (int)strlen (s);
-               do {
-                       for (i = 0; i < 3; i++) {
-                               char ch = cmd[cmd_len];
-                               const char **l = list[i];
-                               if (l) {
-                                       while (*l && (*l)[cmd_len] == ch)
-                                               l++;
-                                       if (*l)
-                                               break;
-                               }
-                       }
-                       if (i == 3)
-                               cmd_len++;
-               } while (i == 3);
-               // 'quakebar'
-               Con_Print("\n\35");
-               for (i = 0; i < con_linewidth - 4; i++)
-                       Con_Print("\36");
-               Con_Print("\37\n");
-
-               // Print Possible Commands
-               if (c) {
-                       Con_Printf("%i possible command%s\n", c, (c > 1) ? "s: " : ":");
-                       Con_DisplayList(list[0]);
-               }
-
-               if (v) {
-                       Con_Printf("%i possible variable%s\n", v, (v > 1) ? "s: " : ":");
-                       Con_DisplayList(list[1]);
-               }
+       if (c)
+               cmd = *(list[0] = Cmd_CompleteBuildList(s));
+       if (v)
+               cmd = *(list[1] = Cvar_CompleteBuildList(s));
+       if (a)
+               cmd = *(list[2] = Cmd_CompleteAliasBuildList(s));
 
-               if (a) {
-                       Con_Printf("%i possible aliases%s\n", a, (a > 1) ? "s: " : ":");
-                       Con_DisplayList(list[2]);
+       for (cmd_len = (int)strlen(s);;cmd_len++)
+       {
+               const char **l;
+               for (i = 0; i < 3; i++)
+                       if (list[i])
+                               for (l = list[i];*l;l++)
+                                       if ((*l)[cmd_len] != cmd[cmd_len])
+                                               goto done;
+               // all possible matches share this character, so we continue...
+               if (!cmd[cmd_len])
+               {
+                       // if all matches ended at the same position, stop
+                       // (this means there is only one match)
+                       break;
                }
        }
+done:
 
-       if (cmd) {
-               strncpy(key_lines[edit_line] + pos, cmd, cmd_len);
-               key_linepos = cmd_len + pos;
-               if (c + v + a == 1) {
-                       key_lines[edit_line][key_linepos] = ' ';
-                       key_linepos++;
-               }
-               if(s2[0])
-                       strcpy(&key_lines[edit_line][key_linepos], s2);
-               else
-                       key_lines[edit_line][key_linepos] = 0;
+       // prevent a buffer overrun by limiting cmd_len according to remaining space
+       cmd_len = min(cmd_len, (int)sizeof(key_lines[edit_line]) - 1 - pos);
+       if (cmd)
+       {
+               key_linepos = pos;
+               memcpy(&key_lines[edit_line][key_linepos], cmd, cmd_len);
+               key_linepos += cmd_len;
+               // if there is only one match, add a space after it
+               if (c + v + a == 1 && key_linepos < (int)sizeof(key_lines[edit_line]) - 1)
+                       key_lines[edit_line][key_linepos++] = ' ';
        }
-       else
-               if(s2[0])
-                       strcpy(&key_lines[edit_line][key_linepos], s2);
+
+       // use strlcat to avoid a buffer overrun
+       key_lines[edit_line][key_linepos] = 0;
+       strlcat(key_lines[edit_line], s2, sizeof(key_lines[edit_line]));
+
+       // free the command, cvar, and alias lists
        for (i = 0; i < 3; i++)
                if (list[i])
                        Mem_Free((void *)list[i]);