[exia]

@Cloud @Arbiter
Can you push the next() and generationTime() methods up into the Path class? Otherwise I have to do this:

Path path = pathBuilder.build(Players.getLocal(), bank.getArea());
if(path instanceof RegionPath) return ((RegionPath) path).getNext().getPosition().minimap();
else if (path instanceof WebPath)return ((WebPath) path).getNext().getPosition().minimap();

Which seems a bit silly

 

[geashaw]

You missed a space on the third line, won't get accepted!

 

[exia]

You missed a space on the third line, won't get accepted!

Where?

 

[valithor]

Where?

The return after WebPath)

1. else if (path instanceof WebPath)return ((WebPath) path).getNext().getPosition().minimap();

 

[cloud]

I'm not entirely sure if I should add them but I'll consider it.

 

[exia]

I'm not entirely sure if I should add them but I'll consider it.

What's the point of the interface except to abstract away common methods?

 

[cloud]

What's the point of the interface except to abstract away common methods?

I've gone ahead and added getNext to Path, but I haven't added get generation time because it's not always something that's available.