[FFmpeg-devel] [PATCH] Implement read_file() cmdutils.c option

Michael Niedermayer michaelni
Mon Mar 29 10:05:10 CEST 2010


On Sun, Mar 28, 2010 at 05:09:43PM +0200, Stefano Sabatini wrote:
> Hi, this is useful for factorize code which will be used by incoming
> patches.
> 
> Regards.
> -- 
> FFmpeg = Friendly and Frightening MultiPurpose Extensive Gargoyle

>  cmdutils.c |   23 +++++++++++++++++++++++
>  cmdutils.h |   11 +++++++++++
>  2 files changed, 34 insertions(+)
> 65b3bf95e566e7bdac72b38005c0b65f57c149e6  0002-Implement-cmdutils.c-read_file.patch
> >From 8a8e7eac5928f8aa175d26c4547a384a508c605e Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> Date: Sun, 28 Mar 2010 16:38:42 +0200
> Subject: [PATCH 2/4] Implement cmdutils.c:read_file().
> 
> ---
>  cmdutils.c |   23 +++++++++++++++++++++++
>  cmdutils.h |   11 +++++++++++
>  2 files changed, 34 insertions(+), 0 deletions(-)

factorization of code may not be split into a patch adding the code and 2
patches removing the code
this breaks even gits blame IIRC

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Thouse who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100329/35ab8ace/attachment.pgp>



More information about the ffmpeg-devel mailing list