Antoine Pitrou <pit...@free.fr> added the comment:

> sbt <shibt...@gmail.com> added the comment:
> 
> s/_win32/_winapi/g

Overlapped's naming is still lagging behind :-)

Other than that, a comment:

+        def Close(self):
+            if not self.closed:
+                self.closed = True
+                _winapi.CloseHandle(self)

Since Close() can be called at shutdown (through __del__), it should
probably cache its references to globals (because of the unpredictable
order of module cleanup), like this:

+        def Close(self, CloseHandle=_winapi.CloseHandle):
+            if not self.closed:
+                self.closed = True
+                CloseHandle(self)

Otherwise, looks good (I haven't tested).

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue11750>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to