[opus] Antw: Re: [PATCH] Support for Channel Mapping 253.
bitllama at google.com
Mon Oct 23 20:08:53 UTC 2017
Thank you for your input! Ptal at the patch I set above and let me know if
it's been resolved to your liking.
On Mon, Oct 16, 2017 at 3:54 AM Ulrich Windl <
Ulrich.Windl at rz.uni-regensburg.de> wrote:
> >>> Drew Allen <bitllama at google.com> schrieb am 10.10.2017 um 20:29 in
> <CABQ9DctQ0+gBgUif7BBJpjjKR7_V_H5OC1JM47w50oaaLXL4Tg at mail.gmail.com>:
> > Hi Jean-Marc,
> > Thanks for the feedback. Attached are my comments and an updated patch.
> > 1) I see that it's adding an #include of stdarg.h to opus_multistream.h
> > Is that left over from the previous version?
> > *That was a typo. Fixed.*
> > 2) Someone on this list might know better than I do on that one, but for
> > the new _ctl_va_list() calls, I believe the convention is for va_start()
> > and va_end() to appear in the caller rather than in in the va_list()
> > function itself.
> > *My understanding is that it's impossible to pass ellipsis to another
> > function.*
> From "man stdarg.h": " The object ap may be passed as an argument to
> another function; (...)". See also "va_copy()".
> opus mailing list
> opus at xiph.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the opus