-
Bug
-
Resolution: Unresolved
-
Trivial
The C++ linter Cpplint is printed like this in the Warnings Next Generation Plugin:
Looking at the source I see that the ampersand in the html special character + ( + ) has been escaped (C++ Lint; ).
Is there any reason for escaping the + in the first place? It's just a simple ascii character...
PS: Is "C++ Lint" a proper name for this linter? In the documentation it is only mentioned by the name cpplint (https://pypi.org/project/cpplint/ and https://github.com/google/styleguide/tree/gh-pages/cpplint)
- is blocking
-
-
- Closed
-
[JENKINS-67101] C++ Lint shows as C++ Lint
Eirik Marthinsen
added a comment - I have made two pull requests that renames "C++ Lint" to Cpplint:
Analysis-model: https://github.com/jenkinsci/analysis-model/pull/707
Warnings-ng-plugin: https://github.com/jenkinsci/warnings-ng-plugin/pull/1111
I still however see the issue of + being escaped twice for the other C++ parser names (iar, sunc and diabc)...
Eirik Marthinsen
added a comment - I have made two pull requests that renames "C++ Lint" to Cpplint:
Analysis-model: https://github.com/jenkinsci/analysis-model/pull/707
Warnings-ng-plugin: https://github.com/jenkinsci/warnings-ng-plugin/pull/1111
I still however see the issue of + being escaped twice for the other C++ parser names (iar, sunc and diabc)... Thanks for fixing the name CppLint tool.
I think it makes sense to let this issue open right now. Maybe I can find a simple way to suppress the escaping for all names that are derived from the deployed archive. Only names that are changed by the user need to be escaped.
Hello drulli for the refactoring. What are the files that I can change, for example - in the PR #707 it looks like the .java and SUPPORTED-FORMATS.md are modified.
Currently the .md file has mentions in IAR Compiler, SUN C++ Compiler and Wind River Diab Compiler.
And, changing the mentions in the .txt, .log and .xml files might also lead to the changes in the testing files right?
So, apart from .java files and Supported-Format.md are there any other type of files that I can change? Or do you have a better approach to tackle this?
Arnab
added a comment - - edited Hello drulli for the refactoring. What are the files that I can change, for example - in the PR #707 it looks like the .java and SUPPORTED-FORMATS.md are modified.
Currently the .md file has mentions in IAR Compiler, SUN C++ Compiler and Wind River Diab Compiler.
And, changing the mentions in the .txt, .log and .xml files might also lead to the changes in the testing files right?
So, apart from .java files and Supported-Format.md are there any other type of files that I can change? Or do you have a better approach to tackle this? 
All names need to be escaped since this is a value that potentially can be changed by the user. But using "CppLint" as name would work for me. Interested in providing a PR In the analysis-model project?