commit 8b12ecb721b5f472b5b0fb592de3751d1e544f0e
author Vedant Kumar <vsk@apple.com> Sat Jun 25 05:48:59 2016 +0000
committer Vedant Kumar <vsk@apple.com> Sat Jun 25 05:48:59 2016 +0000
tree 9d57274e577a6a91a0410726e57cf47b31d4029a
parent ee5a5e9bceb5132714c1a06fd46ea0b3db3129bc

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

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

llvm-svn: 273773

llvm/tools/llvm-cov/SourceCoverageViewText.cpp

diff --git a/llvm/tools/llvm-cov/SourceCoverageViewText.cpp b/llvm/tools/llvm-cov/SourceCoverageViewText.cpp
index c1f281c..e02f4b1 100644
--- a/llvm/tools/llvm-cov/SourceCoverageViewText.cpp
+++ b/llvm/tools/llvm-cov/SourceCoverageViewText.cpp
@@ -158,8 +158,8 @@
 
 unsigned SourceCoverageViewText::renderExpansionView(
     raw_ostream &OS, ExpansionView &ESV, Optional<LineRef> FirstLine,
-    unsigned ExpansionCol, const coverage::CoverageSegment *WrappedSegment,
-    CoverageSegmentArray LineSegments, unsigned ViewDepth) {
+    const coverage::CoverageSegment *WrappedSegment,
+    CoverageSegmentArray Segments, unsigned ExpansionCol, unsigned ViewDepth) {
   unsigned NextExpansionCol = ExpansionCol;
 
   if (FirstLine.hasValue()) {
@@ -168,7 +168,7 @@
     NextExpansionCol = ESV.getStartCol();
     renderLinePrefix(OS, ViewDepth);
     OS.indent(getCombinedColumnWidth(getOptions()) + (ViewDepth == 0 ? 0 : 1));
-    renderLine(OS, *FirstLine, WrappedSegment, LineSegments, ExpansionCol,
+    renderLine(OS, *FirstLine, WrappedSegment, Segments, ExpansionCol,
                ViewDepth);
     renderViewDivider(OS, ViewDepth + 1);
   }