Open Bug 1582069 Opened 6 years ago Updated 3 years ago

Consider to remove PR_CALLBACK support

Categories

(NSPR :: NSPR, task)

Tracking

(Not tracked)

People

(Reporter: KaiE, Unassigned)

Details

In a recent review, glandium suggested that we remove PR_CALLBACK from NSPR headers.

Note that both NSPR and NSS use it.

An advantage of keeping the PR_CALLBACK is, it will be less work, should we ever again need a similar definition to be added to functions.

I cannot predict if that's realistic to happen or not. IMHO keeping it doesn't hurt, given that we don't modify NSPR much.

It's true that we might forget adding the symbol for new functions, but if they'll ever be needed again, it will be less work to add the missing ones.

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.