Patch Set 1:

> Would it make sense to add __attribute__((warn_unused_result)) to
 > rate_ctr_group_alloc() or there might be legitimate case when we
 > can ignore the return value?

yes, could make senes. thanks for the suggestion. but then this could/should be 
part of a larger patchset as this is probably the case for many of our API 
functions?

-- 
To view, visit https://gerrit.osmocom.org/3184
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7dad4a4d52fe05f6b990359841b4408df5990e21
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: Vadim Yanitskiy <[email protected]>
Gerrit-HasComments: No

Reply via email to