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

Re: extending the blame callback

From: Branko Čibej <brane_at_apache.org>
Date: Thu, 10 Jan 2019 06:58:46 +0100

On 10.01.2019 04:58, Branko Čibej wrote:
> On 07.01.2019 20:57, Stefan Kueng wrote:
>> @@ -758,6 +759,33 @@
>> * will be true if the reason there is no blame information is that the line
>> * was modified locally. In all other cases @a local_change will be false.
>> *
>> + * @note the line is split on LF characters. Clients must be aware of this
>> + * when dealing with different encodings of the file/line.
>> + * Blaming non ASCII/UTF-8 files requires the @a force flag to be set when
>> + * calling the svn_client_blame6 function.
>
> I just noticed that svn_client_blame6 does not, of course, have a
> parameter called 'force'. But it does have a parameter called
> 'ignore_mime_type'.

Also the assertion that "lines are split on LF" turns out to be wrong
and misleading. Line endings are translated first, through
svn_subst_stream_translated(), and this happens regardless of the MIME
type. These parts of the new docstrings should be fixed before the next
release.

-- Brane
Received on 2019-01-10 06:58:57 CET

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.