Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(672)

Issue 32042: Implement Uri.resolve(Uri) [#2] (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
16 years, 10 months ago by johnfargo
Modified:
16 years, 6 months ago
Reviewers:
shindig.remailer, plindner, awiner
Base URL:
https://svn.apache.org/repos/asf/incubator/shindig/trunk/
Visibility:
Public.

Description

Continuation of 27117. * New UriTests added for "." and ".." cases. Sussed out a bug. * StringTokenizer used rather than String.split(...) - yielded additional ~20% performance improvement. * LinkedList used over Stack. * ArrayList not used in place of LinkedList due to slight (almost 0 though) performance decrease measured.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+117 lines, -2 lines) Patch
java/common/src/main/java/org/apache/shindig/common/uri/Uri.java View 4 chunks +101 lines, -2 lines 0 comments Download
java/common/src/test/java/org/apache/shindig/common/uri/UriTest.java View 1 chunk +16 lines, -0 lines 0 comments Download

Messages

Total messages: 1
johnfargo
16 years, 10 months ago (2009-03-26 20:50:13 UTC) #1

          
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b