On Sun, 12 Mar 2000, Kevin A. Burton wrote:

> So two things:
> 
> If you compile the current CVS version of Jetspeed if will fail unless
> you rm the icalendar directory.
>

Sorry, Guys
 
> Why:
> 
> - needs GNU Regexp jar in ./lib... please add this.

Will do.

> - ICalendarErrorResources seems to be out of date.  Most of the code is
> referring to a member that is in this object but it doesn't exist in the
> CVS version.
> 

Yes this would be the parsing code i think in the types package. That code
is in a heavy state of flux. Sorry for the premature commits

> Jeff.  On another note.  Are we using Regexp?  If we have to cool but if
> it is for something small it might be better to write a small method to
> do it.  I will let you make the call.  It is just that we currently have
> a lot of .jar files in CVS.  The other thing is that using regular
> expressions in Java, IMO, removes a lot of it's OO features.  Easy at
> first but takes away from the languages elegance in the long run.  
> 

I was using it strictly for field level validation (in the set methods).
For what it is currently doing i think we could write a parser to do it
itself, but there are some other field level validations that are a little
more complicated. I agree
that expanding use of the regular expressions in general takes away from
the elegance, but I think using a tool specifically designed to match
patterns to check for complex patterns makes sense. However maybe it could
be executed better.

On the elegance, I think what I could do would be to take the Regular
expression calls directly out of the set methods and put them in a field
edit class.

I will address this today and email back the resolution.

Jeff Prickett
[EMAIL PROTECTED]

> Kevin
> -- 
> Kevin A Burton ([EMAIL PROTECTED])
> http://relativity.yi.org
> Message to SUN:  "Open Source Java!"
> "For evil to win is for good men to do nothing."
> 
> 
> --
> --------------------------------------------------------------
> Please read the FAQ! <http://java.apache.org/faq/>
> To subscribe:        [EMAIL PROTECTED]
> To unsubscribe:      [EMAIL PROTECTED]
> Archives and Other:  <http://java.apache.org/main/mail.html>
> Problems?:           [EMAIL PROTECTED]
> 



--
--------------------------------------------------------------
Please read the FAQ! <http://java.apache.org/faq/>
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Archives and Other:  <http://java.apache.org/main/mail.html>
Problems?:           [EMAIL PROTECTED]

Reply via email to