<div dir="ltr">Hi,<br><div><div class="gmail_extra"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> +        // add padding to beginning<br>
> +        if (sys->padding) {<br>
> +            const size_t leftover_space = OPUS_FRAME_SIZE - sys->i_nb_samples;<br>
> +            padding_start = fill_buffer(enc, padding_start, sys->padding,<br>
> +                    __MIN(sys->padding->i_nb_samples, leftover_space));<br>
> +            if (sys->padding->i_nb_samples <= 0) {<br>
> +                block_Release(sys->padding);<br>
> +                sys->padding = 0;<br>
> +            }<br>
> +        }<br>
> +<br>
> +        if (!sys->padding) {<br>
</div>use else<br></blockquote><div><br><br></div><div>Since sys->padding may have changed to 0 in the previous if block, the two branches aren't mutually exclusive. That said, I will put a comment in to make that clearer.<br>
</div><br></div><div class="gmail_quote">Thanks for getting back to me so quickly, I will resend a patch once with all of those changes once I've updated the patch to use newer opus API.<br><br></div><div class="gmail_quote">
Best,<br></div><div class="gmail_quote">Tristan<br clear="all"></div><br>-- <br>Tristan Matthews<br>web: <a href="http://tristanswork.blogspot.com">http://tristanswork.blogspot.com</a><br>
</div></div></div>