Why would it?  This is a leftover from the switch to tempfile+replace semantics from the old (unsafe) truncate and rewrite semantics; there's no need to keep the file open--it's unused after the initial read in the contructor, and it's this that breaks renaming on Windows.  We still hold the lockfile, so this isn't unsafe.  At least, this is how I read the code.