<div dir="ltr">Hi Harald,<br><br>> I like this very much. Actually, this could even be used by<br>> something like the new osmo-mgw to have more or less<br>> generic transcoding capabilities for gsm-related codecs<br>> as well as PCM / G.711.<br><br>Thanks for your response! Hope my recent commits would help<br>to move this forward.<br><br>> * it would be nice to convert all allocations to talloc, like in other<br>>   osmo-* code.  This facilitates memory leak debugging.<br>> * rather than listing all symbols individually, you can also simply<br>>   export osmo_gapk_* with a wilcard.  All non-exported symbols then<br>>   have to avoid that prefix.  I prefer this to long lists of symbols<br>>   that you often forget to update when adding new functions<br>> * assuming you have multiple processing queues in a program in the<br>>   future, each queue should have a user-settable identifier which is<br>>   then used as a prefix in all the logging.<br><br>Done.<br><br>Only libgsmhr do use generic malloc / free calls. And I think there<br>is no reason to link this library against talloc as there is only<br>one allocation / deallocation cycle.<br><br>BTW: what about the 'laforge/mmx' branch?<br>Does anything prevents us from merging it to the master?<br><br>> Finally, as a personal wishlist item, I would love to see some unit<br>> tests that create a couple of processing queues, destroy them, check<br>> if the resulting encode/decodes is what was expected, and [if possible?]<br>> even check if allocated memory has been properly cleaned up during<br>> destruction of the processing queue.<br><br>Yeah, I have this idea too. But I don't have enough time<br>right now. Will do it as soon as it will be possible.<br>What do you think about adding GAPK to Gerrit?<br><br>Regarding to the proper memory deallocation check, I think it<br>should be possible via the talloc debugging API.<br><br><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>With best regards,<br></div><div>Vadim Yanitskiy.<br></div></div></div></div></div></div>
</div></div>