klimek added inline comments.

================
Comment at: include-fixer/find-all-symbols/SymbolInfo.h:90-93
@@ +89,6 @@
+
+  static SymbolInfo
+  CreateFunctionSymbolInfo(const std::string &Name, const std::string 
&FilePath,
+                           const std::vector<Context> &Contexts, int 
LineNumber,
+                           const FunctionInfo &FuncInfo);
+
----------------
ioeric wrote:
> klimek wrote:
> > To some degree this looks like we actually want a class hierarchy. But 
> > currently only ClassSymbolInfo is used?
> At this point yes, since Symbolnfo is only created in InMemoryXrefsDB and 
> FindAllSymbols now, and it is not trivial to change FindAllSymbols to use 
> these creators. 
> 
> But one immediate use case I can foresee is converting proto buffer 
> SymbolInfo to clang SymbolInfo when integrating with Google3 XrefsDB. 
I think at this point YAGNI applies and we shouldn't create stuff that we don't 
use yet - code generally starts to rot when it's not used ;)


http://reviews.llvm.org/D19913



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

Reply via email to