I've just been lurking on the list for a month or so now, so my knowledge of Tapestry internals is next to nil... but aren't variable names that start with an underscore reserved for instance variables?
Regards, Daniel On 4/22/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > > Author: bwallace > Date: Sat Apr 22 01:19:02 2006 > New Revision: 396097 > > URL: http://svn.apache.org/viewcvs?rev=396097&view=rev > Log: > [TAPESTRY-271] Moved calendar from instance to local variable. > > Modified: > > > tapestry/tapestry4/branches/branch-3-0/framework/src/org/apache/tapestry/valid/DateValidator.java > > Modified: > tapestry/tapestry4/branches/branch-3-0/framework/src/org/apache/tapestry/valid/DateValidator.java > URL: > http://svn.apache.org/viewcvs/tapestry/tapestry4/branches/branch-3-0/framework/src/org/apache/tapestry/valid/DateValidator.java?rev=396097&r1=396096&r2=396097&view=diff > > ============================================================================== > --- > tapestry/tapestry4/branches/branch-3-0/framework/src/org/apache/tapestry/valid/DateValidator.java > (original) > +++ > tapestry/tapestry4/branches/branch-3-0/framework/src/org/apache/tapestry/valid/DateValidator.java > Sat Apr 22 01:19:02 2006 > @@ -43,7 +43,6 @@ > private String _displayFormat; > private Date _minimum; > private Date _maximum; > - private Calendar _calendar; > private String _scriptPath = > "/org/apache/tapestry/valid/DateValidator.script"; > > private static DateFormat defaultDateFormat = new > SimpleDateFormat("MM/dd/yyyy"); > @@ -144,8 +143,7 @@ > result = format.parse(value); > } > > - if (_calendar == null) > - _calendar = new GregorianCalendar(); > + Calendar _calendar = new GregorianCalendar(); > > _calendar.setTime(result); > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > >
