[MPlayer-dev-eng] [PATCH] Movie fragment support for MOV/MP4 demuxer

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Tue Sep 18 10:39:14 CEST 2007


Hello,
On Tue, Sep 18, 2007 at 02:29:50AM +0200, Diego Biurrun wrote:
> The patch is still huge and randomly indented with a mix of tabs and
> spaces.  Please fix this and try to split it into even smaller chunks.
> This will make finding a reviewer for your code much easier.

And please pay special attention to DOCS/tech/patches.txt, esp. section
5 and the recommended diff command to clean up your patch, I think I
have seen at least one change that is not necessary and will disappear
from the patch with it.

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list