Differential D7140

Ignore cppcheck syntax errors related to prevector
ClosedPublic
Actions

Authored by jasonbcox on Aug 6 2020, 16:48.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Commits

rABC8aadcedb631f: Ignore cppcheck syntax errors related to prevector

Summary

These false positives can be safely ignored since the build system does not complain about
the supposed syntax errors.

A newer version of cppcheck is not available via buster-backports, so it's best that we ignore this
error until a version becomes available that fixes this issue.

Test Plan

arc lint src/bench/prevector.cpp
arc lint src/test/prevector_tests.cpp

Diff Detail

Repository

rABC Bitcoin ABC

Branch

cppcheck-prevector

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 12320
Build 24836: Build Diff	build-diff · build-without-wallet · build-clang-tidy · build-clang-10
Build 24835: arc lint + arc unit

Event Timeline

jasonbcox created this revision.Aug 6 2020, 16:48

Herald added a reviewer: Restricted Project. · View Herald TranscriptAug 6 2020, 16:48

jasonbcox requested review of this revision.Aug 6 2020, 16:48

Harbormaster completed remote builds in B12320: Diff 22766.Aug 6 2020, 17:09

majcosta accepted this revision.Aug 6 2020, 17:25

This revision is now accepted and ready to land.Aug 6 2020, 17:25

Closed by commit rABC8aadcedb631f: Ignore cppcheck syntax errors related to prevector (authored by jasonbcox). · Explain WhyAug 6 2020, 18:06

This revision was automatically updated to reflect the committed changes.

jasonbcox added a commit: rABC8aadcedb631f: Ignore cppcheck syntax errors related to prevector.

Revision Contents
Changeset List

Path

Size

arcanist/

linter/

CppCheckLinter.php

8 lines

Diff 22766

View Options

arcanist/linter/CppCheckLinter.php

Ignore cppcheck syntax errors related to prevectorClosedPublicActions