Re: svn commit: r427541 - /struts/sandbox/trunk/tiles/tiles-core/src/main/java/org/apache/tiles/definition/ComponentDefinitionsImpl.java

2006-08-01 Thread Greg Reddin
On Aug 1, 2006, at 6:39 AM, [EMAIL PROTECTED] wrote: Object attrValue = attr.getValue(); if (attrValue instanceof ComponentDefinition) { retValue = (ComponentDefinition) attrValue; -} else { // It must be a string +} else if (attrValue instanceof

Re: svn commit: r427541 - /struts/sandbox/trunk/tiles/tiles-core/src/main/java/org/apache/tiles/definition/ComponentDefinitionsImpl.java

2006-08-01 Thread Antonio Petrelli
Greg Reddin ha scritto: Thanks for doing this :-) It's nice to see someone else committing. My pleasure :-) -} else { // It must be a string +} else if (attrValue instanceof String) { In the above code do you think it's ok to return null in the unlikely event attrValue is