On 4/15/07, Max Bowsher <[EMAIL PROTECTED]> wrote:
Stephane Nicoll wrote: > On 4/14/07, Max Bowsher <[EMAIL PROTECTED]> wrote: >> >> Too late for this release, but could someone take a look at MRELEASE-137 >> so it gets into the next one? It's a trivial one-liner fix. The attached >> patch is for maven-release-plugin, but the relevant class has since >> moved into maven-release-manager. It's obvious how to forward port the >> single change to the new code. >> > The change does not seem that trivial to me. A potential > IndexOutOfBoundsException might be trhown. Jason is the assignee but > i'll try to have a look.The current code calls: list.get( 0 ) the fix is to change that to: list.get( list.size() - 1 ) Surely the only way the second one could cause an IOOBE is if the list is empty, in which case the current code would also IOOBE?
I'm missing coffee. Forget it :)
Max.
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
