<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 18, 2014 at 12:09 PM, Derek Buitenhuis <span dir="ltr"><<a href="mailto:derek.buitenhuis@gmail.com" target="_blank">derek.buitenhuis@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 2/18/2014 6:00 PM, Steve Borho wrote:<br>
> I disagree; any program that used the old API can use the new one without any change because before that commit internal and input bit depth were always the same.  Both before and after that commit that particular field has only served to validate that the user was aware of whether the library was compiled for 16bpp or not.<br>

<br>
</div>If it really is this, then the API was documented wrong, and it does<br>
some weird things.<br></blockquote><div><br></div><div>It was documented wrongly; I've fixed that in recent commits.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

e.g. x265_picture_init() copies it over into the x265_picture.bitDepth field,<br>
which sure does look like *input* depth...<br></blockquote><div><br></div><div>It is the default input depth. The user is responsible for making pic.bitDepth match the actual bit depth of the pixels they are passing in, if it doesn't match the internal depth. I tried to make that clear in the documentation for pic.bitDepth</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">
> Let's keep this polite.  I'll err on the side of bumping the build number in the future.<br>
<br>
</div>I may be somewhat grumpy due to coming back with my inbox full of people<br>
blaming me.<br></blockquote><div><br></div><div>Point them at me; it was definitely my fault.</div><div><br></div><div>So when I bump X265_BUILD, ffmpeg will no longer try to compile against x265 until it is patched?  How does this work in practice?  Should I be sending patches for ffmpeg/libav when we change API?</div>
<div><br></div></div>-- <br>Steve Borho
</div></div>