Closed
Bug 1855681
Opened 1 year ago
Closed 1 year ago
apply performance-avoid-endl on the code base
Categories
(Developer Infrastructure :: Source Code Analysis, task)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox120 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox120 | --- | fixed |
People
(Reporter: Sylvestre, Assigned: Sylvestre)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Assignee | ||
Comment 1•1 year ago
|
||
done with:
./mach static-analysis check --checks="-*, performance-avoid-endl" --fix security/manager/ssl/tests/unit/tlsserver/cmd/DelegatedCredentialsServer.cpp toolkit/crashreporter/client/crashreporter_linux.cpp dom/media/webrtc/transport/test/nrappkit_unittest.cpp gfx/layers/apz/src/CheckerboardEvent.cpp dom/media/webrtc/transport/test/sockettransportservice_unittest.cpp toolkit/crashreporter/client/crashreporter.cpp dom/media/gtest/TestAudioPacketizer.cpp dom/media/webrtc/sdp/SdpHelper.cpp
dom/media/gmp-plugin-openh264/gmp-fake-openh264.cpp had to be done by hand because of macro
Assignee | ||
Comment 2•1 year ago
|
||
See:
https://clang.llvm.org/extra/clang-tidy/checks/performance/avoid-endl.html
for the rational
Updated•1 year ago
|
Assignee: nobody → sledru
Status: NEW → ASSIGNED
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4fc4397c17c9
Replace std::endl by '\n' in the code base r=emilio,media-playback-reviewers,stransky
Comment 4•1 year ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•