Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: domain of thomas.neidhart@gmail.com
 designates 74.125.82.51 as permitted sender)
Message-ID: <512D0B58.1020403@gmail.com>
Date: Tue, 26 Feb 2013 20:22:00 +0100
From: Thomas Neidhart <thomas.neidhart@gmail.com>
User-Agent: Mozilla/5.0 (X11; Linux i686;
 rv:17.0) Gecko/20130106 Thunderbird/17.0.2
MIME-Version: 1.0
To: Commons Developers List <dev@commons.apache.org>
Subject: Re: svn commit: r1449929 -
 /commons/proper/collections/trunk/src/main/java/org/apache/commons/collections/bag/AbstractMapBag.java
References: <20130225221822.BC05F2388900@eris.apache.org>
 <CAAqLGLPxfdz=G8YmFJ54Z6m5sxXbuXSQhew+MhzemRGwOY_qeA@mail.gmail.com>
 <512C65B9.7090803@gmail.com>
 <CAAqLGLO1yJ57GGuv1F=LDoZmcO6B-pYnfkKpr3vwaH09SrAw0g@mail.gmail.com>
 <CAOGo0VaPZPaY8XQWAsDx4idG+izcpuUTTRgDnzzmXphUASD9CQ@mail.gmail.com>
 <kgipvm$qpl$1@ger.gmane.org>
 <CAOGo0VbKOCrBb2pyYz7+bGxDYcWWr=-DTsQz7B1pPnYDXm2dTg@mail.gmail.com>
In-Reply-To: 
 <CAOGo0VbKOCrBb2pyYz7+bGxDYcWWr=-DTsQz7B1pPnYDXm2dTg@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit

On 02/26/2013 06:37 PM, sebb wrote:
> On 26 February 2013 17:04, J�rg Schaible <Joerg.Schaible@scalaris.com> wrote:
>> sebb wrote:
>>
>>> On 26 February 2013 10:27, Simone Tripodi <simonetripodi@apache.org>
>>> wrote:
>>>>> it is not safe, and it will throw an ArrayStoreException in this case,
>>>>> which is documented in the throws clause.
>>>>>
>>>>
>>>> OK I just read the commit, unless it is documented it is fine for me
>>>
>>> Should still be documented on the @SuppressWarnings line please, but
>>> can refer to Javadoc.
>>
>> In that case I typically prefer the suppression as close as possible, i.e.:
>>
>> =========== %< =========
>>  if (array.length < size)
>>  {
>>    @SuppressWarnings("unchecked")
>>    T[] unchecked = Array.newInstance(array.getClass().getComponentType(),
>> size);
>>    array = unchecked;
>>  }
>> =========== %< =========
>>
>> Otherwise you might suppress more than wanted.
> 
> +1

Well, normally I am quite open for such suggestions, but I did this
change after looking at the openjdk code. The change would look like this:

    /**
     * Returns an array of all of this bag's elements.
     * If the input array has more elements than are in the bag,
     * trailing elements will be set to null.
     *
     * @param <T> the type of the array elements
     * @param array the array to populate
     * @return an array of all of this bag's elements
     * @throws ArrayStoreException if the runtime type of the specified
array is not
     *   a supertype of the runtime type of the elements in this list
     * @throws NullPointerException if the specified array is null
     */
    public <T> T[] toArray(T[] array) {
        final int size = size();
        if (array.length < size) {
            @SuppressWarnings("unchecked") // safe as both are of type T
            final T[] unchecked = (T[])
Array.newInstance(array.getClass().getComponentType(), size);
            array = unchecked;
        }

        int i = 0;
        final Iterator<E> it = map.keySet().iterator();
        while (it.hasNext()) {
            final E current = it.next();
            for (int index = getCount(current); index > 0; index--) {
                // unsafe, will throw ArrayStoreException if types are
not compatible, see javadoc
                @SuppressWarnings("unchecked")
                final T unchecked = (T) current;
                array[i++] = unchecked;
            }
        }
        while (i < array.length) {
            array[i++] = null;
        }
        return array;
    }

Any other toArray(T[]) method does not have this level of detail, but at
least this one now.

Thomas

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org