On 27.03.2010 15:23, Stefan Sperling wrote:
On Fri, Mar 26, 2010 at 07:57:09PM +0100, Stefan Küng wrote:
On 26.03.2010 18:17, Stefan Sperling wrote:
Can you check if the current API matches your requirements?
See subversion/tests/libsvn_client/client-test.c for a trivial example.

Looks very good! Thanks a lot for implementing this.

Great! You're welcome.

Another thought, not sure if it would make sense or not:
would an option to ignore whitespace changes make sense?

Yes, I guess we could easily make the line-matching ignore whitespace.
Matching is currently done using a simple strcmp(), so it's fairly naive.

Please file an enhancement issue, with undefined schedule (if someone
wants to pick this up before 1.7.0, that's fine -- but it's not a
critical feature).

Ignoring whitespace would help dealing with diffs messed up by e.g. gmail.
Though I've seen gmail use some magic UTF-8 sequences for whitespace,
which is really annoying and which a naive approach like ignoring
single-byte characters like ' ', '\t', etc. may not cope with.

Filed as issue 3610:
http://subversion.tigris.org/issues/show_bug.cgi?id=3610

Stefan

--
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net

Reply via email to