This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.
Neels Hofmeyr nhofmeyr at sysmocom.deOn Thu, Apr 07, 2016 at 11:15:31AM +0200, Max wrote: > They were useful for debugging the tests but once it's ready the'd just > clutter the output unnecessary. Still, they'll be useful again when we > want to expand tests further so instead of adding and removing them > every time I'd prefer to keep them. would be nice to mention in the log message body... > > - 5 functions are added, only two are used. Why add unused functions? > > Will they ever be used? > > Perhaps :) also nice to mention in the log message body > > - Two functions are factored out and three others are added afresh. Admitted, > > the second part of the log message mentions adding functions, yet the summary > > above sounds like it's only factoring out. > > > How would you summarize it? "vty_test_runner: ipa_send: factor out 2, add 3 functions" :) BTW, just to set the tone, I would like to mention that my feedback is generally meant as friendly comment, in a happy office, from colleague to colleague ;) (I'd be glad if you'd return such reviews on my patches.) ~Neels -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: <http://lists.osmocom.org/pipermail/openbsc/attachments/20160407/c6e75cf4/attachment.bin>