[issue34580] sqlite doc: clarify the scope of the context manager

2018-09-10 Thread Daniel Jakots
Daniel Jakots added the comment: LGTM -- ___ Python tracker <https://bugs.python.org/issue34580> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34580] sqlite doc: clarify the scope of the context manager

2018-09-08 Thread Daniel Jakots
Daniel Jakots added the comment: Yes, I would totally support this move as it would show users that it's important to close it! -- ___ Python tracker <https://bugs.python.org/issue34

[issue34580] sqlite doc: clarify the scope of the context manager

2018-09-08 Thread Daniel Jakots
Daniel Jakots added the comment: Thanks, I lacked the greater picture. Yes, adding a "con.close()" line to the snippet would clarify for me! -- ___ Python tracker <https://bugs.python.o

[issue34580] sqlite doc: clarify the scope of the context manager

2018-09-08 Thread Daniel Jakots
Daniel Jakots added the comment: Thanks for working on this bug! My message focussed on the closing aspect because that was my problem. :) Maybe the wording should be more general like: - "Note that this does not automatically call :meth:`close` on the connection object.&quo

[issue34580] sqlite doc: clarify the scope of the context manager

2018-09-04 Thread Daniel Jakots
New submission from Daniel Jakots : In my experience, the first encounter for beginners with the context manager is with files. The highlighted feature is that you don't need to close the file, 'with' is going to do it for you. The sqlite3 documentation talks about the context manager