[llvm] r273773 - [llvm-cov] Make an API more consistent, NFC
Vedant Kumar via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 24 22:48:59 PDT 2016
Author: vedantk
Date: Sat Jun 25 00:48:59 2016
New Revision: 273773
URL: http://llvm.org/viewvc/llvm-project?rev=273773&view=rev
Log:
[llvm-cov] Make an API more consistent, NFC
Make renderExpansionView() look a bit more like renderLine(), and
clarify its doxygen comment.
Modified:
llvm/trunk/tools/llvm-cov/SourceCoverageView.cpp
llvm/trunk/tools/llvm-cov/SourceCoverageView.h
llvm/trunk/tools/llvm-cov/SourceCoverageViewText.cpp
llvm/trunk/tools/llvm-cov/SourceCoverageViewText.h
Modified: llvm/trunk/tools/llvm-cov/SourceCoverageView.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-cov/SourceCoverageView.cpp?rev=273773&r1=273772&r2=273773&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-cov/SourceCoverageView.cpp (original)
+++ llvm/trunk/tools/llvm-cov/SourceCoverageView.cpp Sat Jun 25 00:48:59 2016
@@ -133,7 +133,7 @@ void SourceCoverageView::print(raw_ostre
OS, *NextESV,
RenderedSubView ? Optional<LineRef>({*LI, LI.line_number()})
: Optional<LineRef>(),
- ExpansionColumn, WrappedSegment, LineSegments, ViewDepth);
+ WrappedSegment, LineSegments, ExpansionColumn, ViewDepth);
RenderedSubView = true;
}
for (; NextISV != EndISV && NextISV->Line == LI.line_number(); ++NextISV) {
Modified: llvm/trunk/tools/llvm-cov/SourceCoverageView.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-cov/SourceCoverageView.h?rev=273773&r1=273772&r2=273773&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-cov/SourceCoverageView.h (original)
+++ llvm/trunk/tools/llvm-cov/SourceCoverageView.h Sat Jun 25 00:48:59 2016
@@ -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,
Modified: llvm/trunk/tools/llvm-cov/SourceCoverageViewText.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-cov/SourceCoverageViewText.cpp?rev=273773&r1=273772&r2=273773&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-cov/SourceCoverageViewText.cpp (original)
+++ llvm/trunk/tools/llvm-cov/SourceCoverageViewText.cpp Sat Jun 25 00:48:59 2016
@@ -158,8 +158,8 @@ void SourceCoverageViewText::renderRegio
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 @@ unsigned SourceCoverageViewText::renderE
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);
}
Modified: llvm/trunk/tools/llvm-cov/SourceCoverageViewText.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-cov/SourceCoverageViewText.h?rev=273773&r1=273772&r2=273773&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-cov/SourceCoverageViewText.h (original)
+++ llvm/trunk/tools/llvm-cov/SourceCoverageViewText.h Sat Jun 25 00:48:59 2016
@@ -33,9 +33,9 @@ class SourceCoverageViewText : public So
unsigned renderExpansionView(raw_ostream &OS, ExpansionView &ESV,
Optional<LineRef> FirstLine,
- unsigned ExpansionCol,
const coverage::CoverageSegment *WrappedSegment,
- CoverageSegmentArray LineSegments,
+ CoverageSegmentArray Segments,
+ unsigned ExpansionCol,
unsigned ViewDepth) override;
void renderInstantiationView(raw_ostream &OS, InstantiationView &ISV,
More information about the llvm-commits
mailing list