[FFmpeg-devel] [PATCH] SBG decoder

[Michael Niedermayer]

On Mon, Dec 12, 2011 at 02:51:47PM +0100, Nicolas George wrote:
> Le septidi 17 frimaire, an CCXX, Nicolas George a écrit :
> > fa78d64 lavc: add ffwavesynth pseudo-codec.
> > 4d66f04 lavf: add SBG demuxer.
>
> > There was no remarks about the scheme using the ffwavesynth pseudo-codec to
> > separate the parser from the synthesizer instead of having the demuxer just
> > output PCM. This is probably the most controversial in the incoming patches,
> > some advice would be welcome.
>
> Still no remarks?
>
> As a simple new demuxer+codec, the code can not break anything, but I would
> very much like to hear people's opinion about the
> "demuxer->extradata->pseudocodec" scheme versus the simple
> "demuxer+synthesizer->PCM" scheme.

does it make a user vissible difference which way its done ?
if not i dont think it matters which way its done and i agree that it
cant break anything so not much point in waiting with pushing IMHO



[...]

--
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

DNS cache poisoning attacks, popular search engine, Google internet authority
dont be evil, please
-------------- 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/20111212/75f39b90/attachment.asc>