Patch Set 2: Code-Review+1 (2 comments)
two unimportant cosmetics that you could tweak ... otherwise I'd +2 if I had the powers https://gerrit.osmocom.org/#/c/2920/2/Transceiver52M/sigProcLib.cpp File Transceiver52M/sigProcLib.cpp: Line 791: return shape; return convolve(); and drop shape, like above? Line 944: return shape; same? -- To view, visit https://gerrit.osmocom.org/2920 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If53da1bf77b5944b6117765fa98ce12e1ccdeede Gerrit-PatchSet: 2 Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Owner: Tom Tsou <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr <[email protected]> Gerrit-Reviewer: Tom Tsou <[email protected]> Gerrit-HasComments: Yes
