Hi Aurelien<br><br>On 8/14/07, <b class="gmail_sendername">Aurelien Jacobs</b> <<a href="mailto:aurel@gnuage.org">aurel@gnuage.org</a>> wrote:<br><div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
> - outbuffer[i]=av_clip(lrintf(4*ppf_vector[i]), -32768, 32767);<br>> + outbuffer[i]=av_clip(lrintf(4*ppf_vector[i]), -32768, 32768);<br><br>This looks strange. The maximum 16bits signed integer is 32767, not 32768.
</blockquote><div><br>Fixed, thanks. <br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">And if the range you want is indeed -32768 - 32767, you'd better use
<br>av_clip_int16().</blockquote><div><br>It would be nice but I can't find it...</div><br>Thanks for your review<br><br>Reynaldo<br></div><br>