[llvm-cov] Make an API more consistent, NFC

Make renderExpansionView() look a bit more like renderLine(), and
clarify its doxygen comment.

llvm-svn: 273773

1 files changed, 8 insertions, 6 deletions

diff --git a/llvm/tools/llvm-cov/SourceCoverageView.h b/llvm/tools/llvm-cov/SourceCoverageView.h
index 82822ca9ee3..763529a476b 100644
--- a/llvm/tools/llvm-cov/SourceCoverageView.h
+++ b/llvm/tools/llvm-cov/SourceCoverageView.h
@@ -163,12 +163,14 @@ protected:
                                    CoverageSegmentArray Segments,
                                    unsigned ViewDepth) = 0;
 
-  /// \brief Render an expansion view. If \p FirstLine is provided, it points
-  /// to the expansion site, which must be re-rendered for clarity.
-  virtual unsigned renderExpansionView(
-      raw_ostream &OS, ExpansionView &ESV, Optional<LineRef> FirstLine,
-      unsigned ExpansionCol, const coverage::CoverageSegment *WrappedSegment,
-      CoverageSegmentArray LineSegments, unsigned ViewDepth) = 0;
+  /// \brief Render an expansion view. If the expansion site must be re-rendered
+  /// for clarity, it is passed in via \p FirstLine.
+  virtual unsigned
+  renderExpansionView(raw_ostream &OS, ExpansionView &ESV,
+                      Optional<LineRef> FirstLine,
+                      const coverage::CoverageSegment *WrappedSegment,
+                      CoverageSegmentArray Segments, unsigned ExpansionCol,
+                      unsigned ViewDepth) = 0;
 
   /// \brief Render an instantiation view.
   virtual void renderInstantiationView(raw_ostream &OS, InstantiationView &ISV,