On Tue, Nov 4, 2014 at 7:25 AM, Alexander Kornienko <[email protected]> wrote:
> Author: alexfh > Date: Tue Nov 4 09:25:22 2014 > New Revision: 221272 > > URL: http://llvm.org/viewvc/llvm-project?rev=221272&view=rev > Log: > [clang-tidy] Don't print a message if there's no error. > Test coverage? > > Modified: > clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp > > Modified: clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp > URL: > http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp?rev=221272&r1=221271&r2=221272&view=diff > > ============================================================================== > --- clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp (original) > +++ clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp Tue Nov 4 > 09:25:22 2014 > @@ -235,8 +235,9 @@ FileOptionsProvider::TryReadConfigFile(S > llvm::ErrorOr<ClangTidyOptions> ParsedOptions = > ConfigHandler.second((*Text)->getBuffer()); > if (!ParsedOptions) { > - llvm::errs() << "Error parsing " << ConfigFile << ": " > - << ParsedOptions.getError().message() << "\n"; > + if (ParsedOptions.getError()) > + llvm::errs() << "Error parsing " << ConfigFile << ": " > + << ParsedOptions.getError().message() << "\n"; > continue; > } > > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
