[MERGED] libosmo-abis[master]: cosmetic: ipa.c: use msgb_dequeue(), drop local var

2017-09-18 Thread Neels Hofmeyr
Neels Hofmeyr has submitted this change and it was merged. Change subject: cosmetic: ipa.c: use msgb_dequeue(), drop local var .. cosmetic: ipa.c: use msgb_dequeue(), drop local var Make ipa_server_conn_write() a lot simpler an

libosmo-abis[master]: cosmetic: ipa.c: use msgb_dequeue(), drop local var

2017-09-14 Thread Harald Welte
Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3942 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id968e60d081a6d69de482a908ca3fe5e1044aaa3 Gerrit-PatchSet: 2 Gerrit-Project: libosmo-abis Gerrit-Branch: master Ge

libosmo-abis[master]: cosmetic: ipa.c: use msgb_dequeue(), drop local var

2017-09-14 Thread Neels Hofmeyr
Patch Set 2: cosmetic because for all I can tell it does the exact same thing as before, so it's just removing code dup... -- To view, visit https://gerrit.osmocom.org/3942 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id968e60d081a6d6

libosmo-abis[master]: cosmetic: ipa.c: use msgb_dequeue(), drop local var

2017-09-14 Thread Max
Patch Set 2: Code-Review+1 I don't think it's cosmetic change (it's more than updating comments, formatting etc) but seems good anyway. -- To view, visit https://gerrit.osmocom.org/3942 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id9

[PATCH] libosmo-abis[master]: cosmetic: ipa.c: use msgb_dequeue(), drop local var

2017-09-13 Thread Neels Hofmeyr
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3942 to look at the new patch set (#2). cosmetic: ipa.c: use msgb_dequeue(), drop local var Make ipa_server_conn_write() a lot simpler and shorter by using msgb_dequeue() and avoiding local