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

Reply via email to