Closed
Bug 1422843
Opened 7 years ago
Closed 6 years ago
Silence false positive ABI warnings in NSS 3.35
Categories
(NSS :: Libraries, enhancement)
Tracking
(Not tracked)
RESOLVED
FIXED
3.35
People
(Reporter: KaiE, Assigned: KaiE)
Details
Attachments
(2 files)
956 bytes,
patch
|
franziskus
:
review+
KaiE
:
checked-in+
|
Details | Diff | Splinter Review |
1.73 KB,
patch
|
franziskus
:
review+
KaiE
:
checked-in+
|
Details | Diff | Splinter Review |
Our ABI checks can report changes that aren't really ABI breakage. Let's use this bug to track the commits required to silence the false positive warnings.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → kaie
Assignee | ||
Comment 1•7 years ago
|
||
Attachment #8934186 -
Flags: review?(franziskuskiefer)
Updated•7 years ago
|
Attachment #8934186 -
Flags: review?(franziskuskiefer) → review+
Assignee | ||
Comment 2•7 years ago
|
||
https://hg.mozilla.org/projects/nss/rev/f8536fe0e128 I'll keep the bug open. There might be more commits necessary during the 3.35 development period.
Assignee | ||
Updated•6 years ago
|
Target Milestone: --- → 3.35
Assignee | ||
Comment 3•6 years ago
|
||
Attachment #8942184 -
Flags: review?(franziskuskiefer)
Assignee | ||
Updated•6 years ago
|
Summary: Silence false positive warnings about ABI changes in NSS 3.35 → Silence false positive ABI warnings in NSS 3.35
Updated•6 years ago
|
Attachment #8942184 -
Flags: review?(franziskuskiefer) → review+
Assignee | ||
Updated•6 years ago
|
Attachment #8934186 -
Flags: checked-in+
Assignee | ||
Comment 4•6 years ago
|
||
Comment on attachment 8942184 [details] [diff] [review] 1422843-b.patch https://hg.mozilla.org/projects/nss/rev/c5317ccdb021
Attachment #8942184 -
Flags: checked-in+
Assignee | ||
Comment 5•6 years ago
|
||
Patch -b used an incorrect function signature. It was created by an older abidiff version 1.0-RC6, which apparently contained a bug. Because the NSS 3.35 release is completed, let's fix this in bug 1432460 that tracks NSS 3.36
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•