Closed Bug 298962 Opened 15 years ago Closed 15 years ago

Remove nssBestCertificate related dead code

Categories

(NSS :: Libraries, defect, trivial)

3.10
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: wtc, Assigned: wtc)

Details

Attachments

(1 file)

In bug 135521, nssBestCertificate_SetArgs and nssBestCertificate_Callback
were removed, but the removal was not done cleanly.  Some dead code is
still around.
Attached patch Proposed patchSplinter Review
Removed unused functions nssBestCertificate_SetArgs and
nssBestCertificate_Callback and unused type nssBestCertificateCB.
It is easy to verify with LXR that they are unused.

(The two functions used to be declared in lib/pki/pkim.h.)
Attachment #187462 - Flags: superreview?(nelson)
Attachment #187462 - Flags: review?(julien.pierre.bugs)
Attachment #187462 - Flags: review?(julien.pierre.bugs) → review+
Comment on attachment 187462 [details] [diff] [review]
Proposed patch

I'm all for removing dead code.
Attachment #187462 - Flags: superreview?(nelson) → superreview+
Patch checked in on the NSS trunk for NSS 3.11.

Checking in certificate.c;
/cvsroot/mozilla/security/nss/lib/pki/certificate.c,v  <--  certificate.c
new revision: 1.55; previous revision: 1.54
done
Checking in pkitm.h;
/cvsroot/mozilla/security/nss/lib/pki/pkitm.h,v  <--  pkitm.h
new revision: 1.14; previous revision: 1.13
done
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.11
You need to log in before you can comment on or make changes to this bug.