[M3devel] Whitespace checkins

[Tony Hosking]

In principle I agree with everything you say for code *as it is  
edited* for meaningful improvements, but removing whitespace just for  
the fun (?) of it makes tracking meaningful changes a little more  
difficult.

On Mar 17, 2008, at 4:35 AM, Jay wrote:

> Tony, Can you use diff -w or diff -b?

Sure, I can, but I have to type an extra "-w".  ;-)

> Can the derivatives be cleaned up?

Derivatives?

> I'll volunteer.
>
> It'd be nice to get rid of
>  whitespace at ends of lines
>  all tabs
>  overly indented "END", like I sometimes see on records.

Other than white-space It's all a matter of style I guess.  Most of  
this code was written in emacs which defaults tabs equivalent to 8  
spaces.  Yes, I know other editors do 4-space tabs.  Modula-3 mode in  
emacs indents using spaces instead of tabs.

> The Modula-3 codebase isn't so large as to make this infeasible,
> even if that extends to the PM3 copy.

True.

> My editor uses 4 space tabs and a fair amount of code with tabs
> assumes 8. It'd be nice just to get rid of them.

Agreed.

> ok either way. A lot stuff bugs me and I /often/ but not always  
> ignore it..
>
>  - Jay
>
> From: hosking at cs.purdue.edu
> To: jayk123 at hotmail.com
> Subject: Whitespace checkins
> Date: Mon, 17 Mar 2008 03:43:09 -0400
> CC: m3devel at elegosoft.com
>
> Jay,
>
> I notice you just checked in a slew of whitespace removal.   I hate  
> extra whitespace as much as anyone, but these sorts of checkins play  
> havoc with those of us who track changes via diff (and  
> derivatives).  Please restrain your whitespace editing to files  
> where you make meaningful changes.
>
> Thanks,
>
> Tony
>
> Antony Hosking | Associate Professor | Computer Science | Purdue  
> University
> 305 N. University Street | West Lafayette | IN 47907 | USA
> Office +1 765 494 6001 | Mobile +1 765 427 5484
>
>
>
>
> Shed those extra pounds with MSN and The Biggest Loser! Learn more.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://m3lists.elegosoft.com/pipermail/m3devel/attachments/20080317/a2ede02c/attachment-0002.html>