]> de.git.xonotic.org Git - xonotic/darkplaces.git/blobdiff - console.c
major overhaul for thread-safety - many global variables and static
[xonotic/darkplaces.git] / console.c
index 2f7e99b72f96d41a0d22c861a256247d1385a858..537cdc72019ced668af841aa9e8fbed16c153b2b 100644 (file)
--- a/console.c
+++ b/console.c
@@ -289,27 +289,9 @@ int ConBuffer_FindPrevLine(conbuffer_t *buf, int mask_must, int mask_mustnot, in
        return -1;
 }
 
-int Con_FindNextLine(conbuffer_t *buf, int mask_must, int mask_mustnot, int start)
-{
-       int i;
-       for(i = start + 1; i < buf->lines_count; ++i)
-       {
-               con_lineinfo_t *l = &CONBUFFER_LINES(buf, i);
-
-               if((l->mask & mask_must) != mask_must)
-                       continue;
-               if(l->mask & mask_mustnot)
-                       continue;
-
-               return i;
-       }
-
-       return -1;
-}
-
 const char *ConBuffer_GetLine(conbuffer_t *buf, int i)
 {
-       static char copybuf[MAX_INPUTLINE];
+       static char copybuf[MAX_INPUTLINE]; // client only
        con_lineinfo_t *l = &CONBUFFER_LINES(buf, i);
        size_t sz = l->len+1 > sizeof(copybuf) ? sizeof(copybuf) : l->len+1;
        strlcpy(copybuf, l->start, sz);
@@ -340,23 +322,13 @@ size_t logq_size = 0;
 
 void Log_ConPrint (const char *msg);
 //@}
-/*
-====================
-Log_DestBuffer_Init
-====================
-*/
 static void Log_DestBuffer_Init(void)
 {
        memcpy(log_dest_buffer, "\377\377\377\377n", 5); // QW rcon print
        log_dest_buffer_pos = 5;
 }
 
-/*
-====================
-Log_DestBuffer_Flush
-====================
-*/
-void Log_DestBuffer_Flush(void)
+static void Log_DestBuffer_Flush_NoLock(void)
 {
        lhnetaddress_t log_dest_addr;
        lhnetsocket_t *log_dest_socket;
@@ -392,12 +364,21 @@ void Log_DestBuffer_Flush(void)
 
 /*
 ====================
-Log_Timestamp
+Log_DestBuffer_Flush
 ====================
 */
-const char* Log_Timestamp (const char *desc)
+void Log_DestBuffer_Flush(void)
+{
+       if (con_mutex)
+               Thread_LockMutex(con_mutex);
+       Log_DestBuffer_Flush_NoLock();
+       if (con_mutex)
+               Thread_UnlockMutex(con_mutex);
+}
+
+static const char* Log_Timestamp (const char *desc)
 {
-       static char timestamp [128];
+       static char timestamp [128]; // init/shutdown only
        time_t crt_time;
 #if _MSC_VER >= 1400
        struct tm crt_tm;
@@ -424,13 +405,7 @@ const char* Log_Timestamp (const char *desc)
        return timestamp;
 }
 
-
-/*
-====================
-Log_Open
-====================
-*/
-void Log_Open (void)
+static void Log_Open (void)
 {
        if (logfile != NULL || log_file.string[0] == '\0')
                return;
@@ -443,7 +418,6 @@ void Log_Open (void)
        }
 }
 
-
 /*
 ====================
 Log_Close
@@ -492,7 +466,7 @@ void Log_Start (void)
                                        n = min(sizeof(log_dest_buffer) - log_dest_buffer_pos - 1, logq_ind - pos);
                                        memcpy(log_dest_buffer + log_dest_buffer_pos, temp + pos, n);
                                        log_dest_buffer_pos += n;
-                                       Log_DestBuffer_Flush();
+                                       Log_DestBuffer_Flush_NoLock();
                                        pos += n;
                                }
                        }
@@ -625,7 +599,7 @@ void Con_ClearNotify (void)
 Con_MessageMode_f
 ================
 */
-void Con_MessageMode_f (void)
+static void Con_MessageMode_f (void)
 {
        key_dest = key_message;
        chat_mode = 0; // "say"
@@ -642,7 +616,7 @@ void Con_MessageMode_f (void)
 Con_MessageMode2_f
 ================
 */
-void Con_MessageMode2_f (void)
+static void Con_MessageMode2_f (void)
 {
        key_dest = key_message;
        chat_mode = 1; // "say_team"
@@ -658,7 +632,7 @@ void Con_MessageMode2_f (void)
 Con_CommandMode_f
 ================
 */
-void Con_CommandMode_f (void)
+static void Con_CommandMode_f (void)
 {
        key_dest = key_message;
        if(Cmd_Argc() > 1)
@@ -713,7 +687,7 @@ static void Con_Maps_f (void)
                GetMapList("", NULL, 0);
 }
 
-void Con_ConDump_f (void)
+static void Con_ConDump_f (void)
 {
        int i;
        qfile_t *file;
@@ -827,14 +801,14 @@ All console printing must go through this in order to be displayed
 If no console is visible, the notify window will pop up.
 ================
 */
-void Con_PrintToHistory(const char *txt, int mask)
+static void Con_PrintToHistory(const char *txt, int mask)
 {
        // process:
        //   \n goes to next line
        //   \r deletes current line and makes a new one
 
        static int cr_pending = 0;
-       static char buf[CON_TEXTSIZE];
+       static char buf[CON_TEXTSIZE]; // con_mutex
        static int bufpos = 0;
 
        if(!con.text) // FIXME uses a non-abstracted property of con
@@ -931,7 +905,7 @@ void Con_Rcon_Redirect_Init(lhnetsocket_t *sock, lhnetaddress_t *dest, qboolean
        rcon_redirect_bufferpos = 5;
 }
 
-void Con_Rcon_Redirect_Flush(void)
+static void Con_Rcon_Redirect_Flush(void)
 {
        rcon_redirect_buffer[rcon_redirect_bufferpos] = 0;
        if (rcon_redirect_proquakeprotocol)
@@ -964,7 +938,7 @@ Con_Rcon_AddChar
 ================
 */
 /// Adds a character to the rcon buffer.
-void Con_Rcon_AddChar(int c)
+static void Con_Rcon_AddChar(int c)
 {
        if(log_dest_buffer_appending)
                return;
@@ -985,7 +959,7 @@ void Con_Rcon_AddChar(int c)
                        Log_DestBuffer_Init();
                log_dest_buffer[log_dest_buffer_pos++] = c;
                if(log_dest_buffer_pos >= sizeof(log_dest_buffer) - 1) // minus one, to allow for terminating zero
-                       Log_DestBuffer_Flush();
+                       Log_DestBuffer_Flush_NoLock();
        }
        else
                log_dest_buffer_pos = 0;
@@ -1440,7 +1414,7 @@ Modified by EvilTypeGuy eviltypeguy@qeradiant.com
 ================
 */
 extern cvar_t r_font_disable_freetype;
-void Con_DrawInput (void)
+static void Con_DrawInput (void)
 {
        int             y;
        int             i;
@@ -1479,7 +1453,8 @@ void Con_DrawInput (void)
                                int ofs = u8_bytelen(text + key_linepos, 1);
                                size_t len;
                                const char *curbuf;
-                               curbuf = u8_encodech(0xE000 + 11 + 130 * key_insert, &len);
+                               char charbuf16[16];
+                               curbuf = u8_encodech(0xE000 + 11 + 130 * key_insert, &len, charbuf16);
 
                                if (curbuf)
                                {
@@ -1517,7 +1492,8 @@ void Con_DrawInput (void)
                        {
                                size_t len;
                                const char *curbuf;
-                               curbuf = u8_encodech(0xE000 + 11 + 130 * key_insert, &len);
+                               char charbuf16[16];
+                               curbuf = u8_encodech(0xE000 + 11 + 130 * key_insert, &len, charbuf16);
                                memcpy(text, curbuf, len);
                                text[len] = 0;
                        }
@@ -1545,7 +1521,7 @@ typedef struct
 }
 con_text_info_t;
 
-float Con_WordWidthFunc(void *passthrough, const char *w, size_t *length, float maxWidth)
+static float Con_WordWidthFunc(void *passthrough, const char *w, size_t *length, float maxWidth)
 {
        con_text_info_t *ti = (con_text_info_t *) passthrough;
        if(w == NULL)
@@ -1565,7 +1541,7 @@ float Con_WordWidthFunc(void *passthrough, const char *w, size_t *length, float
        }
 }
 
-int Con_CountLineFunc(void *passthrough, const char *line, size_t length, float width, qboolean isContinuation)
+static int Con_CountLineFunc(void *passthrough, const char *line, size_t length, float width, qboolean isContinuation)
 {
        (void) passthrough;
        (void) line;
@@ -1575,7 +1551,7 @@ int Con_CountLineFunc(void *passthrough, const char *line, size_t length, float
        return 1;
 }
 
-int Con_DisplayLineFunc(void *passthrough, const char *line, size_t length, float width, qboolean isContinuation)
+static int Con_DisplayLineFunc(void *passthrough, const char *line, size_t length, float width, qboolean isContinuation)
 {
        con_text_info_t *ti = (con_text_info_t *) passthrough;
 
@@ -1596,7 +1572,7 @@ int Con_DisplayLineFunc(void *passthrough, const char *line, size_t length, floa
        return 1;
 }
 
-int Con_DrawNotifyRect(int mask_must, int mask_mustnot, float maxage, float x, float y, float width, float height, float fontsize, float alignment_x, float alignment_y, const char *continuationString)
+static int Con_DrawNotifyRect(int mask_must, int mask_mustnot, float maxage, float x, float y, float width, float height, float fontsize, float alignment_x, float alignment_y, const char *continuationString)
 {
        int i;
        int lines = 0;
@@ -1762,7 +1738,8 @@ void Con_DrawNotify (void)
                //static char *cursor[2] = { "\xee\x80\x8a", "\xee\x80\x8b" }; // { off, on }
                int colorindex = -1;
                const char *cursor;
-               cursor = u8_encodech(0xE00A + ((int)(realtime * con_cursorspeed)&1), NULL);
+               char charbuf16[16];
+               cursor = u8_encodech(0xE00A + ((int)(realtime * con_cursorspeed)&1), NULL, charbuf16);
 
                // LordHavoc: speedup, and other improvements
                if (chat_mode < 0)
@@ -1788,7 +1765,7 @@ Con_LineHeight
 Returns the height of a given console line; calculates it if necessary.
 ================
 */
-int Con_LineHeight(int lineno)
+static int Con_LineHeight(int lineno)
 {
        con_lineinfo_t *li = &CON_LINES(lineno);
        if(li->height == -1)
@@ -1812,7 +1789,7 @@ If alpha is 0, the line is not drawn, but still wrapped and its height
 returned.
 ================
 */
-int Con_DrawConsoleLine(int mask_must, int mask_mustnot, float y, int lineno, float ymin, float ymax)
+static int Con_DrawConsoleLine(int mask_must, int mask_mustnot, float y, int lineno, float ymin, float ymax)
 {
        float width = vid_conwidth.value;
        con_text_info_t ti;
@@ -2208,7 +2185,7 @@ void Con_DisplayList(const char **list)
        SanitizeString strips color tags from the string in
        and writes the result on string out
 */
-void SanitizeString(char *in, char *out)
+static void SanitizeString(char *in, char *out)
 {
        while(*in)
        {
@@ -2263,7 +2240,7 @@ static int Nicks_offset[MAX_SCOREBOARD]; // when nicks use a space, we need this
 static int Nicks_matchpos;
 
 // co against <<:BLASTER:>> is true!?
-int Nicks_strncasecmp_nospaces(char *a, char *b, unsigned int a_len)
+static int Nicks_strncasecmp_nospaces(char *a, char *b, unsigned int a_len)
 {
        while(a_len)
        {
@@ -2289,7 +2266,7 @@ int Nicks_strncasecmp_nospaces(char *a, char *b, unsigned int a_len)
        }
        return 0;
 }
-int Nicks_strncasecmp(char *a, char *b, unsigned int a_len)
+static int Nicks_strncasecmp(char *a, char *b, unsigned int a_len)
 {
        char space_char;
        if(!(con_nickcompletion_flags.integer & NICKS_ALPHANUMERICS_ONLY))
@@ -2340,7 +2317,7 @@ int Nicks_strncasecmp(char *a, char *b, unsigned int a_len)
 
    Count the number of possible nicks to complete
  */
-int Nicks_CompleteCountPossible(char *line, int pos, char *s, qboolean isCon)
+static int Nicks_CompleteCountPossible(char *line, int pos, char *s, qboolean isCon)
 {
        char name[128];
        int i, p;
@@ -2407,14 +2384,14 @@ int Nicks_CompleteCountPossible(char *line, int pos, char *s, qboolean isCon)
        return count;
 }
 
-void Cmd_CompleteNicksPrint(int count)
+static void Cmd_CompleteNicksPrint(int count)
 {
        int i;
        for(i = 0; i < count; ++i)
                Con_Printf("%s\n", Nicks_list[i]);
 }
 
-void Nicks_CutMatchesNormal(int count)
+static void Nicks_CutMatchesNormal(int count)
 {
        // cut match 0 down to the longest possible completion
        int i;
@@ -2437,7 +2414,7 @@ void Nicks_CutMatchesNormal(int count)
        //Con_Printf("List0: %s\n", Nicks_sanlist[0]);
 }
 
-unsigned int Nicks_strcleanlen(const char *s)
+static unsigned int Nicks_strcleanlen(const char *s)
 {
        unsigned int l = 0;
        while(*s)
@@ -2452,7 +2429,7 @@ unsigned int Nicks_strcleanlen(const char *s)
        return l;
 }
 
-void Nicks_CutMatchesAlphaNumeric(int count)
+static void Nicks_CutMatchesAlphaNumeric(int count)
 {
        // cut match 0 down to the longest possible completion
        int i;
@@ -2511,7 +2488,7 @@ void Nicks_CutMatchesAlphaNumeric(int count)
        }
 }
 
-void Nicks_CutMatchesNoSpaces(int count)
+static void Nicks_CutMatchesNoSpaces(int count)
 {
        // cut match 0 down to the longest possible completion
        int i;
@@ -2567,7 +2544,7 @@ void Nicks_CutMatchesNoSpaces(int count)
        }
 }
 
-void Nicks_CutMatches(int count)
+static void Nicks_CutMatches(int count)
 {
        if(con_nickcompletion_flags.integer & NICKS_ALPHANUMERICS_ONLY)
                Nicks_CutMatchesAlphaNumeric(count);
@@ -2577,7 +2554,7 @@ void Nicks_CutMatches(int count)
                Nicks_CutMatchesNormal(count);
 }
 
-const char **Nicks_CompleteBuildList(int count)
+static const char **Nicks_CompleteBuildList(int count)
 {
        const char **buf;
        int bpos = 0;
@@ -2597,7 +2574,7 @@ const char **Nicks_CompleteBuildList(int count)
        Nicks_AddLastColor
        Restores the previous used color, after the autocompleted name.
 */
-int Nicks_AddLastColor(char *buffer, int pos)
+static int Nicks_AddLastColor(char *buffer, int pos)
 {
        qboolean quote_added = false;
        int match;
@@ -2718,6 +2695,7 @@ void Con_CompleteCommandLine (void)
        int c, v, a, i, cmd_len, pos, k;
        int n; // nicks --blub
        const char *space, *patterns;
+       char vabuf[1024];
 
        //find what we want to complete
        pos = key_linepos;
@@ -2738,7 +2716,7 @@ void Con_CompleteCommandLine (void)
        {
                strlcpy(command, key_line + 1, min(sizeof(command), (unsigned int)(space - key_line)));
 
-               patterns = Cvar_VariableString(va("con_completion_%s", command)); // TODO maybe use a better place for this?
+               patterns = Cvar_VariableString(va(vabuf, sizeof(vabuf), "con_completion_%s", command)); // TODO maybe use a better place for this?
                if(patterns && !*patterns)
                        patterns = NULL; // get rid of the empty string