View Change
1 comment:
File tests/conv/conv.c:
Patch Set #4, Line 42: return -1;
Would it be possible to separate test code in Coverity from 'regular' code? I'd say that would make […]
Sorry for very late feedback. It's possible to exclude specific files or even directories from the analysis (in the project settings). But it's not like we're getting too much of those issues. For now I am fine with clicking "ignore" on such CIDs. Also, we may not see real bugs in tests if we exclude them.
To view, visit change 1627. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Idbdc7e19cb9b9a36cd1fccd621cd858e87530d98
Gerrit-Change-Number: 1627
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: tnt <tnt@246tNt.com>
Gerrit-CC: arehbein <arehbein@sysmocom.de>
Gerrit-Comment-Date: Sun, 10 Dec 2023 17:03:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: arehbein <arehbein@sysmocom.de>
Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de>
Gerrit-MessageType: comment