[Pogamut-list] Bug in UCCWrapper

[nvh]

Bug in UCCWrapper

Author: nvh

Currently I'm using UCCWrapper to start the server for my experiments, and I found a bug in the shutdown method of the wrapper. At the bottom of the file UCCWrapper.java the following code can be found:
^        // hook that will release all UCC instances when the program finishes
        Runtime.getRuntime().addShutdownHook(new Thread("UCC wrapper finalizer") {
            public void run() {
                synchronized (poolRetrievalLock) {
                    while (!uccPool.isEmpty()) {
                        uccPool.iterator().next().close();
                    }

                    while (!uccUnreleased.isEmpty()) {
                        uccUnreleased.iterator().next().close();
                    }
                }
            }
        });
^
The last while loop is an infinite one, because the close method only removes the wrappers from uccPool and not from the uccUnreleased. The line should be replaced with
^
uccUnreleased.iterator().next().release();
^
that way, the will not get stuck in an infinite loop of trying to close the UCCWrappers from uccUnreleased.