Open Bug 1485590 Opened 7 years ago Updated 3 years ago

NSPR uses isdigit()

Categories

(NSPR :: NSPR, defect, P2)

Tracking

(Not tracked)

People

(Reporter: hsivonen, Unassigned)

References

(Blocks 1 open bug)

Details

NSPR uses isdigit(): https://searchfox.org/mozilla-central/search?q=isdigit&case=true&regexp=false&path=nsprpub Per https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/isdigit-iswdigit-isdigit-l-iswdigit-l isdigit() locale-dependent on Windows and https://stackoverflow.com/questions/2898228/can-isdigit-legitimately-be-locale-dependent-in-c gives an example of how. mozilla::IsAsciiDigit() should be used in place of isdigit() in C++ code, but since NSPR is C, it should probably have its own isdigit() replacement.
Blocks: 1485587
Priority: -- → P2
QA Contact: jjones
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.