[MPlayer-cvslog] CVS: main/libvo vo_macosx.m, 1.3, 1.4 vo_macosx.h, 1.1, 1.2

Jindrich Makovicka makovick at kmlinux.fjfi.cvut.cz
Tue May 3 10:36:53 CEST 2005


Reimar Döffinger wrote:
> Hi,
> On Mon, May 02, 2005 at 08:17:53PM -0400, Nicolas Plourde wrote:
> 
>>On 2-May-05, at 7:41 PM, Alexander Strasser wrote:
>>Common sense should be the rule here. If the patch only change few lines
>>of code and indentation is used to make things clearer, why not?
> 
> 
> Yes, the point is making things clearer. And this makes the patch as it
> appears in the cvslog _a lot_ less clear, since I can't immediately see
> if you only added the if block or also changed the code.

there is always diff -b. the same for cvsweb - it ignores leading
whitespace too, so it is immediately clear that only the if block was added.

-- 
Jindrich Makovicka




More information about the MPlayer-cvslog mailing list