<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"></div>Hi Xuefeng<br><br></div><div class="gmail_extra">Thanks for your patch, there's a few thing I'd like to see changed though to get started.<br><br></div><div class="gmail_extra">Begin by splitting the 12-bit support into a standalone patch, and the AVC-Intra extensions into another patch on top of that.<br><br></div><div class="gmail_extra">There's also a lot of code duplication going on here. You should try to extend the existing AVC-Intra code at <a href="https://git.videolan.org/?p=x264.git;a=blob;f=encoder/encoder.c;h=79997b96ef82fda8838e1eaeeeb7551a3d1aa651;hb=HEAD#l650">https://git.videolan.org/?p=x264.git;a=blob;f=encoder/encoder.c;h=79997b96ef82fda8838e1eaeeeb7551a3d1aa651;hb=HEAD#l650</a> instead of using those apply_* functions.<br><br></div><div class="gmail_extra">Henrik<br></div><div class="gmail_extra"><br></div></div>