Actions
Bug #36
closedUndefinierter Rückgabewert bei isalpha(), isdigit(), isupper(),...
Status:
Closed
Priority:
High
Assignee:
-
Category:
-
Target version:
-
Start date:
Due date:
% Done:
100%
Estimated time:
Module:
alle
Operating System:
Compiler:
Description
Für DcmItem ist ein Bug Report eingegangen (und gefixt worden), bei dem
char-Werte an isalpha() übergeben wurden. isalpha() arbeitet aber nur
auf vorzeichenlosen Werten (und -1 = EOF). Es gibt noch viele weitere Stellen
im Toolkit, an denen beispielsweise isalpha(), isdigit() und isupper()
mit dem Typ char aufgerufen werden. Konsequent wäre es, an allen diesen
Stellen unsigned char bzw. Uint8 zu verwenden oder entsprechend zu casten.
Alternativ darauf warten, daß jemand damit Probleme bekommt...;)
US: Ich hab kiloweise Casts eingebaut. //EDIT//: Habe auch noch toupper() / tolower() erledigt.
Updated by Uli Schlachter almost 13 years ago
For future reference, the list of functions which take an unsigned char cast to int as their argument:
$ git grep -E 'toupper|tolower|isalnum|isalpha|isascii|isblank|iscntrl|isdigit|isgraph|islower|isprint|ispunct|isspace|isupper|isxdigit'
Actions