Closed
Bug 1318299
Opened 8 years ago
Closed 8 years ago
Convert some code in security/manager to C++11
Categories
(Developer Infrastructure :: Source Code Analysis, defect)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox53 fixed)
RESOLVED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: andi, Assigned: andi)
References
Details
Attachments
(3 files)
Using the checkers provided by clang-tidy we can refresh the code to make it use the features of C++11 like: - auto variables declarations - default CTORS and DTORS - using new range loop operators
Assignee | ||
Updated•8 years ago
|
Summary: Convert some code in security/manager/ to C++11 → Convert some code in security/manager to C++11
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → bpostelnicu
Assignee | ||
Updated•8 years ago
|
Attachment #8811681 -
Flags: review?(dkeeler)
Attachment #8811682 -
Flags: review?(dkeeler)
Attachment #8811683 -
Flags: review?(dkeeler)
Comment 4•8 years ago
|
||
mozreview-review |
Comment on attachment 8811681 [details] Bug 1318299 - Converts for(...; ...; ...) loops to use the new range-based loops in C++11 in security/manager. https://reviewboard.mozilla.org/r/93706/#review93934 ::: security/manager/ssl/tests/unit/tlsserver/cmd/GenerateOCSPResponse.cpp:80 (Diff revision 1) > /*out*/ OCSPResponseType* OCSPType) > { > if (!OCSPType) { > return false; > } > - for (uint32_t i = 0; i < mozilla::ArrayLength(kOCSPResponseNameList); i++) { > + for (auto i : kOCSPResponseNameList) { 'i' isn't very descriptive. Let's maybe use 'ocspResponseName' to reflect the underlying type.
Attachment #8811681 -
Flags: review?(dkeeler) → review+
Comment 5•8 years ago
|
||
mozreview-review |
Comment on attachment 8811682 [details] Bug 1318299 - Replace string literals containing escaped characters with raw string literals in security/manager. https://reviewboard.mozilla.org/r/93708/#review93964
Attachment #8811682 -
Flags: review?(dkeeler) → review+
Comment 6•8 years ago
|
||
mozreview-review |
Comment on attachment 8811683 [details] Bug 1318299 - Use C++11's override and remove virtual where applicable in security/manager. https://reviewboard.mozilla.org/r/93710/#review93968 Great - thanks!
Attachment #8811683 -
Flags: review?(dkeeler) → review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 10•8 years ago
|
||
Pushed by bpostelnicu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9fad88b953cc Converts for(...; ...; ...) loops to use the new range-based loops in C++11 in security/manager. r=keeler https://hg.mozilla.org/integration/autoland/rev/beeffee0c4ee Replace string literals containing escaped characters with raw string literals in security/manager. r=keeler https://hg.mozilla.org/integration/autoland/rev/1c1707b6cef5 Use C++11's override and remove virtual where applicable in security/manager. r=keeler
Comment 11•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9fad88b953cc https://hg.mozilla.org/mozilla-central/rev/beeffee0c4ee https://hg.mozilla.org/mozilla-central/rev/1c1707b6cef5
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Comment 12•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9fad88b953cc https://hg.mozilla.org/mozilla-central/rev/beeffee0c4ee https://hg.mozilla.org/mozilla-central/rev/1c1707b6cef5
Updated•6 years ago
|
Product: Core → Firefox Build System
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•