Closed Bug 1092028 Opened 5 years ago Closed 5 years ago

Fix -Wunused-const-variable warning in security/pkix/test/gtest

Categories

(Core :: Security: PSM, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla36
Tracking Status
firefox35 --- unaffected
firefox36 --- fixed

People

(Reporter: cpeterson, Assigned: cpeterson)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

ipv4_addr_bytes_FFFFFFFF was added to pkixnames_tests.cpp in bug 1089393, but it is never referenced. This directory is marked FAIL_ON_WARNINGS, so I'm not sure why neither the build machines nor other developers hit this warning-as-error:

> security/pkix/test/gtest/pkixnames_tests.cpp:1030:22: error: unused variable 'ipv4_addr_bytes_FFFFFFFF' [-Werror,-Wunused-const-variable]

Brian, did you intend to add test cases that would use ipv4_addr_bytes_FFFFFFFF?
Attachment #8514804 - Flags: review?(brian)
(In reply to Chris Peterson (needinfo? :cpeterson) from comment #0)
> This directory is marked FAIL_ON_WARNINGS, so
> I'm not sure why neither the build machines nor other developers hit this
> warning-as-error:

(I just hit it myself, FWIW, and almost filed a dupe of this bug. I think -Wunused-const-variable is a relatively new clang-only warning, and the build machines don't have new-enough clang to report it.)
OS: Mac OS X → All
Hardware: x86 → All
Version: unspecified → Trunk
Comment on attachment 8514804 [details] [diff] [review]
fix-pkix-warning.patch

Review of attachment 8514804 [details] [diff] [review]:
-----------------------------------------------------------------

This was the result of a bad merge. This unused value will be used in an upcoming patch series.
Attachment #8514804 - Flags: review?(brian) → review+
https://hg.mozilla.org/mozilla-central/rev/218b552c815b
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
See Also: → 1108637
You need to log in before you can comment on or make changes to this bug.