https://issues.apache.org/bugzilla/show_bug.cgi?id=50006
Helder Magalhães <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #26087|0 |1 is obsolete| | --- Comment #5 from Helder Magalhães <[email protected]> 2010-09-30 01:40:47 EDT --- Created an attachment (id=26103) --> (https://issues.apache.org/bugzilla/attachment.cgi?id=26103) Performance improved (In reply to comment #4) > > There's probably a better way to map the glyph ID back to the corresponding > > codepoint but I was unable to find it. > > The code sounds fine, I'm just thinking a bit about performance. I had "forgotten" about hash set [1], which seems a better fit [1] [2]: "This class offers constant time performance for the basic operations (add, remove, contains and size)[...]" [1] I've also reworked the patch so it includes the full source path - otherwise, one needs to already know where the patch target is. ;-) If there's nothing against, I'll make some tests (I don't have a test environment handy right now) and commit as-is within the next couple of days. :-) [1] http://download.oracle.com/javase/6/docs/api/java/util/HashSet.html [2] http://www.jusfortechies.com/core-java/inside-set.html -- Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
