[Svnmerge] [PATCH] Indented log patch modified v2

[Alan Barrett]

On Tue, 28 Feb 2006, Raman Gupta wrote:
> Attached is a modified version of Alan Barrett's patch for indented log
> entries for the commit log message. It takes into account some feedback
> from Giovanni related to performance, and also fixes the problem brought
> up by Alan himself regarding extra lines.
> 
> Note that I have removed the logic for making the log separator longer
> depending on the longest previous separator. I find this is unnecessary
> when indenting is used.

Thank you for doing this.  However, I don't like indenting the
"........" separator lines themselves (just the nested log messages
between the separators), and I rather like the behaviour of making the
separator longer on subsequent merges.

I like the log to look like this:

[[[
Merged revision 123 from test branch to production branch

................
  r123 | author | date

  Merged revisions 99-100 from development branch to test branch

  ........
    r99 | author | date

    Fixed a bug
  ........
    r100 | author | date

    Fixed typo in previous
  ........
................
]]]

whereas your patch would produce logs like this:

[[[
Merged revision 123 from test branch to production branch

  ........
  r123 | author | date

  Merged revisions 99-100 from development branch to test branch

    ........
    r99 | author | date

    Fixed a bug
    ........
    r100 | author | date

    Fixed typo in previous
    ........
  ........
]]]

and the existing behaviour is like this:

[[[
Merged revision 123 from test branch to production branch

................
r123 | author | date

Merged revisions 99-100 from development branch to test branch

........
r99 | author | date

Fixed a bug
........
r100 | author | date

Fixed typo in previous
........
................
]]]

--apb (Alan Barrett)