[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] 'svn blame --xml' - v3

From: Øyvind A. Holm <sunny_at_sunbase.org>
Date: 2005-05-04 17:00:19 CEST

On 2005-05-04 04:44:37 alexander@collab.net wrote:
> Modified :
> * Now date in GMT format for XML output
> * changed XML element <time> to <date>
> * stripped space padding in the revision= attribute

Ah, nice. For the record, this is how the XML output looks like now:

<?xml version="1.0" encoding="utf-8"?>
<blame>
<target
   path=".bashrc">
<entry
   revision="32">
<author>sunny</author>
<date>1999-07-02T16:36:59.000000Z</date>
</entry>
<entry
   revision="3245">
<author>sunny</author>
<date>2005-02-09T02:34:58.425406Z</date>
</entry>
...
</blame>

I still think a line number element in the <entry></entry> element would
be useful. I don't have a specific example on hand, but there are
several reasons why I think it should be included:

  - We don't want to lose information when using the xml format compared
    to non-xml output. As we can't output text from the file there
    should be some data that replaces it.
  - Including a line number element would make it possible for programs
    to use binary search instead of doing a linear search in a file with
    the XML output.
  - The program does not need the whole XML file to keep track of the
    line number data but can manage with only the relevant part.

I'm sure I come into a situation sometime when I would find the line
number information valuable.

-- sunny256

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed May 4 17:02:06 2005

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.