Eugene.Zelenko added a subscriber: Eugene.Zelenko.
Eugene.Zelenko added a comment.

Please also run Clang-tidy modernize checks.


================
Comment at: source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp:895
@@ +894,3 @@
+
+
+void
----------------
Unnecessary line.

================
Comment at: source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h:300
@@ -297,2 +299,3 @@
 public:
+    PythonList() {}
     explicit PythonList(PyInitialValue value);
----------------
Should be PythonList() = default; Same for other similar default constructors.

================
Comment at: source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h:337
@@ +336,3 @@
+
+    ~PythonTuple() override;
+
----------------
Should be ~PythonTuple() override = default; Same for PythonCallable.


http://reviews.llvm.org/D14555



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to