[FFmpeg-devel] [PATCH 11/11] libmodplug: remove mathematical constants now redundant.

[Michael Niedermayer]

On Thu, Oct 13, 2011 at 06:09:41PM +0200, Clément Bœsch wrote:
> On Thu, Oct 13, 2011 at 05:54:11PM +0200, Nicolas George wrote:
> > 
> > Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> > ---
> >  libavformat/libmodplug.c |    5 -----
> >  1 files changed, 0 insertions(+), 5 deletions(-)
> > 
> > diff --git a/libavformat/libmodplug.c b/libavformat/libmodplug.c
> > index 83e9322..5e9121f 100644
> > --- a/libavformat/libmodplug.c
> > +++ b/libavformat/libmodplug.c
> > @@ -59,7 +59,6 @@ typedef struct ModPlugContext {
> >  } ModPlugContext;
> >  
> >  static const char *var_names[] = {
> > -    "E", "PHI", "PI",
> >      "x", "y",
> >      "w", "h",
> >      "t",
> > @@ -68,7 +67,6 @@ static const char *var_names[] = {
> >  };
> >  
> >  enum var_name {
> > -    VAR_E, VAR_PHI, VAR_PI,
> >      VAR_X, VAR_Y,
> >      VAR_W, VAR_H,
> >      VAR_TIME,
> > @@ -275,9 +273,6 @@ static int modplug_read_packet(AVFormatContext *s, AVPacket *pkt)
> >          if (modplug->video_switch) {
> >              double var_values[VAR_VARS_NB];
> >  
> > -            var_values[VAR_E      ] = M_E;
> > -            var_values[VAR_PHI    ] = M_PHI;
> > -            var_values[VAR_PI     ] = M_PI;
> >              var_values[VAR_W      ] = modplug->w;
> >              var_values[VAR_H      ] = modplug->h;
> >              var_values[VAR_TIME   ] = modplug->packet_count * modplug->ts_per_packet;
> 
> This should be fine, thanks :)

applied, thanks to all

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111013/266da959/attachment.asc>