Changeset View
Changeset View
Standalone View
Standalone View
addons/project/tools/kateprojectcodeanalysistoolcppcheck.cpp
Show All 30 Lines | |||||
31 | 31 | | |||
32 | KateProjectCodeAnalysisToolCppcheck::~KateProjectCodeAnalysisToolCppcheck() | 32 | KateProjectCodeAnalysisToolCppcheck::~KateProjectCodeAnalysisToolCppcheck() | ||
33 | { | 33 | { | ||
34 | 34 | | |||
35 | } | 35 | } | ||
36 | 36 | | |||
37 | QString KateProjectCodeAnalysisToolCppcheck::name() | 37 | QString KateProjectCodeAnalysisToolCppcheck::name() | ||
38 | { | 38 | { | ||
39 | return i18n("cppcheck"); | 39 | return i18n("Cppcheck (C++)"); | ||
40 | } | ||||
41 | | ||||
42 | QString KateProjectCodeAnalysisToolCppcheck::description() const | ||||
43 | { | ||||
44 | return i18n("Cppcheck is a static analysis tool for C/C++ code"); | ||||
ngraham: Does this need to be in quotes? | |||||
I was not sure, because I grabbed the exact text from the projects web site. gregormi: I was not sure, because I grabbed the exact text from the projects web site. | |||||
ngraham: I think we can omit the quotation marks. | |||||
40 | } | 45 | } | ||
41 | 46 | | |||
42 | QString KateProjectCodeAnalysisToolCppcheck::fileExtensions() | 47 | QString KateProjectCodeAnalysisToolCppcheck::fileExtensions() | ||
43 | { | 48 | { | ||
44 | return QStringLiteral("cpp|cxx|cc|c++|c|tpp|txx"); | 49 | return QStringLiteral("cpp|cxx|cc|c++|c|tpp|txx"); | ||
45 | } | 50 | } | ||
46 | 51 | | |||
47 | QStringList KateProjectCodeAnalysisToolCppcheck::filter(const QStringList &files) | 52 | QStringList KateProjectCodeAnalysisToolCppcheck::filter(const QStringList &files) | ||
Show All 35 Lines | |||||
83 | QString KateProjectCodeAnalysisToolCppcheck::stdinMessages() | 88 | QString KateProjectCodeAnalysisToolCppcheck::stdinMessages() | ||
84 | { | 89 | { | ||
85 | // filenames are written to stdin (--file-list=-) | 90 | // filenames are written to stdin (--file-list=-) | ||
86 | 91 | | |||
87 | if (!m_project) { | 92 | if (!m_project) { | ||
88 | return QString(); | 93 | return QString(); | ||
89 | } | 94 | } | ||
90 | 95 | | |||
91 | return filter(m_project->files()).join(QStringLiteral("\n")); | 96 | auto&& fileList = filter(m_project->files()); | ||
97 | setActualFilesCount(fileList.size()); | ||||
98 | return fileList.join(QStringLiteral("\n")); | ||||
92 | } | 99 | } |
Does this need to be in quotes?