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

Re: svn commit: r992041 - in /subversion/trunk/subversion/bindings/javahl: native/CreateJ.cpp src/org/apache/subversion/javahl/CommitItem.java src/org/tigris/subversion/javahl/CommitItem.java tests/org/apache/subversion/javahl/SVNTests.java

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: Tue, 7 Sep 2010 08:42:53 -0500

On Thu, Sep 2, 2010 at 2:46 PM, Mark Phippard <markphip_at_gmail.com> wrote:
> On Thu, Sep 2, 2010 at 3:17 PM, Mark Phippard <markphip_at_gmail.com> wrote:
>> I would not be so sure. When you are dealing with a GUI, adding a
>> handful of milliseconds to something that is going to be done
>> thousands of times can add up to less responsiveness.
>
> I did some micro-benchmarks and the URI class does not perform too
> badly -- although it is clearly slower. I did a loop to construct a
> million objects and see how long it took:
>
> String class: 0.417
> URI class: 3.968
>
> I then took the loop and added a call to a method that took a String
> and just did a couple of simple checks of the String to make sure the
> compiler was not optimizing away any problems. That gives:
>
> String class: 0.455
> URI class: 4.003
>
> That said, in this example, if this were someone retrieving History
> entries that would be 4 more seconds added to the UI wait. Granted it
> might have been a minute or more to get these million entries from the
> server, but it is still added time.

While I don't agree with your reasoning, I've reverted r992041 in r993356.

-Hyrum
Received on 2010-09-07 15:43:38 CEST

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