[mythtv] [patch] tab cleanup of libmyth

Daniel Thor Kristjansson danielk at mrl.nyu.edu
Thu Jan 20 10:14:19 EST 2005


On Thu, 20 Jan 2005, Andrew Wilson wrote:

]In case it helps, 
]
]"cvs diff -w" will ignore whitespace-only diffs
]
]You can put "diff -w" in your ~/.cvsrc to achieve this.
]

This is bad. If you put an if around a block of code it will ignore the 
change in indentation. It's best to apply outstanding patches, then 
apply the tab patches (so that the rejects will be in the tab patch and 
not a functionality patch). Then regenerate all outstanding patches that 
weren't committed for some reason, after a 'cvs update' Rejects in the 
tab patch are of course safe and I can just generate a new tab patch for 
the missed tabs later.

-- Daniel


Example of "cvs diff -w" problem:

orignal:
    a;
    b;
    c;

modified:
    if (x)
    {
        a;
        b;
        c;
    }
    else
        d;

modified by a "cvs diff -w" patch
    if (x)
    {
    a;
    b;
    c;
    }
    else
        d;



More information about the mythtv-dev mailing list