[MPlayer-dev-eng] Indentation changes in patches

Alexander Strasser eclipse7 at gmx.net
Mon Jul 12 14:56:30 CEST 2004


Hi Michael,
On Mon, Jul 12, 2004 at 02:38:02PM +0200, Michael Niedermayer wrote:
> maybe theres a solution which would allow to commit whitespace changes in 
> functional patches but still display them separated on cvslog
> by simply changing the script which generates the diff which is posted, 
> actually i had this idea a long time ago but generating diffs which hide 
> whitespace changes is risky as mistakes which destroy correct indention would 
> be missed
> a possible solution would be to generate 2 diffs for each commit, one with 
> whitespace change the other without, and if they are near identical == 
> similar size or so then only post the one with whitespace changes included 
> otherwise post both in one message
If i got you write, this sounds much more acceptable to me. Maybe we should
go on and elaborate a little more on this idea.

  Alex (beastd)




More information about the MPlayer-dev-eng mailing list