Closed
Bug 924684
Opened 11 years ago
Closed 11 years ago
Rename the PRErrorCode variable |status| to |error| in tstclnt.c
Categories
(NSS :: Test, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
3.15.4
People
(Reporter: wtc, Assigned: wtc)
Details
Attachments
(1 file)
|
1.50 KB,
patch
|
briansmith
:
review+
|
Details | Diff | Splinter Review |
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)
|
||
Updated•11 years ago
|
Attachment #814632 -
Flags: review?(brian) → review+
|
Assignee | |
Comment 1•11 years ago
|
||
Patch checked in: https://hg.mozilla.org/projects/nss/rev/c623bea95f82
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•