Skip to content

Commit

Permalink
Created new coverity/codecov/lint environment for sexpp project, fixe…
Browse files Browse the repository at this point in the history
…d coverity CID 453360
  • Loading branch information
maxirmx committed Jun 24, 2023
1 parent 37d8e66 commit e5f242f
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 16 deletions.
6 changes: 1 addition & 5 deletions .github/workflows/coverity.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,8 @@ jobs:
env:
CC: gcc
CXX: g++
COVERITY_TOKEN: qjcM1CWLcq9PJB3yL0ZXIw
MAKEFLAGS: j4
TOKEN: qjcM1CWLcq9PJB3yL0ZXIw
steps:
- name: Checkout
uses: actions/checkout@v3
Expand All @@ -56,8 +56,6 @@ jobs:
run: cmake -Bbuild -DDOWNLOAD_GTEST=OFF

- name: Download Coverity
env:
TOKEN: AK2euT-neBcvZB1g_m3pbg
run: |
wget -q https://scan.coverity.com/download/cxx/linux64 --post-data "token=$TOKEN&project=$GITHUB_REPOSITORY" -O cov-analysis-linux64.tar.gz
mkdir cov-analysis-linux64
Expand All @@ -70,8 +68,6 @@ jobs:
cov-build --dir cov-int cmake --build build
- name: Submit
env:
TOKEN: AK2euT-neBcvZB1g_m3pbg
run: |
tar czvf results.tgz cov-int
curl \
Expand Down
16 changes: 8 additions & 8 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,14 +41,14 @@ jobs:
source: '.'
extensions: 'h,cpp,c'
clangFormatVersion: 11
inplace: True
- uses: EndBug/add-and-commit@v9
with:
author_name: A robot on behalf of Maxim Samsonov
author_email: m.samsonov@computer.org
message: 'Committing clang-format changes'
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# inplace: True
# - uses: EndBug/add-and-commit@v9
# with:
# author_name: A robot on behalf of Maxim Samsonov
# author_email: m.samsonov@computer.org
# message: 'Committing clang-format changes'
# env:
# GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
shellcheck:
runs-on: ubuntu-latest
steps:
Expand Down
4 changes: 2 additions & 2 deletions README.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ image:https://github.com/rnpgp/sexp/workflows/build-and-test-deb/badge.svg["Buil
image:https://github.com/rnpgp/sexp/workflows/build-and-test-msys/badge.svg["Build status MSys", link="https://github.com/rnpgp/sexp/actions?workflow=build-and-test-msys"]


image:https://codecov.io/gh/rnpgp/sexp/branch/feat/g23/graph/badge.svg["Code coverage", link="https://codecov.io/gh/rnpgp/sexpp"]
image:https://github.com/rnpgp/sexp/workflows/CodeQL/badge.svg["CodeQL analysis", link="https://github.com/rnpgp/sexp/actions?workflow=CodeQL"]
image:https://codecov.io/gh/rnpgp/sexpp/branch/main/graph/badge.svg["Code coverage", link="https://codecov.io/gh/rnpgp/sexpp"]
image:https://github.com/rnpgp/sexpp/workflows/CodeQL/badge.svg["CodeQL analysis", link="https://github.com/rnpgp/sexpp/actions?workflow=CodeQL"]
image:https://scan.coverity.com/projects/28717/badge.svg["Coverity Scan Build Status", link="https://scan.coverity.com/projects/rnpgp-sexpp"]


Expand Down
2 changes: 1 addition & 1 deletion include/sexpp/sexp-error.h
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ class sexp_exception_t : public std::exception {
int error_position,
const char *prefix = "SEXP")
: position{error_position}, level{error_level},
message{format(prefix, error_message, error_level, error_position)} {};
message{format(prefix, std::move(error_message), error_level, error_position)} {};

static std::string format(std::string prf,
std::string message,
Expand Down

0 comments on commit e5f242f

Please sign in to comment.