<div dir="ltr">Hi Ulrich, <br><br>Thank you for your input! Ptal at the patch I set above and let me know if it's been resolved to your liking.<div><br></div><div>Cheers,</div><div>Drew</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 16, 2017 at 3:54 AM Ulrich Windl <<a href="mailto:Ulrich.Windl@rz.uni-regensburg.de">Ulrich.Windl@rz.uni-regensburg.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>>> Drew Allen <<a href="mailto:bitllama@google.com" target="_blank">bitllama@google.com</a>> schrieb am 10.10.2017 um 20:29 in Nachricht<br>
<<a href="mailto:CABQ9DctQ0%2BgBgUif7BBJpjjKR7_V_H5OC1JM47w50oaaLXL4Tg@mail.gmail.com" target="_blank">CABQ9DctQ0+gBgUif7BBJpjjKR7_V_H5OC1JM47w50oaaLXL4Tg@mail.gmail.com</a>>:<br>
> Hi Jean-Marc,<br>
><br>
> Thanks for the feedback. Attached are my comments and an updated patch.<br>
><br>
> 1) I see that it's adding an #include of stdarg.h to opus_multistream.h<br>
> Is that left over from the previous version?<br>
><br>
><br>
> *That was a typo. Fixed.*<br>
><br>
> 2) Someone on this list might know better than I do on that one, but for<br>
> the new _ctl_va_list() calls, I believe the convention is for va_start()<br>
> and va_end() to appear in the caller rather than in in the va_list()<br>
> function itself.<br>
><br>
> *My understanding is that it's impossible to pass ellipsis to another<br>
> function.*<br>
<br>
>From "man stdarg.h": " The object ap may be passed as an argument to another function; (...)". See also "va_copy()".<br>
<br>
[...]<br>
<br>
Regards,<br>
Ulrich<br>
<br>
<br>
<br>
_______________________________________________<br>
opus mailing list<br>
<a href="mailto:opus@xiph.org" target="_blank">opus@xiph.org</a><br>
<a href="http://lists.xiph.org/mailman/listinfo/opus" rel="noreferrer" target="_blank">http://lists.xiph.org/mailman/listinfo/opus</a><br>
</blockquote></div>