[FFmpeg-devel] lavfi/fspp: Remove two unused macros

Submitted by Carl Eugen Hoyos on April 14, 2019, 8:57 p.m.

Details

Message ID CAB0OVGp-K+_6EG6w2vpqEdK29ScCRq3PghhAPp_6-o7EGKbqxA@mail.gmail.com
State Accepted
Headers show

Commit Message

Carl Eugen Hoyos April 14, 2019, 8:57 p.m.
Hi!

Attached patch removes two unused macros as a preparation for a new patch
to silence a clang warning.

Please comment, Carl Eugen

Comments

Michael Niedermayer April 15, 2019, 9:56 p.m.
On Sun, Apr 14, 2019 at 10:57:35PM +0200, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch removes two unused macros as a preparation for a new patch
> to silence a clang warning.
> 
> Please comment, Carl Eugen

>  vf_fspp.h |    2 --
>  1 file changed, 2 deletions(-)
> ec3246d97102d701f4a979e29a42857bf6745078  0001-lavfi-fspp-Remove-two-unused-macros.patch
> From 133f21a9df22f5ae15ea7feef57bb6c1bc7a7bf4 Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos <ceffmpeg@gmail.com>
> Date: Sun, 14 Apr 2019 22:56:17 +0200
> Subject: [PATCH] lavfi/fspp: Remove two unused macros.

LGTM

thx

[...]
Carl Eugen Hoyos April 15, 2019, 10:44 p.m.
2019-04-15 23:56 GMT+02:00, Michael Niedermayer <michael@niedermayer.cc>:
> On Sun, Apr 14, 2019 at 10:57:35PM +0200, Carl Eugen Hoyos wrote:
>> Hi!
>>
>> Attached patch removes two unused macros as a preparation for a new patch
>> to silence a clang warning.
>>
>> Please comment, Carl Eugen
>
>>  vf_fspp.h |    2 --
>>  1 file changed, 2 deletions(-)
>> ec3246d97102d701f4a979e29a42857bf6745078
>> 0001-lavfi-fspp-Remove-two-unused-macros.patch
>> From 133f21a9df22f5ae15ea7feef57bb6c1bc7a7bf4 Mon Sep 17 00:00:00 2001
>> From: Carl Eugen Hoyos <ceffmpeg@gmail.com>
>> Date: Sun, 14 Apr 2019 22:56:17 +0200
>> Subject: [PATCH] lavfi/fspp: Remove two unused macros.
>
> LGTM

Patch applied.

Thank you, Carl Eugen

Patch hide | download patch | download mbox

From 133f21a9df22f5ae15ea7feef57bb6c1bc7a7bf4 Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos <ceffmpeg@gmail.com>
Date: Sun, 14 Apr 2019 22:56:17 +0200
Subject: [PATCH] lavfi/fspp: Remove two unused macros.

---
 libavfilter/vf_fspp.h |    2 --
 1 file changed, 2 deletions(-)

diff --git a/libavfilter/vf_fspp.h b/libavfilter/vf_fspp.h
index 74a3447..802db14 100644
--- a/libavfilter/vf_fspp.h
+++ b/libavfilter/vf_fspp.h
@@ -32,8 +32,6 @@ 
 #define DCTSIZE_S "8"
 
 #define FIX(x,s)  ((int) ((x) * (1 << s) + 0.5) & 0xffff)
-#define C64(x)    ((uint64_t)((x) | (x) << 16)) <<32 | (uint64_t)(x) | (uint64_t)(x) << 16
-#define FIX64(x,s)  C64(FIX(x,s))
 
 #define MULTIPLY16H(x,k)   (((x) * (k)) >> 16)
 #define THRESHOLD(r,x,t)                         \
-- 
1.7.10.4