ncurses 6.2 - patch 20200531
[ncurses.git] / ncurses / base / lib_slk.c
index 8fafdbca83832530aa11f347106bd107e58f6aaf..f4b3ce6dbf0ae0fd9c06a9f3353b9fe20d18fd70 100644 (file)
@@ -1,5 +1,6 @@
 /****************************************************************************
- * Copyright (c) 1998-2009,2010 Free Software Foundation, Inc.              *
+ * Copyright 2020 Thomas E. Dickey                                          *
+ * Copyright 1998-2010,2011 Free Software Foundation, Inc.                  *
  *                                                                          *
  * Permission is hereby granted, free of charge, to any person obtaining a  *
  * copy of this software and associated documentation files (the            *
@@ -47,7 +48,7 @@
 #define CUR SP_TERMTYPE
 #endif
 
-MODULE_ID("$Id: lib_slk.c,v 1.45 2010/05/01 20:44:34 tom Exp $")
+MODULE_ID("$Id: lib_slk.c,v 1.49 2020/02/02 23:34:34 tom Exp $")
 
 #ifdef USE_TERM_DRIVER
 #define NumLabels    InfoOf(SP_PARM).numlabels
@@ -79,7 +80,7 @@ NCURSES_EXPORT(int)
 _nc_format_slks(NCURSES_SP_DCLx int cols)
 {
     int gap, i, x;
-    unsigned max_length;
+    int max_length;
 
     if (!SP_PARM || !SP_PARM->_slk)
        return ERR;
@@ -98,7 +99,7 @@ _nc_format_slks(NCURSES_SP_DCLx int cols)
        }
     } else {
        if (SP_PARM->slk_format == 2) {         /* 4-4 */
-           gap = cols - (SP_PARM->_slk->maxlab * max_length) - 6;
+           gap = cols - (int) (SP_PARM->_slk->maxlab * max_length) - 6;
 
            if (gap < 1)
                gap = 1;
@@ -138,7 +139,7 @@ _nc_slk_initialize(WINDOW *stwin, int cols)
 {
     int i;
     int res = OK;
-    unsigned max_length;
+    size_t max_length;
     SCREEN *sp;
     int numlab;
 
@@ -172,24 +173,24 @@ _nc_slk_initialize(WINDOW *stwin, int cols)
     else
        SetAttr(SP_PARM->_slk->attr, A_REVERSE);
 
-    SP_PARM->_slk->maxlab = ((numlab > 0)
-                            ? numlab
-                            : MAX_SKEY(SP_PARM->slk_format));
-    SP_PARM->_slk->maxlen = ((numlab > 0)
-                            ? LabelWidth * LabelHeight
-                            : MAX_SKEY_LEN(SP_PARM->slk_format));
-    SP_PARM->_slk->labcnt = ((SP_PARM->_slk->maxlab < MAX_SKEY(SP_PARM->slk_format))
-                            ? MAX_SKEY(SP_PARM->slk_format)
-                            : SP_PARM->_slk->maxlab);
+    SP_PARM->_slk->maxlab = (short) ((numlab > 0)
+                                    ? numlab
+                                    : MAX_SKEY(SP_PARM->slk_format));
+    SP_PARM->_slk->maxlen = (short) ((numlab > 0)
+                                    ? LabelWidth * LabelHeight
+                                    : MAX_SKEY_LEN(SP_PARM->slk_format));
+    SP_PARM->_slk->labcnt = (short) ((SP_PARM->_slk->maxlab < MAX_SKEY(SP_PARM->slk_format))
+                                    ? MAX_SKEY(SP_PARM->slk_format)
+                                    : SP_PARM->_slk->maxlab);
 
     if (SP_PARM->_slk->maxlen <= 0
        || SP_PARM->_slk->labcnt <= 0
        || (SP_PARM->_slk->ent = typeCalloc(slk_ent,
-                                           (unsigned) SP_PARM->_slk->labcnt))
+                                           (size_t) SP_PARM->_slk->labcnt))
        == NULL)
        returnCode(slk_failed(NCURSES_SP_ARG));
 
-    max_length = SP_PARM->_slk->maxlen;
+    max_length = (size_t) SP_PARM->_slk->maxlen;
     for (i = 0; i < SP_PARM->_slk->labcnt; i++) {
        size_t used = max_length + 1;
 
@@ -201,9 +202,12 @@ _nc_slk_initialize(WINDOW *stwin, int cols)
        SP_PARM->_slk->ent[i].form_text = (char *) _nc_doalloc(0, used);
        if (SP_PARM->_slk->ent[i].form_text == 0)
            returnCode(slk_failed(NCURSES_SP_ARG));
-       memset(SP_PARM->_slk->ent[i].form_text, 0, used);
 
-       memset(SP_PARM->_slk->ent[i].form_text, ' ', max_length);
+       if (used > 1) {
+           memset(SP_PARM->_slk->ent[i].form_text, ' ', used - 1);
+       }
+       SP_PARM->_slk->ent[i].form_text[used - 1] = '\0';
+
        SP_PARM->_slk->ent[i].visible = (char) (i < SP_PARM->_slk->maxlab);
     }