From: Vedant Kumar Date: Thu, 15 Sep 2016 04:45:59 +0000 (+0000) Subject: [llvm-cov] Make a method name more accurate (NFC) X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7551d48f2f7b5650590262f86461a2b5bca3943d;p=llvm [llvm-cov] Make a method name more accurate (NFC) git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@281581 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/tools/llvm-cov/SourceCoverageView.cpp b/tools/llvm-cov/SourceCoverageView.cpp index ad6aa666839..8cf17f4a0a9 100644 --- a/tools/llvm-cov/SourceCoverageView.cpp +++ b/tools/llvm-cov/SourceCoverageView.cpp @@ -169,7 +169,7 @@ void SourceCoverageView::addInstantiation( void SourceCoverageView::print(raw_ostream &OS, bool WholeFile, bool ShowSourceName, unsigned ViewDepth) { if (WholeFile && getOptions().hasOutputDirectory()) - renderCellInTitle(OS, "Coverage Report"); + renderTitle(OS, "Coverage Report"); renderViewHeader(OS); diff --git a/tools/llvm-cov/SourceCoverageView.h b/tools/llvm-cov/SourceCoverageView.h index 68e8251e9b8..129c5902070 100644 --- a/tools/llvm-cov/SourceCoverageView.h +++ b/tools/llvm-cov/SourceCoverageView.h @@ -240,9 +240,9 @@ protected: virtual void renderInstantiationView(raw_ostream &OS, InstantiationView &ISV, unsigned ViewDepth) = 0; - /// \brief Render the project title, the report title \p CellText and the - /// created time for the view. - virtual void renderCellInTitle(raw_ostream &OS, StringRef CellText) = 0; + /// \brief Render \p Title, a project title if one is available, and the + /// created time. + virtual void renderTitle(raw_ostream &OS, StringRef CellText) = 0; /// \brief Render the table header for a given source file. virtual void renderTableHeader(raw_ostream &OS, unsigned FirstUncoveredLineNo, diff --git a/tools/llvm-cov/SourceCoverageViewHTML.cpp b/tools/llvm-cov/SourceCoverageViewHTML.cpp index 375b0dee06d..2fef1b4175d 100644 --- a/tools/llvm-cov/SourceCoverageViewHTML.cpp +++ b/tools/llvm-cov/SourceCoverageViewHTML.cpp @@ -588,11 +588,10 @@ void SourceCoverageViewHTML::renderInstantiationView(raw_ostream &OS, OS << EndExpansionDiv; } -void SourceCoverageViewHTML::renderCellInTitle(raw_ostream &OS, - StringRef CellText) { +void SourceCoverageViewHTML::renderTitle(raw_ostream &OS, StringRef Title) { if (getOptions().hasProjectTitle()) OS << tag(ProjectTitleTag, escape(getOptions().ProjectTitle, getOptions())); - OS << tag(ReportTitleTag, escape(CellText, getOptions())); + OS << tag(ReportTitleTag, escape(Title, getOptions())); if (getOptions().hasCreatedTime()) OS << tag(CreatedTimeTag, escape(getOptions().CreatedTimeStr, getOptions())); diff --git a/tools/llvm-cov/SourceCoverageViewHTML.h b/tools/llvm-cov/SourceCoverageViewHTML.h index b97c1802af8..ad4b2b8ab06 100644 --- a/tools/llvm-cov/SourceCoverageViewHTML.h +++ b/tools/llvm-cov/SourceCoverageViewHTML.h @@ -78,7 +78,7 @@ class SourceCoverageViewHTML : public SourceCoverageView { void renderRegionMarkers(raw_ostream &OS, CoverageSegmentArray Segments, unsigned ViewDepth) override; - void renderCellInTitle(raw_ostream &OS, StringRef CellText) override; + void renderTitle(raw_ostream &OS, StringRef Title) override; void renderTableHeader(raw_ostream &OS, unsigned FirstUncoveredLineNo, unsigned IndentLevel) override; diff --git a/tools/llvm-cov/SourceCoverageViewText.cpp b/tools/llvm-cov/SourceCoverageViewText.cpp index 16bbe5648b8..51c6463db30 100644 --- a/tools/llvm-cov/SourceCoverageViewText.cpp +++ b/tools/llvm-cov/SourceCoverageViewText.cpp @@ -218,13 +218,12 @@ void SourceCoverageViewText::renderInstantiationView(raw_ostream &OS, ISV.View->print(OS, /*WholeFile=*/false, /*ShowSourceName=*/true, ViewDepth); } -void SourceCoverageViewText::renderCellInTitle(raw_ostream &OS, - StringRef CellText) { +void SourceCoverageViewText::renderTitle(raw_ostream &OS, StringRef Title) { if (getOptions().hasProjectTitle()) getOptions().colored_ostream(OS, raw_ostream::CYAN) << getOptions().ProjectTitle << "\n"; - getOptions().colored_ostream(OS, raw_ostream::CYAN) << CellText << "\n"; + getOptions().colored_ostream(OS, raw_ostream::CYAN) << Title << "\n"; if (getOptions().hasCreatedTime()) getOptions().colored_ostream(OS, raw_ostream::CYAN) diff --git a/tools/llvm-cov/SourceCoverageViewText.h b/tools/llvm-cov/SourceCoverageViewText.h index 7fdeef50ea2..3968aa70dd5 100644 --- a/tools/llvm-cov/SourceCoverageViewText.h +++ b/tools/llvm-cov/SourceCoverageViewText.h @@ -71,7 +71,7 @@ class SourceCoverageViewText : public SourceCoverageView { void renderRegionMarkers(raw_ostream &OS, CoverageSegmentArray Segments, unsigned ViewDepth) override; - void renderCellInTitle(raw_ostream &OS, StringRef CellText) override; + void renderTitle(raw_ostream &OS, StringRef Title) override; void renderTableHeader(raw_ostream &OS, unsigned FirstUncoveredLineNo, unsigned IndentLevel) override;