[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' - v2

From: Øyvind A. Holm <sunny_at_sunbase.org>
Date: 2005-05-04 08:48:42 CEST

On 2005-05-04 08:24:09 Peter N. Lundblad wrote:
> On Wed, 4 May 2005, [utf-8] ?^Xyvind A. Holm wrote:
> > On 2005-05-02 05:09:09 alexander@collab.net wrote:
> > > Version: 2, Patch to add XML output for 'svn blame'
> > [...]
> > - And the actual text line with it's own element from the file is
> > missing in the XML.
> That's deliberate (see the feedback to the first version of the
> patch). In short, we don't know the encodig of the file content, so we
> can't include it in XML (which has to be UTF8, or some or some other
> encoding, but it has to be consistent.) It is easy to get at the
> content with svn cat.

Yes, that's a good reason not to include the actual line text. But what
about including the actual line number instead? XML suggestion:

<?xml version="1.0" encoding="utf-8"?>

- <time> is changed to <date> (svn log --xml uses <date>)
- A line number element is inserted in a <line></line> element. Which
  also could be called <linenum> for readability.
- Indent of the attributes is changed from three to two spaces.

-- 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 08:49:38 2005

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