Hi,

This very simple patch allows spaces in xdoclet attributes which are parsed as comma separated values. Example:

  @ojb.update-procedure name="myupdateproc" arguments="id, name, address"

Makes life a bit easier. Patch is against OJB_1_0_RELEASE.

Thanks,
Vadim
Index: src/xdoclet/java/src/xdoclet/modules/ojb/CommaListIterator.java
===================================================================
RCS file: 
/home/cvspublic/db-ojb/src/xdoclet/java/src/xdoclet/modules/ojb/CommaListIterator.java,v
retrieving revision 1.4
diff -u -r1.4 CommaListIterator.java
--- src/xdoclet/java/src/xdoclet/modules/ojb/CommaListIterator.java     5 Apr 
2004 00:32:30 -0000       1.4
+++ src/xdoclet/java/src/xdoclet/modules/ojb/CommaListIterator.java     1 Feb 
2005 16:48:34 -0000
@@ -20,7 +20,7 @@
 
 /**
  * Small helper class for dealing with comma-separated string lists.
- * 
+ *
  * @author <a href="mailto:[EMAIL PROTECTED]">Thomas Dudziak ([EMAIL 
PROTECTED])</a>
  */
 public class CommaListIterator implements Iterator
@@ -32,7 +32,7 @@
 
     /**
      * Creates a new string iterator for the given comma-separated list.
-     *  
+     *
      * @param list The list
      */
     public CommaListIterator(String list)
@@ -70,7 +70,7 @@
 
     /**
      * Returns the next string from the list.
-     * 
+     *
      * @return The next string
      */
     public String getNext()
@@ -78,14 +78,14 @@
         String result = _current;
 
         _current = "";
-        return result;
+        return result.trim();
     }
 
     /**
      * Determines whether one of the remaining elements is the given element. 
If it is found
      * then the iterator is moved after the element, otherwise the iterator is 
after the end
      * of the list.
-     *  
+     *
      * @param element The element to search for
      * @return <code>true</code> if the element has been found
      */

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to