From: Vedant Kumar <vsk@apple.com>
Date: Fri, 15 Jul 2016 22:44:54 +0000 (+0000)
Subject: [llvm-cov] Document a few private fields of CodeCoverageTool (NFC)
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a5ec41d9a9a577b91f3390a46590a5827a84d6dd;p=llvm

[llvm-cov] Document a few private fields of CodeCoverageTool (NFC)

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@275639 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/tools/llvm-cov/CodeCoverage.cpp b/tools/llvm-cov/CodeCoverage.cpp
index ed48c28c0fc..59cb708a42c 100644
--- a/tools/llvm-cov/CodeCoverage.cpp
+++ b/tools/llvm-cov/CodeCoverage.cpp
@@ -101,11 +101,14 @@ public:
   std::string CoverageArch;
 
 private:
+  /// File paths (absolute, or otherwise) to input source files.
   std::vector<std::string> CollectedPaths;
 
+  /// Errors and warnings which have not been printed.
   std::mutex DeferredMessagesLock;
   std::vector<std::string> DeferredMessages;
 
+  /// A container for input source file buffers.
   std::mutex LoadedSourceFilesLock;
   std::vector<std::pair<std::string, std::unique_ptr<MemoryBuffer>>>
       LoadedSourceFiles;
diff --git a/tools/llvm-cov/CoverageViewOptions.h b/tools/llvm-cov/CoverageViewOptions.h
index 873eae885a7..eafc0cfd165 100644
--- a/tools/llvm-cov/CoverageViewOptions.h
+++ b/tools/llvm-cov/CoverageViewOptions.h
@@ -40,7 +40,7 @@ struct CoverageViewOptions {
   }
 
   /// \brief Check if an output directory has been specified.
-  bool hasOutputDirectory() const { return ShowOutputDirectory != ""; }
+  bool hasOutputDirectory() const { return !ShowOutputDirectory.empty(); }
 };
 }