diff mbox

[FFmpeg-devel] avfilter/scale_eval: remove redundant mathematical constants

Message ID 70d50b66-6075-87fb-98e1-379049c4f965@gyani.pro
State Accepted
Commit 78676ee8f81269937afdcaa2d9bb7bf77f53c1ba
Headers show

Commit Message

Gyan Doshi Dec. 10, 2019, 2 p.m. UTC 
Regards,
Gyan
From 5466e975523502944af84bb43edf17777e793554 Mon Sep 17 00:00:00 2001
From: Gyan Doshi <ffmpeg@gyani.pro>
Date: Tue, 10 Dec 2019 19:24:30 +0530
Subject: [PATCH] avfilter/scale_eval: remove redundant mathematical constants

Even though removed from vf_scale in 3b316f9f22, they were reintroduced
when scale.c, now scale_eval.c, was split off in 037bb4021c
---
 libavfilter/scale_eval.c | 12 ------------
 1 file changed, 12 deletions(-)

Comments

Gyan Doshi Dec. 11, 2019, 6:15 a.m. UTC | #1 
On 10-12-2019 07:30 pm, Gyan wrote:
>
> Regards,
> Gyan

Will push soon.

Gyan
Gyan Doshi Dec. 11, 2019, 10:40 a.m. UTC | #2 
On 11-12-2019 11:45 am, Gyan wrote:
>
>
> On 10-12-2019 07:30 pm, Gyan wrote:
>>
>> Regards,
>> Gyan
>
> Will push soon.

Pushed as 78676ee8f81269937afdcaa2d9bb7bf77f53c1ba

Gyan
diff mbox

Patch

diff --git a/libavfilter/scale_eval.c b/libavfilter/scale_eval.c
index a3439a95e0..6c526a97af 100644
--- a/libavfilter/scale_eval.c
+++ b/libavfilter/scale_eval.c
@@ -25,9 +25,6 @@ 
 #include "libavutil/pixdesc.h"
 
 static const char *const var_names[] = {
-    "PI",
-    "PHI",
-    "E",
     "in_w",   "iw",
     "in_h",   "ih",
     "out_w",  "ow",
@@ -43,9 +40,6 @@  static const char *const var_names[] = {
 };
 
 enum var_name {
-    VAR_PI,
-    VAR_PHI,
-    VAR_E,
     VAR_IN_W,   VAR_IW,
     VAR_IN_H,   VAR_IH,
     VAR_OUT_W,  VAR_OW,
@@ -68,9 +62,6 @@  enum var_name {
  * non-scale2ref specific values.
  */
 static const char *const var_names_scale2ref[] = {
-    "PI",
-    "PHI",
-    "E",
     "in_w",   "iw",
     "in_h",   "ih",
     "out_w",  "ow",
@@ -124,9 +115,6 @@  int ff_scale_eval_dimensions(void *log_ctx,
         main_desc = av_pix_fmt_desc_get(main_link->format);
     }
 
-    var_values[VAR_PI]    = M_PI;
-    var_values[VAR_PHI]   = M_PHI;
-    var_values[VAR_E]     = M_E;
     var_values[VAR_IN_W]  = var_values[VAR_IW] = inlink->w;
     var_values[VAR_IN_H]  = var_values[VAR_IH] = inlink->h;
     var_values[VAR_OUT_W] = var_values[VAR_OW] = NAN;