mixed documentation of PDF.close() and Page.flush_cache() in README

[luketudge]

Something is not right about the main README and perhaps needs updating.

The explanation of PDF.close() here mentions clearing the cache for Page objects. But calling PDF.close() does not clear the cache for the individual pages. To do that, we need Page.flush_cache(). This part of the README also suggests that .flush_cache() is deprecated in favor of .close(), but Page objects do not have a .close() method.

Maybe the documentation of PDF and Page has gotten mixed up? Clearer would be to explain here what .close() does for the PDF object, and to add documentation of cache clearing for Page objects to a different section.

[jsvine]

Thanks for flagging this, @luketudge! It does look like things got a bit mixed up over the course of some changes. I'm proposing fixing the issue via this commit: ba58e16

It (re-)adds the missing Page.close() method, has PDF.close() close all pages, and adjusts the documentation. Does this seem right to you? Or still a bit confusing/wrong?

[luketudge]

@jsvine Great! This looks exactly right. Thanks for picking it up so quickly.

[jsvine]

Super! Closing this as fixed via ba58e16