bkramer added a comment. Can you add a lit test for this? We should've added that earlier :|
================ Comment at: include-fixer/IncludeFixer.h:53 @@ -51,2 +52,3 @@ /// The context that contains all information about the symbol being queried. + std::vector<IncludeFixerContext> &Contexts; ---------------- This comment is now outdated, there are multiple contexts. ================ Comment at: include-fixer/IncludeFixerContext.h:21 @@ -20,2 +20,3 @@ -/// \brief A context for the symbol being queried. +/// \brief A context for the file being processed. It includes all query +/// information, e.g. symbols being queried in database, all header candiates. ---------------- A context for **a** file being processed. ================ Comment at: include-fixer/IncludeFixerContext.h:78 @@ -72,1 +77,3 @@ + /// \brief The absolute path to the file being processed. + std::string FilePath; ---------------- Does it have to be absolute? https://reviews.llvm.org/D23266 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
