Skip to content

Commit

Permalink
[analyzer] [NFC] Use StringRef when returning a large string literal …
Browse files Browse the repository at this point in the history
…in HTMLDiagnostics

(NB: could be a clang-tidy / analyzer check)

Differential Revision: https://reviews.llvm.org/D51512

llvm-svn: 341723
  • Loading branch information
George Karpenkov committed Sep 7, 2018
1 parent 5577cb7 commit e4707e7
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions clang/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp
Expand Up @@ -115,12 +115,13 @@ class HTMLDiagnostics : public PathDiagnosticConsumer {
void RewriteFile(Rewriter &R, const SourceManager& SMgr,
const PathPieces& path, FileID FID);

/// \return Javascript for navigating the HTML report using j/k keys.
std::string generateKeyboardNavigationJavascript();

private:
/// \return Javascript for displaying shortcuts help;
std::string showHelpJavascript();
StringRef showHelpJavascript();

/// \return Javascript for navigating the HTML report using j/k keys.
StringRef generateKeyboardNavigationJavascript();
};

} // namespace
Expand Down Expand Up @@ -571,7 +572,7 @@ void HTMLDiagnostics::FinalizeHTML(const PathDiagnostic& D, Rewriter &R,
html::AddHeaderFooterInternalBuiltinCSS(R, FID, Entry->getName());
}

std::string HTMLDiagnostics::showHelpJavascript() {
StringRef HTMLDiagnostics::showHelpJavascript() {
return R"<<<(
<script type='text/javascript'>
Expand Down Expand Up @@ -929,7 +930,7 @@ void HTMLDiagnostics::HighlightRange(Rewriter& R, FileID BugFileID,
html::HighlightRange(R, InstantiationStart, E, HighlightStart, HighlightEnd);
}

std::string HTMLDiagnostics::generateKeyboardNavigationJavascript() {
StringRef HTMLDiagnostics::generateKeyboardNavigationJavascript() {
return R"<<<(
<script type='text/javascript'>
var digitMatcher = new RegExp("[0-9]+");
Expand Down

0 comments on commit e4707e7

Please sign in to comment.