Skip to content

Commit 2827b74

Browse files
MallocBench: Remove unused private field
https://bugs.webkit.org/show_bug.cgi?id=213989 Patch by Philippe Normand <[email protected]> on 2020-07-06 Reviewed by Alex Christensen. The m_argc private field is never used. Remove it. * MallocBench/MallocBench/CommandLine.cpp: * MallocBench/MallocBench/CommandLine.h: git-svn-id: http://svn.webkit.org/repository/webkit/trunk@263972 268f45cc-cd09-0410-ab3c-d52691b4dbfc
1 parent d8c8302 commit 2827b74

File tree

3 files changed

+14
-4
lines changed

3 files changed

+14
-4
lines changed

PerformanceTests/ChangeLog

+12
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,15 @@
1+
2020-07-06 Philippe Normand <[email protected]>
2+
3+
MallocBench: Remove unused private field
4+
https://bugs.webkit.org/show_bug.cgi?id=213989
5+
6+
Reviewed by Alex Christensen.
7+
8+
The m_argc private field is never used. Remove it.
9+
10+
* MallocBench/MallocBench/CommandLine.cpp:
11+
* MallocBench/MallocBench/CommandLine.h:
12+
113
2020-07-01 Saam Barati <[email protected]>
214

315
Make it so JetStream2 can run with the d8 shell

PerformanceTests/MallocBench/MallocBench/CommandLine.cpp

+1-2
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,7 @@ struct option CommandLine::longOptions[] =
4040
};
4141

4242
CommandLine::CommandLine(int argc, char** argv)
43-
: m_argc(argc)
44-
, m_argv(argv)
43+
: m_argv(argv)
4544
, m_detailedReport(false)
4645
, m_isParallel(false)
4746
, m_useThreadID(false)

PerformanceTests/MallocBench/MallocBench/CommandLine.h

+1-2
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ class CommandLine {
4646
private:
4747
static struct option longOptions[];
4848

49-
int m_argc;
5049
char** m_argv;
5150
std::string m_benchmarkName;
5251
bool m_detailedReport;
@@ -57,4 +56,4 @@ class CommandLine {
5756
size_t m_runs;
5857
};
5958

60-
#endif // CommandLine_h
59+
#endif // CommandLine_h

0 commit comments

Comments
 (0)