The data type is changed to uint64_t and ok() is also changed accordingly
in this patch. Please tell any other changes are required?

On Mon, Sep 21, 2015 at 9:52 PM, chen <[email protected]> wrote:

> The ok() is self verify code, we need correct it.
>
> At 2015-09-21 12:39:24,"Divya Manivannan" <[email protected]>
> wrote:
>
> There is no dynamic range overflow in 8bpp and 10bpp mode. But the
> threshold in the ok() function is below the dynamic range for 10bpp mode.
>
> On Wed, Sep 16, 2015 at 8:28 PM, chen <[email protected]> wrote:
>
>> Are you found dynamic range overflow in 8bpp and 10bpp mode?
>>
>>
>> At 2015-09-16 17:11:29,"Divya Manivannan" <[email protected]> wrote:
>> ># HG changeset patch
>> ># User Divya Manivannan <[email protected]>
>> ># Date 1442392516 -19800
>> >#      Wed Sep 16 14:05:16 2015 +0530
>> ># Node ID ee39c10fd573444710aca40ec3bd572aac8b3cd0
>> ># Parent  365f7ed4d89628d49cd6af8d81d4edc01f73ffad
>> >sse: fix data type for sse functions
>> >
>> >diff -r 365f7ed4d896 -r ee39c10fd573 source/common/common.h
>> >--- a/source/common/common.h        Tue Sep 08 16:38:01 2015 +0530
>> >+++ b/source/common/common.h        Wed Sep 16 14:05:16 2015 +0530
>> >@@ -134,12 +134,6 @@
>> > typedef int32_t  ssum2_t; // Signed sum
>> > #endif // if HIGH_BIT_DEPTH
>> >
>> >-#if X265_DEPTH <= 10
>> >-typedef uint32_t sse_ret_t;
>> >-#else
>> >-typedef uint64_t sse_ret_t;
>> >-#endif
>> >-
>> > #ifndef NULL
>> > #define NULL 0
>> > #endif
>>
>>
>> _______________________________________________
>> x265-devel mailing list
>> [email protected]
>> https://mailman.videolan.org/listinfo/x265-devel
>>
>>
>
> _______________________________________________
> x265-devel mailing list
> [email protected]
> https://mailman.videolan.org/listinfo/x265-devel
>
>
_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel

Reply via email to