Project

General

Profile

Actions

Bug #36

closed

Undefinierter Rückgabewert bei isalpha(), isdigit(), isupper(),...

Added by Michael Onken about 18 years ago. Updated almost 11 years ago.

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.

Actions #1

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 #2

Updated by Michael Onken about 11 years ago

  • Private changed from No to Yes
Actions #3

Updated by Michael Onken almost 11 years ago

  • Private changed from Yes to No
Actions

Also available in: Atom PDF