[FFmpeg-devel] [PATCH] Several, actually, regarding the QuickTime palette issue in Matroska

Mats Peterson matsp888 at yahoo.com
Wed Dec 16 19:52:22 CET 2015


On 12/16/2015 07:45 PM, Lou Logan wrote:
> On Wed, 16 Dec 2015 19:18:33 +0100, Mats Peterson wrote:
>
>> If you can create a proper patchset out of the archive, you can remove
>> that single line.
>
> That will not happen.
>
>> It's unnecessary for me to upload a new archive with such a small
>> change. It's a whole new file, at that.
>
> Another reason nobody is looking at your patch(es) is that it has been
> compressed; adding another step reviewers must take to accommodate your
> non-standard methods of patch submission. Most people review via their
> mail client, and if there is a patch that takes more effort than
> usually required then it is more likely to be ignored because there are
> always other patches to review and test.
>
> I looked at the contents of your archive and it contains four .diff
> files and a single .c file. This is not possible to review or apply.
> Please use git to make a proper patch as requested previously.
>
> See:
> <http://www.ffmpeg.org/developer.html#Submitting-patches>
> <http://www.ffmpeg.org/git-howto.html>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

Haha... OK then. I've done what I can. Take it or leave it.

Mats
-- 
Mats Peterson
http://matsp888.no-ip.org/~mats/


More information about the ffmpeg-devel mailing list