ioeric added a comment.

In, @hokein wrote:

> Sorry for the delay. 
>  The patch only contains an unittest for `HeaderGenerato`r, which is not 
> quite enough. Should we create a fake migrate-tool binary to illustrate APIs 
> usage?


Good idea. I wanted to add a dummy tool once the interfaces are stable. But I 
guess now is the right time.

Comment at: migrate-tool/MigrateTool.h:50
@@ +49,3 @@
+  public:
+    enum class MigrateType {
+      Class,
hokein wrote:
> What is the 'MigrateType' used for? I can't find any usage in the patch.
Deleted it for now. Will add it when it is actually used in the future.

cfe-commits mailing list

Reply via email to