Christoph Reck wrote:
> 
> Two patches below:
> 1. ResourceManager to return a null template if the file was not
>    found (this is what the comment some lines above states it
>    would do; I tested that this is the correct behaviour -
>    otherwise it would emit wrong error doing template.parse() ).

Ok, I committed this patch.  But staring at it, I got rather confused
about what we are trying to do - should we be returning a null, or a
valid Resource object that has a member to determine it's health?  Or
should we throw an exception?  Runtime.getTemplate() is declared to
throw, even though getResource() doesn't.... Hmmmm

geir


-- 
Geir Magnusson Jr.                               [EMAIL PROTECTED]
Velocity : it's not just a good idea. It should be the law.
http://jakarta.apache.org/velocity

Reply via email to