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?

Max.

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to