Summary: 6 binding +1 (jesse, emmanuel, jdcasey, snicoll, arnaud, vsiveton)
I'll get this moved over and synced out today. thanks! next up, continuum 1.1 alpha 1 On 4/15/07, Stephane Nicoll <[EMAIL PROTECTED]> wrote:
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]
-- jesse mcconnell [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]