right, ok, it's partly the fact that the method is called 'getX' and is a void..
It wouldn't jar so much if it was passing the tree in and it was called something like: void populate(..tree..)