Re: [x265] [PATCH] Added fast intra search option

2014-08-13 Thread Deepthi Nandakumar
There are a couple of warnings our regression tests caught with this. Can you take a look? source\encoder\predict.cpp(78): warning C4800: 'const unsigned char' : forcing value to bool 'true' or 'false' (performance warning) (IntraFilterType can be bool, I think?).

Re: [x265] [PATCH] Added fast intra search option

2014-08-13 Thread dave
In building with gcc debian 4.7.2-5 I get no warnings. On 08/13/2014 05:46 AM, Deepthi Nandakumar wrote: There are a couple of warnings our regression tests caught with this. Can you take a look? source\encoder\predict.cpp(78): warning C4800: 'const unsigned char' : forcing value to bool

Re: [x265] [PATCH] Added fast intra search option

2014-08-13 Thread dave
On 08/12/2014 10:22 PM, Steve Borho wrote: On 08/12, dtyx...@gmail.com wrote: # HG changeset patch # User David T Yuen dtyx...@gmail.com # Date 1407882999 25200 # Node ID 75e4ad481b3668b1e420ede300287aa3ea3fb8d5 # Parent 8a7f4bb1d1be32fe668d410450c2e320ccae6098 Added fast intra search option

Re: [x265] [PATCH] Added fast intra search option

2014-08-12 Thread Steve Borho
On 08/12, dtyx...@gmail.com wrote: # HG changeset patch # User David T Yuen dtyx...@gmail.com # Date 1407882999 25200 # Node ID 75e4ad481b3668b1e420ede300287aa3ea3fb8d5 # Parent 8a7f4bb1d1be32fe668d410450c2e320ccae6098 Added fast intra search option This version calls intra_pred_allangs