[MPlayer-dev-eng] [PATCH] Audio support for AVISynth
Reimar Döffinger
Reimar.Doeffinger at stud.uni-karlsruhe.de
Sun Jul 23 15:32:40 CEST 2006
Hello,
On Sun, Jul 23, 2006 at 02:53:26AM +0200, Gianluigi Tiesi wrote:
> Revised, "cosmetized" and cleaned patch:
> http://oss.netfarm.it/mplayer/patches/xx_demux_avs_audio.diff
> - if (!AVS->x) { mp_msg(MSGT_DEMUX,MSGL_V,"AVS: failed to load "Q(x)"()\n"); return 0; }
> + if (!AVS->x) { mp_msg(MSGT_DEMUX, MSGL_V,"AVS: failed to load "Q(x)"()\n"); return 0; }
can't see a non-cosmetic change here :-)
Same here:
> - mp_msg(MSGT_DEMUX ,MSGL_V, "AVS: failed to load avisynth.dll\n");
> + mp_msg(MSGT_DEMUX, MSGL_V, "AVS: failed to load avisynth.dll\n");
here:
> - }
> -
> + }
And many, many more places (including some conversions of // to /*
comments).
Also I think this:
> + sh_audio_t *sh_audio = d_audio->sh;
Might crash in video-only case, though I didn't check properly.
This seems like an unrelated change:
> - d_video->pts=AVS->frameno / sh_video->fps; // OSD
> + dp->pts = (float) AVS->frameno / sh_video->fps;
I'm against commiting it in this form.
Greetings,
Reimar Döffinger
More information about the MPlayer-dev-eng
mailing list