Attention is currently required from: laforge, dexter.

Jenkins Builder has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmo-abis/+/32401 )

Change subject: e1d: initialize file descriptor numbers to -1 on startup
......................................................................


Patch Set 3:

(2 comments)

File src/input/e1d.c:

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-6471):
https://gerrit.osmocom.org/c/libosmo-abis/+/32401/comment/3d35ed49_ae045291
PS3, Line 526:  for (ts=1; ts<line->num_ts; ts++) {
spaces required around that '=' (ctx:VxV)


Robot Comment from checkpatch (run ID jenkins-gerrit-lint-6471):
https://gerrit.osmocom.org/c/libosmo-abis/+/32401/comment/a80fa09c_09bd10de
PS3, Line 526:  for (ts=1; ts<line->num_ts; ts++) {
spaces required around that '<' (ctx:VxV)



--
To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/32401
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I2de8fccad56279748ed9cc035aebf4e2d3935172
Gerrit-Change-Number: 32401
Gerrit-PatchSet: 3
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Wed, 26 Apr 2023 08:09:24 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to