pespin submitted this change.

View Change

Approvals: osmith: Looks good to me, but someone else must approve fixeria: Looks good to me, approved Jenkins Builder: Verified
tests/osmo_io: Remove unused variable

gcc 16.1.1 warns about it.

Change-Id: I0bdfe5ed6803042277ed8d21171b5f9c11495ca3
---
M tests/osmo_io/osmo_io_backpressure_test.c
1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/tests/osmo_io/osmo_io_backpressure_test.c b/tests/osmo_io/osmo_io_backpressure_test.c
index 69bbb35..87259dc 100644
--- a/tests/osmo_io/osmo_io_backpressure_test.c
+++ b/tests/osmo_io/osmo_io_backpressure_test.c
@@ -106,7 +106,6 @@
file_bytes_write_compl = 0;

char drain_buffer[512];
- int messages_queued = 0;

for (int i = 0; i < 10; i++) {
/* Add a message */
@@ -114,9 +113,7 @@
memset(msgb_put(msg, 1024), 0xAA + (i % 16), 1024);

rc = osmo_iofd_write_msgb(iofd, msg);
- if (rc == 0)
- messages_queued++;
- else
+ if (rc != 0)
msgb_free(msg);

/* Process events with explicit timeout */

To view, visit change 43097. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I0bdfe5ed6803042277ed8d21171b5f9c11495ca3
Gerrit-Change-Number: 43097
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>