add TODO to remove readContents() call again if it proves helpful

instead, we should factor out the part of readContents() (probably acquiring
the modification revision or so) and just do that, instead of reading
the contents twice