Patch Set 1: (1 comment)
https://gerrit.osmocom.org/#/c/4232/1/src/common/oml.c File src/common/oml.c: Line 212: bool processed; > you can move that into the scope of the for loop Hmm, I thought the style was to always put it in the beginning of the function. What's the general rule? -- To view, visit https://gerrit.osmocom.org/4232 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id36885e507c4a3203b5662c0fde12c5206174d82 Gerrit-PatchSet: 1 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: Max <msur...@sysmocom.de> Gerrit-Reviewer: Holger Freyther <hol...@freyther.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Max <msur...@sysmocom.de> Gerrit-HasComments: Yes