Skip to content
This repository has been archived by the owner on Sep 28, 2023. It is now read-only.

Adds to the supression list maliput_drake's header files. #13

Merged
merged 2 commits into from
Nov 2, 2021

Conversation

agalbachicar
Copy link
Collaborator

Dependent packages should not fail because of this package
(which is a dependency). The chain of supressions is collected
by the run_scan_build.py script in maliput_infrastructure.

Fixes maliput/maliput_infrastructure#236

Tried with:

CC=clang-8 CCX=clang++-8 LDFLAGS=-fuse-ld=lld-8 ./run_scan_build.py --event-handlers=console_direct+
...
Summary: 16 packages finished [21.9s]
scan-build: Removing directory '/tmp/scan-build-2021-11-02-113047-25110-1' because it contains no reports.
scan-build: No bugs found.

Dependent packages should not fail because of this package
(which is a dependency). The chain of supressions is collected
by the run_scan_build.py script in maliput_infrastructure.
francocipollone
francocipollone previously approved these changes Nov 2, 2021
Copy link
Collaborator

@francocipollone francocipollone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

tools/scan_build.supp Outdated Show resolved Hide resolved
tools/scan_build.supp Outdated Show resolved Hide resolved
@agalbachicar agalbachicar merged commit 39273ee into main Nov 2, 2021
@agalbachicar agalbachicar deleted the agalbachicar/#236_fix_scan_build branch November 2, 2021 12:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

scan-build errors in weekly check
2 participants