10 Jan
2022
10 Jan
'22
3:13 p.m.
laforge has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/26841 )
Change subject: icE1usb fw: Add multi-port support in USB code
......................................................................
Patch Set 1: Verified+1 Code-Review+1
see https://gerrit.osmocom.org/c/osmo-ci/+/26858 and https://osmocom.org/issues/5399
--
To view, visit https://gerrit.osmocom.org/c/osmo-e1-hardware/+/26841
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-e1-hardware
Gerrit-Branch: master
Gerrit-Change-Id: I134410b542461c34339baa8d9120a5c86fc35d46
Gerrit-Change-Number: 26841
Gerrit-PatchSet: 1
Gerrit-Owner: tnt <tnt(a)246tNt.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Mon, 10 Jan 2022 15:13:58 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
so will you
keep your slightly different-than-osmocom codign style for this project? If so, we should
probably remove the linter for this project.
Yes. Not sure if it's possible to disable only some of the options.
Mostly it's the brace position which I put for whatever looks better (which depends
on the surounding code, I don't like when code isn't "aerated"), and the
'=' in for loops.