PL_HashTableDump should be obsoleted

NEW
Assigned to

Status

NSPR
NSPR
9 years ago
8 years ago

People

(Reporter: Julien Pierre, Assigned: Wan-Teh Chang)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

9 years ago
This API takes a FILE* . This is a problem, particularly on Windows, where the caller of NSPR may not be using the same CRT as NSPR itself, due to differences in compilers, or debug heap CRT, etc.
IMO, an API that takes a FILE* has no place in NSPR. This function appears to be mostly for debugging.

I would suggest we mark it as obsolete as a minimum, and perhaps make it return PR_NOT_IMPLEMENTED_ERROR .

If somebody really needs this functionality, we could rewrite the function to take a PRFileDesc* instead.
Assignee: julien.pierre.boogz → wtc
You need to log in before you can comment on or make changes to this bug.