On Thu, Mar 03, 2016 at 10:01:28AM -0800, NagaChaitanya Vellanki wrote:
> ---
> libavutil/Makefile | 1 +
> libavutil/color_utils.c| 29 +
> tests/fate/libavutil.mak | 4 +
> tests/ref/fate/color_utils | 285
> +
> 4 files
Derek,
I agree with you, checked this on ubuntu, mac os both x86_32, x86_64.
On Thu, Mar 3, 2016 at 10:11 AM, Derek Buitenhuis <
derek.buitenh...@gmail.com> wrote:
> On 3/3/2016 6:01 PM, NagaChaitanya Vellanki wrote:
> > + printf("AVColorTransferCharacteristic=%d calling func(%f)
>
On Thu, Mar 03, 2016 at 06:11:19PM +, Derek Buitenhuis wrote:
> On 3/3/2016 6:01 PM, NagaChaitanya Vellanki wrote:
> > + printf("AVColorTransferCharacteristic=%d calling func(%f)
> > expected=%f\n",
> > + i, test_data[j], result);
>
> Comparing floats exactly
On 3/3/2016 6:01 PM, NagaChaitanya Vellanki wrote:
> + printf("AVColorTransferCharacteristic=%d calling func(%f)
> expected=%f\n",
> + i, test_data[j], result);
Comparing floats exactly probably isn't a great idea, since
floating point implementations differ
---
libavutil/Makefile | 1 +
libavutil/color_utils.c| 29 +
tests/fate/libavutil.mak | 4 +
tests/ref/fate/color_utils | 285 +
4 files changed, 319 insertions(+)
create mode 100644 tests/ref/fate/color_utils
diff --git
Please ignore my previous patches.
The commit message was incorrectly formatted. Please consider this patch.
NagaChaitanya Vellanki (1):
Add test for avpriv_get_trc_function_from_trc function
libavutil/Makefile | 1 +
libavutil/color_utils.c| 29 +
tests/fate/libavutil.mak
Hi all,
This is my first patch. git send-mail did not work for me. Sending the
patch via gmail gui. Please review my patch.
Thank you,
NagaChaitanya Vellanki
0001-Add-test-for-avpriv_get_trc_function_from_trc-functi.patch
Description: Binary data
___
On Tue, Mar 01, 2016 at 09:11:21PM +0100, Michael Niedermayer wrote:
> On Tue, Mar 01, 2016 at 11:37:37AM -0800, NagaChaitanya Vellanki wrote:
> > ---
> > libavutil/Makefile | 1 +
> > libavutil/color_utils.c | 53
> > +
> > 2 files changed,
On Tue, Mar 01, 2016 at 11:37:37AM -0800, NagaChaitanya Vellanki wrote:
> ---
> libavutil/Makefile | 1 +
> libavutil/color_utils.c | 53
> +
> 2 files changed, 54 insertions(+)
to test avpriv_get_trc_function_from_trc(), one could
for each
---
libavutil/Makefile | 1 +
libavutil/color_utils.c | 53 +
2 files changed, 54 insertions(+)
diff --git a/libavutil/Makefile b/libavutil/Makefile
index a4d79cd..934564f 100644
--- a/libavutil/Makefile
+++ b/libavutil/Makefile
@@ -176,6
On Tue, Mar 01, 2016 at 10:41:32AM -0800, NagaChaitanya Vellanki wrote:
> ---
> libavutil/Makefile | 1 +
> libavutil/color_utils.c | 52
> +
> 2 files changed, 53 insertions(+)
>
I'm afraid testing the result of a switch with an array has
---
libavutil/Makefile | 1 +
libavutil/color_utils.c | 52 +
2 files changed, 53 insertions(+)
diff --git a/libavutil/Makefile b/libavutil/Makefile
index a4d79cd..934564f 100644
--- a/libavutil/Makefile
+++ b/libavutil/Makefile
@@ -176,6
Improve coverage by adding test.
NagaChaitanya Vellanki (1):
Add test for avpriv_get_trc_function_from_trc function
libavutil/Makefile | 1 +
libavutil/color_utils.c | 52 +
2 files changed, 53 insertions(+)
--
2.7.2
Hi all,
This is my first patch. git send-mail did not work for me. Sending the
patch via gmail gui. Please review my patch.
Thank you,
NagaChaitanya Vellanki
0001-Add-test-for-avpriv_get_trc_function_from_trc-functi.patch
Description: Binary data
___
14 matches
Mail list logo
