Hmm, I liked the idea of just closing the handle to the named temporary file and then using filecmp.

Oh. It just seemed less clean. eg:

• You're now manually in charge of file cleanup, which feels like it should be the context manager's job
• The use of filecmp with a filename when you've already got a handle on the file always felt a bit hacky.
• The file comparison is now both inside and outside of the context manager, which breaks it up visually even though it's the same conceptual operation

Granted, this is a bit more verbose and low-level, but I feel like it flows better.

Can we just remove the file comparison altogether from this sample?

Oh yeah - sure. Apparently this file isn't even referenced in the docs anymore.
Done.