Rename the PRErrorCode variable |status| to |error| in tstclnt.c

RESOLVED FIXED in 3.15.4

Status

P2
trivial
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: wtc, Assigned: wtc)

Tracking

3.13.2
3.15.4

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 814632 [details] [diff] [review]
Patch

tstclnt.c has a local variable that is poorly named:
    PRErrorCode status;
because |status| is conventionally used with the PRStatus or SECStatus type.
The variable should be renamed |error|, which is also the name of the argument
for SSL_AuthCertificateComplete.
Attachment #814632 - Flags: review?(brian)
Attachment #814632 - Flags: review?(brian) → review+
(Assignee)

Comment 1

5 years ago
Patch checked in: https://hg.mozilla.org/projects/nss/rev/c623bea95f82
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Comment 2

5 years ago
changing target milestone to 3.15.4
Target Milestone: 3.15.3 → 3.15.4
You need to log in before you can comment on or make changes to this bug.