Great, I like it!

just a minor observation:

> +        for ( int i = 0; i < sortedEdge.size(); i++ )
>         {
> -            WE we = iterator.next();
> -            iterator.remove();
> +            WE we = sortedEdge.poll();
>

do we really need to cycle guarding an accumulator? I'd replace it with:

while ( !sortedEdge.isEmpty() )
{
    WE we = sortedEdge.poll();
}

best,
-Simo

http://people.apache.org/~simonetripodi/
http://simonetripodi.livejournal.com/
http://twitter.com/simonetripodi
http://www.99soft.org/

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

Reply via email to