Attention is currently required from: fixeria.
View Change
1 comment:
File src/common/osmux.c:
Patch Set #1, Line 96: strerror_r
Why not just using strerror()? osmo-bts is single-thread, and (I hope) will unlikely be multi-thread […]
There's nothing wrong with using strerror_r, and provides more robustness in case somebody wishes to move to multithread or reuse the code somewhere else.
To view, visit change 29424. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I195ebe0fdb05195a7f3b1390630e83084b5dea3a
Gerrit-Change-Number: 29424
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Comment-Date: Tue, 20 Sep 2022 19:17:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de>
Gerrit-MessageType: comment