I'm still fairly certain we don't want to allow the caller to supply the SHALLOW flag to our public constructor. This needs to be built by the RevWalk instance itself to ensure the flags are allocated right.