BaseRegionScanner.next should be abstract

As pointed out by Yuhao Bi on a dev list mail, BaseRegionScanner implements next as a recursive call to itself.

All current subclasses of BaseRegionScanner implement next, but as soon as there is an implementation that doesn't do this, it will end up with a stack overflow.

Easy fix is to make this method abstract instead of having a default recursive implementation.