Eugene.Zelenko added a comment.

Please run Clang-format and Clang-tidy modernize.



================
Comment at: clang-doc/generators/Generators.h:46
+public:
+  MDGenerator(std::unique_ptr<InfoSet> &IS, StringRef Root, StringRef Format) 
: Generator(IS, Root, Format) {};
+  virtual ~MDGenerator() {};
----------------
Please remove semicolon after constructor body. Please enable CLang's 
-Wextra-semi.


================
Comment at: clang-doc/generators/Generators.h:47
+  MDGenerator(std::unique_ptr<InfoSet> &IS, StringRef Root, StringRef Format) 
: Generator(IS, Root, Format) {};
+  virtual ~MDGenerator() {};
+  
----------------
Please use = default;


https://reviews.llvm.org/D43424



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

Reply via email to