fixeria has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmocore/+/1627?usp=email )
Change subject: tests/conv: separate test logic
......................................................................
Patch Set 4:
(1 comment)
File tests/conv/conv.c:
https://gerrit.osmocom.org/c/libosmocore/+/1627/comment/3b257a42_c8f2d659
PS4, 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
https://gerrit.osmocom.org/c/libosmocore/+/1627?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Idbdc7e19cb9b9a36cd1fccd621cd858e87530d98
Gerrit-Change-Number: 1627
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: tnt <tnt(a)246tNt.com>
Gerrit-CC: arehbein <arehbein(a)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(a)sysmocom.de>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment