JDevlieghere marked an inline comment as done.
JDevlieghere added inline comments.


================
Comment at: include/lldb/Utility/Reproducer.h:91-92
+
+const char *FileInfo::name = "files";
+const char *FileInfo::file = "files.yaml";
+
----------------
labath wrote:
> Are you sure this can be in a header? I would expect this to give you 
> multiply-defined symbols errors.
You are correct, should be in the implementation. 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54617/new/

https://reviews.llvm.org/D54617



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

Reply via email to