[FFmpeg-cvslog] r10197 - in trunk: doc/optimization.txt ffplay.c

[Måns Rullgård]

Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:

> Hello,
> On Thu, Aug 23, 2007 at 08:09:49PM +0100, M?ns Rullg?rd wrote:
>> Diego Biurrun <diego at biurrun.de> writes:
>> > On Thu, Aug 23, 2007 at 06:42:09PM +0200, Luca Barbato wrote:
>> >> lu_zero wrote:
>> >> > 
>> >> > Log:
>> >> > PowerPC/CELL/Altivec pointers
>> >> > 
>> >> > Modified:
>> >> >    trunk/doc/optimization.txt
>> >> >    trunk/ffplay.c
>> >> 
>> >> I hate that... stray commit reverted...
>> >
>> > *Always* doublecheck with 'svn diff' before committing.  This has saved
>> > me from embarassment countless times ...
>> 
>> That's a thing I like about git.  You have explicitly add the files
>> you want to the commit.  That makes for much fewer accidents of that
>> kind, and you also have another chance to review your commits before
>> pushing them out.
>
> Huh? You can explicitly specify the files with svn as well, just as
> you can use git-commit -a which commits all modifications. I can't
> see the difference here.

The difference is that -a isn't the default, and that commits
initially only go to your local repo so if you make a mistake it's
easy to fix before making it public.

> I decided the only sure way for me is a script like this (now also shows
> the log messages):
> http://www1.mplayerhq.hu/~reimar/git-mycommit.sh

That scripts assumes you've already committed your changes with
git-commit, and asks for confirmation before pushing to svn with
git-svn.  Useful as it may be, it's not relevant to this discussion.

-- 
M?ns Rullg?rd
mans at mansr.com