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)

          [JENKINS-67101] C++ Lint shows as C++ Lint

          Ulli Hafner added a comment - - edited

          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?

          Ulli Hafner added a comment - - edited 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?

          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)...

          Ulli Hafner added a comment -

          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.

          Ulli Hafner added a comment - 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.

            Unassigned Unassigned
            eirik Eirik Marthinsen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: