[Biojava-dev] [Bug 3140] Required Correction in GenbankLocationParser class

[bugzilla-daemon at portal.open-bio.org]

http://bugzilla.open-bio.org/show_bug.cgi?id=3140


gwaldon at geneinfinity.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




------- Comment #1 from gwaldon at geneinfinity.org  2010-09-29 00:52 EST -------
Applied proposed patch and test case. 

Index: GenbankLocationParser.java
===================================================================
--- GenbankLocationParser.java  (revision 8212)
+++ GenbankLocationParser.java  (working copy)
@@ -133,7 +133,7 @@

     // O beautiful regex, we worship you. (:-)
     // this matches grouped locations
-    private static Pattern gp =
Pattern.compile("^([^\\(\\):]*?:)?(complement|join|order)?\\(*{0,1}(.*?)\\)*{0,1}$");
+    private static Pattern gp =
Pattern.compile("^([^\\(\\):]*?:)?(complement|join|order)?\\({0,1}(.*?\\)*{0,1})$");
     // this matches range locations
     private static Pattern rp =
Pattern.compile("^\\(*(.*?)\\)*(\\.\\.\\(*(.*)\\)*)?$");
     // this matches accession/version pairs


-- 
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.