[FFmpeg-devel] [PATCH] Add FITS Demuxer

Rostislav Pehlivanov atomnuker at gmail.com
Tue Jul 18 23:35:44 EEST 2017

On 18 July 2017 at 19:42, Nicolas George <george at nsup.org> wrote:

> Le decadi 30 messidor, an CCXXV, Paul B Mahol a écrit :
> > To not cause drama on git repo I kindly ask Michael to remove your git
> > push access.
> I will leave the people who have that kind of power judge who must be
> banished :
> The one who made useful reviews to the patches, with constructive
> comments, and insists on high-quality code.
> Or the one who makes ad-hominem attacks, threatens to fork when he
> cannot have his way and never addresses the actual technical question.
> --
>   Nicolas George
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Chill, both of you.

Its a crappy format, no reason to blame anyone else but the format.
We have plenty of crappy formats which have no clear separation between
packets so demuxers have to give not-entirely-demuxed packets to the
decoder which also has to skip past junk. Its both wrong and its not,
because it isn't defined anywhere and the packets packed in the file were
never meant to go anywhere but the format they were packed in.

I think the image2 demuxer shouldn't handle this type of junk/useless data
filtering and would rather see a separate demuxer like the current patch
which deals with crap. Its not elegant but blame the format and the fact
that back when it was designed there weren't even any universal containers
for audio/video.

More information about the ffmpeg-devel mailing list