This is a bit too magic for our style.  It works because the statement is actually pulled into the constructor after super(repo) is called, and thus repo in the super class is already defined.  But its funny to read it like that.

I would prefer to do this inside of the constructor, similar to now the RevWalk is allocated inside of the constructor.