Mattias Gaertner wrote:
On Tue, 4 Oct 2005 09:51:19 +0200 (CEST)
[EMAIL PROTECTED] wrote:


[...]
- improved indenting


Beware. There are only a very few indenting styles that most people find
easier to read.
What you believe to be easier to read, is harder for other people. For example: left aligned versus column aligned:

-- snip --
property LastLineIsProgress: boolean Read FLastLineIsProgress
  Write SetLastLineIsProgress;
property Message: string Read GetMessage;

-- snap --
property LastLineIsProgress: boolean Read FLastLineIsProgress
                                     Write SetLastLineIsProgress;
property Message: string Read GetMessage;


It's nice, that you want to make the code more 'readable'. But IMO it's
better to add comments and rename identifiers, than to apply your personal
style to the code.

A related note: I read somewhere as a 'best practice for svn', that you should commit code changes seperate from formatting changes: it makes looking back at the revisions easier, no need to search actual changes between the formatting changes.

I think this would be a good practice to follow.

Vincent.

_________________________________________________________________
    To unsubscribe: mail [EMAIL PROTECTED] with
               "unsubscribe" as the Subject
  archives at http://www.lazarus.freepascal.org/mailarchives

Reply via email to