Page MenuHomePhabricator
Differential D6866

Report most cppcheck errors
ClosedPublic
Actions

Authored by deadalnix on Jul 9 2020, 01:02.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCf76270a18711: Report most cppcheck errors
Summary

If we are going to run it, we might as well get the most bang for the buck.

Test Plan
arc lint --everything

cppcheck raises a handful of error still. They look legitimate.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

deadalnix created this revision.Jul 9 2020, 01:02
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 9 2020, 01:02
deadalnix requested review of this revision.Jul 9 2020, 01:02
Harbormaster completed remote builds in B11917: Diff 22101.Jul 9 2020, 01:18
Fabien accepted this revision.Jul 9 2020, 06:11
This revision is now accepted and ready to land.Jul 9 2020, 06:11
Closed by commit rABCf76270a18711: Report most cppcheck errors (authored by deadalnix). · Explain WhyJul 9 2020, 09:10
This revision was automatically updated to reflect the committed changes.
deadalnix added a commit: rABCf76270a18711: Report most cppcheck errors.

Revision Contents
Changeset List

PathSize
arcanist/
linter/
50 lines

Diff 22109

View Options

arcanist/linter/CppCheckLinter.php

Loading...