[Ffmpeg-devel] [PATCH] DXA demuxer and decoder

Michael Niedermayer michaelni
Tue Mar 13 22:36:20 CET 2007


Hi

On Tue, Mar 13, 2007 at 08:06:59PM +0000, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > Hi
> >
> > On Tue, Mar 13, 2007 at 08:27:23PM +0200, Kostya wrote:
> > [...]
> >> [...]
> >> > this is not thread safe, please pass palette changes like any other packet
> >> 
> >> reworked
> >>  
> >> > [...]
> >> > 
> >> > you also could update the Changelog (with svn rev)
> >> 
> >> ?
> >
> > the Changelog file in svn (tunk/Changelog or such) with svn revision, 
> > no other entry there has a svn revision sadly, i thought it would be nice
> > if all big changes would be added during commit also to this file with
> > a revision when it was added, maybe something like
> >
> > FOO demuxer (in r1234)
> 
> Why include the svn version in the file when the file history already
> has this?

2 reasons
1. you might have Changelog but not a proper svn checkput with svn instaleed
   a network connection, ...
2. for how many entries is the time when a entry was added the time when a 
   feature was added? AFIAK there are many which have been added to Changelog
   much later

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070313/cfe9774c/attachment.pgp>



More information about the ffmpeg-devel mailing list