<snip>
You will see in CVS that Robert's new "named stack" pop method does
throw an EmptyStackException when an empty stack is popped, and I think
that is the correct behaviour. If we do a digester 2.0, I would
recommend that the pop() method throw an exception in this case - unless
I've missed a good reason for the null to be returned. I can't think of
one, though.
IIRC i preferred the version that returned null but was persuaded that throwing an exception was more consistent with the sun collections API and was therefore what users would probably expect. had i check the original API and discovered that the rest of digester returned nulls, i would probably have changed my mind again. so, i suppose that i don't have a strong opinion on what's better :)
- robert
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
