tinkerpop/pipes — Issue #53: Make LoopBundle a static inner class of LoopPipe
Author: zcox
Created: 2011-10-31 20:32:18+00:00
State: open

Currently, LoopBundle is a non-static inner class of LoopPipe. So every instance of LoopBundle is tied to a parent instance of LoopPipe.

Scala also sees the type LoopBundle as being part of an instance of LoopPipe. In GremlinScalaPipeline, the loop() method needs an argument of type LoopBundle => Boolean for the while function. However I can't use the type LoopBundle unless it is a static inner class of LoopPipe.

I think just making LoopBundle a static inner class will resolve this issue:

```java
public static class LoopBundle {
```

It doesn't look like LoopBundle accesses any fields of LoopPipe, so this change should not cause any issues.