[FFmpeg-cvslog] r22522 - in trunk: configure libavcodec/Makefile libavcodec/dsputil.c libavcodec/dsputil.h libavcodec/dwt.c libavcodec/dwt.h libavcodec/ivi_dsp.c libavcodec/snow.c libavcodec/snow.h libavcodec/x86/...

[Michael Niedermayer]

On Sun, Mar 14, 2010 at 07:57:00PM +0000, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> R> On Sun, Mar 14, 2010 at 07:38:38PM +0000, M?ns Rullg?rd wrote:
> >> Michael Niedermayer <michaelni at gmx.at> writes:
> >> 
> >> > On Sun, Mar 14, 2010 at 06:50:12PM +0100, mru wrote:
> >> >> Author: mru
> >> >> Date: Sun Mar 14 18:50:12 2010
> >> >> New Revision: 22522
> >> >> 
> >> >> Log:
> >> >> Separate DWT from snow and dsputil
> >> >> 
> >> >> This moves the DWT functions from snow.c and dsputil.c to a file of
> >> >> their own.  A new struct, DWTContext, holds the function pointers
> >> >> previously part of DSPContext.
> >> >> 
> >> >> Added:
> >> >>    trunk/libavcodec/dwt.c
> >> >>    trunk/libavcodec/dwt.h
> >> >>       - copied, changed from r22521, trunk/libavcodec/snow.h
> >> >
> >> > is dwt.c not missing a svn cp?
> >> 
> >> It's only a tiny part of snow.c, the diff would be enormous.  Besides,
> >> git blame has no trouble tracking the origin of the code.
> >
> > we are using svn for trunk, it would thus be nice if svn metadata would
> > stay correct.
> > and gits tracking is guessing, svn stores the true origin.
> 
> Nevertheless git blame give much more details, and correct,
> information.  You should try it some time.

i did, its not always correct, svn was more correct for many cases
that was when i last tried which is a while ago.

also correct is correcter or equally correct than a guess, always
the only case where git can be better is when svn lacks the flexibility
to store true ancestry


but the core thing is we use svn, like we use C99
adding GNU specific things is not ok as it breaks some peoples build
in the same sense your behavior breaks some features of the repository 
for people using svn instead of git-svn.
This is not nice to these people

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

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- 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-cvslog/attachments/20100314/ce6b6d57/attachment.pgp>