[clang 13] unused-but-set-variable bulk fix
Categories
(Firefox Build System :: General, defect)
Tracking
(firefox95 fixed)
Tracking | Status | |
---|---|---|
firefox95 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(10 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review |
Assignee | ||
Comment 1•3 years ago
|
||
gfx/2d/SourceSurfaceD2D1.cpp(201,20): error: variable 'options' set but not used [-Werror,-Wunused-but-set-variable]
D2D1_MAP_OPTIONS options;
^
gfx/thebes/gfxDWriteFontList.h(269,13): error: variable 'hr' set but not used [-Werror,-Wunused-but-set-variable]
HRESULT hr = S_OK;
^
gfx/thebes/gfxGDIFontList.cpp(1020,12): error: variable 'cmapLoaded' set but not used [-Werror,-Wunused-but-set-variable]
bool cmapLoaded = false;
^
gfx/thebes/gfxMacPlatformFontList.mm:1771:10: error: variable 'isStandardFace' set but not used [-Werror,-Wunused-but-set-variable]
bool isStandardFace = false;
^
gfx/thebes/gfxPlatformFontList.cpp(1064,18): error: variable 'rejectedFallbackVisibility' set but not used [-Werror,-Wunused-but-set-variable]
FontVisibility rejectedFallbackVisibility = FontVisibility::Unknown;
^
gfx/vr/service/OculusSession.cpp(1329,12): error: variable 'bNewController' set but not used [-Werror,-Wunused-but-set-variable]
bool bNewController = false;
^
Assignee | ||
Comment 2•3 years ago
|
||
ipc/chromium/src/chrome/common/ipc_channel_win.cc(133,8): error: variable 'waited' set but not used [-Werror,-Wunused-but-set-variable]
bool waited = false;
^
ipc/mscom/InterceptorLog.cpp(183,12): error: variable 'rv' set but not used [-Werror,-Wunused-but-set-variable]
nsresult rv = mThread->Dispatch(
^
Assignee | ||
Comment 3•3 years ago
|
||
vm/TraceLogging.cpp:1685:12: error: variable 'totalJSTime' set but not used [-Werror,-Wunused-but-set-variable]
uint32_t totalJSTime = 0;
^
wasm/WasmBinary.h:175:15: error: variable 'assertByte' set but not used [-Werror,-Wunused-but-set-variable]
uint8_t assertByte = assertBits & 0x7f;
^
jit/x86-shared/MacroAssembler-x86-shared.cpp:406:11: error: variable 'diffG' set but not used [-Werror,-Wunused-but-set-variable]
int32_t diffG = set.gprs().size() * sizeof(intptr_t);
^
jit/x86-shared/MacroAssembler-x86-shared.cpp:404:12: error: variable 'numFpu' set but not used [-Werror,-Wunused-but-set-variable]
unsigned numFpu = fpuSet.size();
^
jit/x86-shared/MacroAssembler-x86-shared.cpp:455:11: error: variable 'diffG' set but not used [-Werror,-Wunused-but-set-variable]
int32_t diffG = set.gprs().size() * sizeof(intptr_t);
^
jit/x86-shared/MacroAssembler-x86-shared.cpp:453:12: error: variable 'numFpu' set but not used [-Werror,-Wunused-but-set-variable]
unsigned numFpu = fpuSet.size();
^
jit/x86-shared/MacroAssembler-x86-shared.cpp:503:12: error: variable 'numFpu' set but not used [-Werror,-Wunused-but-set-variable]
unsigned numFpu = fpuSet.size();
^
Assignee | ||
Comment 4•3 years ago
|
||
js/xpconnect/src/XPCJSRuntime.cpp:2438:10: error: variable 'gcThingTotal' set but not used [-Werror,-Wunused-but-set-variable]
size_t gcThingTotal = 0;
^
Assignee | ||
Comment 5•3 years ago
|
||
parser/html/nsHtml5StreamParser.cpp:1046:10: error: variable 'totalRead' set but not used [-Werror,-Wunused-but-set-variable]
size_t totalRead = 0;
^
Assignee | ||
Comment 6•3 years ago
|
||
platform-linux-android.cpp:199:9: error: variable 'r' set but not used [-Werror,-Wunused-but-set-variable]
int r = sem_init(&mMessage2, /* pshared */ 0, 0);
^
platform-linux-android.cpp:206:9: error: variable 'r' set but not used [-Werror,-Wunused-but-set-variable]
not used [-Werror,-Wunused-but-set-variable]
int r = sem_destroy(&mMessage2);
^
Assignee | ||
Comment 7•3 years ago
|
||
toolkit/crashreporter/breakpad-client/windows/crash_generation/minidump_generator.cc(319,35): error: variable 'dump_exception_pointers' set but not used [-Werror,-Wunused-but-set-variable]
MINIDUMP_EXCEPTION_INFORMATION* dump_exception_pointers = NULL;
^
Assignee | ||
Comment 8•3 years ago
|
||
widget/windows/nsAppShell.cpp(685,10): error: variable 'uiMessage' set but not used [-Werror,-Wunused-but-set-variable]
bool uiMessage = false;
^
widget/windows/TaskbarPreview.cpp(370,11): error: variable 'hr' set but not used [-Werror,-Wunused-but-set-variable]
HRESULT hr;
^
Assignee | ||
Comment 9•3 years ago
|
||
xpcom/tests/gtest/TestAvailableMemoryWatcherWin.cpp(130,22): error: variable 'x' set but not used [-Werror,-Wunused-but-set-variable]
volatile uint8_t x = 0;
^
xpcom/tests/gtest/TestAvailableMemoryWatcherWin.cpp(152,12): error: variable 'consumed' set but not used [-Werror,-Wunused-but-set-variable]
size_t consumed = 0;
^
xpcom/tests/gtest/TestSTLWrappers.cpp:48:7: error: variable 'rv' set but not used [-Werror,-Wunused-but-set-variable]
int rv = 1;
^
Assignee | ||
Comment 10•3 years ago
|
||
Assignee | ||
Updated•3 years ago
|
Comment 11•3 years ago
|
||
Comment 12•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2ec6f0c93be9
https://hg.mozilla.org/mozilla-central/rev/23d04c1a252e
https://hg.mozilla.org/mozilla-central/rev/7e04dd613276
https://hg.mozilla.org/mozilla-central/rev/3799a2ddc4da
https://hg.mozilla.org/mozilla-central/rev/d7e13f9ba1d3
https://hg.mozilla.org/mozilla-central/rev/bf67b9659820
https://hg.mozilla.org/mozilla-central/rev/30d2381f3062
https://hg.mozilla.org/mozilla-central/rev/3e99d43f5199
https://hg.mozilla.org/mozilla-central/rev/72cb687968f8
Assignee | ||
Updated•3 years ago
|
Comment 13•3 years ago
|
||
Comment 14•3 years ago
|
||
bugherder |
Description
•