Funny, I was just looking at this because I was also bothered by the
fact that many methods throw Exception. I made the changes to remove
Exception but found that the ResourceLoader also throws Exception, and
removing it would break this plugin for existing users (Even though I
doubt it is really used that much). I plan to make the changes, but
It will have to be a 1.7, or 2.0 change.
It looks like there has already been some work that moves Velocity in
the direction of unchecked Exceptions, and removing throwing Exception
would complete it.
On Dec 28, 2008, at 17:29 , Nathan Bubna wrote:
Suspected initial cause:
IO code that throws Exception and unwillingness to catch, wrap
and re-throw
Relevant anecdote:
Velocity has been around longer than "Effective Java" and is still
doing b.c. 1.x releases
Current holdup:
no one has been bothered enough to offer a patch to fix it yet
Possible fixes:
a patch from you? a 2.x version? enough nagging from you and
others to get one of us committers to change it? :)
On Sun, Dec 28, 2008 at 4:13 PM, Steve Cohen
<[email protected]> wrote:
I must admit I am surprised to see that many key methods of
Velocity such as
Velocity.init() and Velocity.mergeTemplate() have a signature
includes that
includes "throws Exception" which is what I would be tempted to
consider a
beginner's mistake. Certainly Joshua Bloch in "Effective Java"
indicates
his distaste for the practice and it's something I always try to
avoid in my
code.
Can someone explain why Velocity does it this way and an effective
strategy
for wrapping this ugliness away someplace?
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
