RESOLVED WORKSFORME 125434
check-webkit-style thinks that the super-constructor initialization needs "Comma should be at the beginning of the line in a member initialization list." 
https://bugs.webkit.org/show_bug.cgi?id=125434
Summary check-webkit-style thinks that the super-constructor initialization needs "Co...
Filip Pizlo
Reported 2013-12-08 20:05:33 PST
The following code: class SSALoweringPhase : public Phase { static const bool verbose = false; public: SSALoweringPhase(Graph& graph) : Phase(graph, "SSA lowering") , m_insertionSet(graph) { } Causes check-webkit-style to claim that the ": Phase(graph, ..." line needs a comma.
Attachments
Add attachment proposed patch, testcase, etc.
Laszlo Vidacs
Comment 1 2013-12-18 01:58:17 PST
Failed to reproduce the error. No error message issued with r160755, tried on the example code and on DFGSSALoweringPhase.cpp
Laszlo Vidacs
Comment 2 2014-03-21 03:41:31 PDT
No error message in 166057.
Note You need to log in before you can comment on or make changes to this bug.