[issue32684] asyncio.gather(..., return_exceptions=True) swallows cancellation

R. David Murray added the comment: See also issue 26923, but I think this problem is different. -- nosy: +r.david.murray ___ Python tracker

Re: Please Help

On 01/26/2018 08:33 PM, mohammedfaraz...@gmail.com wrote: import numpy as np x=np.unit8([250) print(x) y=np.unit8([10]) print(y) z=x+y print(z) output [250] [10] [4] My question how is z [4] Despite all the typos in your post, you appear to be doing 8 bit unsigned arithmetic.  Do you

[issue26219] implement per-opcode cache in ceval

Nick Coghlan added the comment: With the 3.7 beta deadline just around the corner, 3.8 will be the next opportunity to reconsider this idea. -- versions: +Python 3.8 -Python 3.6 ___ Python tracker

[issue32668] deepcopy() fails on ArgumentParser instances

Terry J. Reedy added the comment: Hi, you seem to be in luck, regardless of version. Python 3.7.0a4+ (heads/master:a49ac99029, Jan 27 2018, 00:11:50) [MSC v.1900 32 bit (Intel)] on win32 Type "help", "copyright", "credits" or "license" for more information. >>> import

[issue32684] asyncio.gather(..., return_exceptions=True) swallows cancellation

New submission from Марк Коренберг : Proof: import asyncio async def test(): while True: print('sleeping 1') await asyncio.gather(asyncio.sleep(1), return_exceptions=True) async def amain(): print('creating task') qwe =

[issue32677] Add.isascii() to str, bytes and bytearray

Change by INADA Naoki : -- pull_requests: +5202 ___ Python tracker ___ ___

Re: Please Help

Please copy and paste the exact code you are running. The code you show has several syntax errors, and would not execute at all. Now, I think that I can read through your errors anyway, so let me ask you a question: what is the largest possible value that can be represented with an unsigned

[issue32666] Valgrind documentation seems to need updating

Terry J. Reedy added the comment: Christian or Brett, do either of you know who maintains the Valgrind stuff? There is no listing in the Expert's index. -- nosy: +brett.cannon, christian.heimes, terry.reedy ___ Python tracker

[issue32645] test_asyncio: TLS tests fail on "x86 Windows7" buildbot

Terry J. Reedy added the comment: I ran f:\dev\3x>python -m test -j5 test_asyncio test_asyncio test_asyncio test_asyncio test_asyncio on fresh Win10 build and got ... 0:01:02 [5/5] test_asyncio passed (62 sec) F:\dev\3x\lib\asyncio\selector_events.py:648: ResourceWarning:

[issue32644] unittest.mock.call len() error

Terry J. Reedy added the comment: Please either upload or include in a post a minimal testcase that reproduces the problem. We can neither debug the problem or know if we fix it without one. -- nosy: +michael.foord, terry.reedy

[issue32677] Add.isascii() to str, bytes and bytearray

INADA Naoki added the comment: New changeset a49ac9902903a798fab4970ccf563c531199c3f8 by INADA Naoki in branch 'master': bpo-32677: Add .isascii() to str, bytes and bytearray (GH-5342) https://github.com/python/cpython/commit/a49ac9902903a798fab4970ccf563c531199c3f8

[issue32664] Connector "|" missing between ImportError and LookupError

Terry J. Reedy added the comment: PR-1173 fixed this for 3.7, but it was not backported then. I added a comment there. -- keywords: -patch stage: patch review -> backport needed versions: -Python 3.7 ___ Python tracker

[issue32664] Connector "|" missing between ImportError and LookupError

Change by KINEBUCHI Tomohiko : -- keywords: +patch pull_requests: +5201 stage: needs patch -> patch review ___ Python tracker ___

[issue32664] Connector "|" missing between ImportError and LookupError

Terry J. Reedy added the comment: Agreed, like there is a bar between NameError and OSError. The 2.7 version does not have the problem. -- nosy: +terry.reedy ___ Python tracker

Please Help

import numpy as np x=np.unit8([250) print(x) y=np.unit8([10]) print(y) z=x+y print(z) output [250] [10] [4] My question how is z [4] -- https://mail.python.org/mailman/listinfo/python-list

[issue32513] dataclasses: make it easier to use user-supplied special methods

Nick Coghlan added the comment: Eric's current proposal sounds sensible to me, but I'll note that if we deem it necessary, the code that implicitly sets `__hash__ = None` to override object.__hash__ when __eq__ is defined could also set some other marker to make it more

[issue27505] Missing documentation for setting module __class__ attribute

Nick Coghlan added the comment: Thanks for the PRs! -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue27505] Missing documentation for setting module __class__ attribute

Nick Coghlan added the comment: New changeset 8f68cb7db37322cfeeb8338e78474e5f25d930c5 by Nick Coghlan (Cheryl Sabella) in branch '3.6': [3.6] bpo-27505: Retrofit module __class__ documentation from 3.7 (GH-5321)

[issue27505] Missing documentation for setting module __class__ attribute

Nick Coghlan added the comment: New changeset 85527cf50a9a4eecaca4022f7c6cb9e0bae1fa5f by Nick Coghlan (Cheryl Sabella) in branch 'master': bpo-27505: Add change notes in module attribute docs (GH-5320)

[issue32304] Upload failed (400): Digests do not match on .tar.gz ending with x0d binary code

Éric Araujo added the comment: New changeset 995c60d2656c022359aac3fe713d8464c8db5716 by Éric Araujo (Bo Bayles) in branch '3.6': [3.6] bpo-32304: Fix distutils upload for tar files ending with b'\r' (GH-5264) (GH-5330)

[issue31368] Add os.preadv() and os.pwritev()

STINNER Victor added the comment: FYI in 2013 I proposed to use writev() in the io module, but it was inefficient: bpo-17655. -- ___ Python tracker

[issue32513] dataclasses: make it easier to use user-supplied special methods

Eric V. Smith added the comment: I apologize for the length of this, but I want to be as precise as possible. I've no doubt made some mistakes, so corrections and discussion are welcomed. I'm adding the commented text at the end of this message to dataclasses.py. I'm

[issue31368] Add os.preadv() and os.pwritev()

Change by STINNER Victor : -- title: Support asynchronous, non-blocking buffered reads (RWF_NONBLOCK) -> Add os.preadv() and os.pwritev() ___ Python tracker

[issue31368] Support asynchronous, non-blocking buffered reads (RWF_NONBLOCK)

STINNER Victor added the comment: Antoine Pitrou: "In any case, I think this would take the form of new low-level calls in the posix module, i.e. add thin wrappers around the new system / libc calls." "new system / libc calls"... well, preadv() is available on

[issue31368] Support asynchronous, non-blocking buffered reads (RWF_NONBLOCK)

STINNER Victor added the comment: I set the priority to release blocker to make sure that I don't forget to merge the PR before Python 3.7b1 (next monday?). The PR is *almost* ready, remaining issues are mostly related to documentation and coding style. --

[issue32668] deepcopy() fails on ArgumentParser instances

Nitish added the comment: This seems to have been fixed in the latest master. Commit I tested: 37420deb80dcf0fc41a728838b0340b93ca01d90 -- nosy: +nitishch ___ Python tracker

[issue32678] Lazy import ast in inspect

Change by INADA Naoki : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue32678] Lazy import ast in inspect

INADA Naoki added the comment: New changeset 37420deb80dcf0fc41a728838b0340b93ca01d90 by INADA Naoki in branch 'master': bpo-32678: inspect: Import ast lazily (GH-5344) https://github.com/python/cpython/commit/37420deb80dcf0fc41a728838b0340b93ca01d90 --

[issue32626] Subscript unpacking raises SyntaxError

Ben Burrill added the comment: Yeah, but in this case, you don't need parentheses unless you use unpacking. That is unexpected behavior. -- ___ Python tracker

[issue32626] Subscript unpacking raises SyntaxError

Terry J. Reedy added the comment: Tuples often need parentheses when embedded in expressions. So often that beginners often think that they are required. I believe this should be closed as 'not a bug'. I nosied the two authors of the implementation for their opinion.

[issue32683] isinstance is calling ob.__getattribute__ as a fallback instead of object.__class__.__get__

Steven D'Aprano added the comment: Dan, I don't understand what you think the code snippet shows: you're calling isclass on an object which *actually is a class* and show that it returns True. What did you expect it to return? How does the code snippet you give

[issue32621] Problem of consistence in collection.abc documentation

Change by Terry J. Reedy : -- assignee: -> docs@python components: +Documentation, Library (Lib) nosy: +docs@python stage: -> needs patch ___ Python tracker

[issue32621] Problem of consistence in collection.abc documentation

Change by Terry J. Reedy : -- nosy: +rhettinger ___ Python tracker ___ ___

[issue32616] Significant performance problems with Python 2.7 built with clang 3.x or 4.x

Change by Terry J. Reedy : -- components: +macOS nosy: +ned.deily, ronaldoussoren ___ Python tracker ___

[issue32615] Inconsistent behavior if globals is a dict subclass

Terry J. Reedy added the comment: I was referring back to your original a) and b) items. Perhaps 'accidental' versus 'intentional, for whatever reason', would be the better contrast. The weekly new issue report comes out on Fridays around noon eastern US. I hope someone

[issue32630] Migrate decimal to use PEP 567 context variables

STINNER Victor added the comment: > Likewise, on the same builds, running _decimal/tests/bench.py does not show a > significant difference: > https://gist.github.com/elprans/fb31510ee28a3aa091aee3f42fe65e00 Note: it may be interesting to rewrite this benchmark my

[issue32630] Migrate decimal to use PEP 567 context variables

STINNER Victor added the comment: Since the root of the discussion is a performance regression, let me take a look since I also care of not regressing in term of performance. We (CPython core developers, as as team) spent a lot of time on optimizing CPython to make

[issue32615] Inconsistent behavior if globals is a dict subclass

Change by pmpp : -- nosy: +pmpp ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue32615] Inconsistent behavior if globals is a dict subclass

ppperry added the comment: Uh, I'm not undertsanding the relevance of whether the code is run at module-level or not. It can't possibly be a feature that * some code * uses the overrides and *other code* doesn't. -- ___ Python

[issue32545] Unable to install Python 3.7.0a4 on Windows 10 - Error 0x80070643: Failed to install MSI package.

Terry J. Reedy added the comment: Can one of the Windows experts look at the attached log? -- ___ Python tracker ___

[issue32630] Migrate decimal to use PEP 567 context variables

Guido van Rossum added the comment: Apologies accepted. I did not imply that -- I was simply stating that Yury needed your help reproducing your result so he could do something about it. It seems you two are taking this offline so I trust that there will be no more barbs.

[issue15115] Duplicated Content-Transfer-Encoding header when applying email.encoders

Cheryl Sabella added the comment: Hi David, I've made a pull request for the way I think you wanted this documented. Please take a look and let me know if it's even close to what you were suggesting. Thanks! :-) -- stage: patch review -> needs patch

[issue32630] Migrate decimal to use PEP 567 context variables

Elvis Pranskevichus added the comment: Likewise, on the same builds, running _decimal/tests/bench.py does not show a significant difference: https://gist.github.com/elprans/fb31510ee28a3aa091aee3f42fe65e00 -- ___ Python tracker

[issue15115] Duplicated Content-Transfer-Encoding header when applying email.encoders

Change by Cheryl Sabella : -- keywords: +patch pull_requests: +5200 stage: needs patch -> patch review ___ Python tracker ___

[issue32521] NIS module fails to build due to the removal of interfaces related to Sun RPC from glibc.

Change by Christian Heimes : -- pull_requests: +5199 ___ Python tracker ___ ___

[issue32683] isinstance is calling ob.__getattribute__ as a fallback instead of object.__class__.__get__

R. David Murray added the comment: I don't think this is a bug. There are many ways to lie in Python. If your object lies, it is on your head when things break :) On the flip side, the ability to lie is very handy in many circumstances, and is often a case of duck

[issue32521] NIS module fails to build due to the removal of interfaces related to Sun RPC from glibc.

Change by Christian Heimes : -- pull_requests: +5198 ___ Python tracker ___ ___

[issue32521] NIS module fails to build due to the removal of interfaces related to Sun RPC from glibc.

Christian Heimes added the comment: New changeset 29a7df78277447cf6b898dfa0b1b42f8da7abc0c by Christian Heimes in branch 'master': bpo-32521: nis libnsl (#5190) https://github.com/python/cpython/commit/29a7df78277447cf6b898dfa0b1b42f8da7abc0c --

[issue32436] Implement PEP 567

Yury Selivanov added the comment: New changeset 226e50049da43097d89e9e99f9c55f212d1f7bd5 by Yury Selivanov in branch 'master': bpo-32436: Make PyContextVar_Get a little bit faster (#5350) https://github.com/python/cpython/commit/226e50049da43097d89e9e99f9c55f212d1f7bd5

[issue32630] Migrate decimal to use PEP 567 context variables

Elvis Pranskevichus added the comment: FWIW, I ran bm_telco with pyperformance on a benchmark-tuned system and did not observe the slowdown. Benchmarks were done on a release build (--enable-optimizations) $ sudo (which python3) -m perf system tune MASTER: $ pyperformance

[issue32615] Inconsistent behavior if globals is a dict subclass

Terry J. Reedy added the comment: I did take the code, and it would not run because of the indent. With that fixed, it still did not run because the the indent in the quoted code fed to exec. This does run: import builtins class K(dict): def __getitem__(self, k):

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: Guido, I apologize for the outburst. I had the impression that msg310799 implicitly asserted my incompetence in benchmarking. -- ___ Python tracker

[issue32446] ResourceLoader.get_data() should accept a PathLike

Barry A. Warsaw added the comment: Hi Jay. Thanks for your interest in contributing to Python! Actually, I'm going to close this issue since we ultimately decided to deprecate the ResourceLoader ABC:

[issue32675] dict.__contains__(unhashable) raises TypeError where False was expected

Change by R. David Murray : -- resolution: -> not a bug stage: -> resolved status: open -> closed ___ Python tracker ___

[issue32614] Fix documentation examples of using re with escape sequences

Terry J. Reedy added the comment: Definitely. x.y code, including doc examples, should free of such warnings. -- nosy: +terry.reedy ___ Python tracker

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: Yury, would you be willing to work this out by email? -- I think it was you who I discussed the context-subclassing with and that was quite a pleasant experience. -- ___ Python tracker

[issue32630] Migrate decimal to use PEP 567 context variables

Guido van Rossum added the comment: Stefan this is unacceptable abuse. Please read the code of conduct. -- ___ Python tracker ___

[issue32446] ResourceLoader.get_data() should accept a PathLike

Jay Yin added the comment: I hope this isn't a dumb question but where is ResourceLoader.get_data() implemented, using ATOM to search the whole project and I can't seem to find it. -- ___ Python tracker

[issue32630] Migrate decimal to use PEP 567 context variables

Yury Selivanov added the comment: Sorry Stefan, I never wanted this to look like "I'm pushing this without listening to Stefan". I apologize if it looked that way. I ran bm_telco on my machine before submitting the PR, and I indeed did not see any performance impact.

[issue32446] ResourceLoader.get_data() should accept a PathLike

Jay Yin added the comment: nvm I temporarily forgot I was working with python here, I'd need to use "is" to check what the entered "path" is huh? -- ___ Python tracker

[issue32446] ResourceLoader.get_data() should accept a PathLike

Jay Yin added the comment: this would extend to set and get data right?, also would it be good to use the current get_data() (one that reads only string) function for the overloaded one and extract the string path from the os.path object? or would we want the os.path

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: I have run about 1000 times more decimal benchmarks than both Yury and you. You attempt to hurt my reputation is laughable. Show me some top-performance code that you have written. -- ___ Python

[issue32630] Migrate decimal to use PEP 567 context variables

Guido van Rossum added the comment: Stefan, I don't think a module author should retain veto over everything affecting their code forever. (We've had spectacular process failures with this in the past.) Please take a deep breath and patiently answer Yury's questions. If you

[issue32446] ResourceLoader.get_data() should accept a PathLike

Jay Yin added the comment: I'm taking a look now, this looks interesting -- nosy: +jayyin11043 ___ Python tracker ___

[issue32436] Implement PEP 567

Change by Yury Selivanov : -- pull_requests: +5197 ___ Python tracker ___ ___

[issue32613] Use PEP 397 py launcher in windows faq

Terry J. Reedy added the comment: I am inclined to believe. The answer to "How do I run a Python program under Windows?" should include how to use 'py'. 'python' will not work unless the install directory has been added to the path. Of course, 'py' does not work if its

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: Guido, I have the feeling that the feature -- about which I was actually positive in the first place -- is being pushed aggressively with no respect for the module author. BTW, prec is changed quite frequently in decimal code, so if people

[issue32630] Migrate decimal to use PEP 567 context variables

Guido van Rossum added the comment: Guys. Please stop with the editorializing. "I cannot believe ..." (used essentially by both of you) is not constructive. -- ___ Python tracker

[issue32613] Use PEP 397 py launcher in windows faq

Change by Terry J. Reedy : -- components: +Windows nosy: +paul.moore, steve.dower, tim.golden, zach.ware ___ Python tracker ___

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: On Fri, Jan 26, 2018 at 09:06:38PM +, Yury Selivanov wrote: > This benchmark is specially constructed to profile creating decimal contexts > and doing almost nothing with the It is not constructed at all. It was the first thing I wrote

[issue32683] isinstance is calling ob.__getattribute__ as a fallback instead of object.__class__.__get__

New submission from Dan Snider : It doesn't even care if the result it gets from ob.__getattribute__("__class__") isn't a type. Surely `isinstance` is intended to do what it is named and be guaranteed to return True if an object `ob` is an instance of class `cls`.

[issue32611] Tkinter taskbar icon (Windows)

Terry J. Reedy added the comment: On Win10, with 3.5.4, 3.6.4, 3.7.0a4, the pinned IDLE icons on my taskbar look normal, with the notebook and snake icon. Running import tkinter as tk r = tk.Tk() stacks the edge of a shadow icon behind the IDLE icon. When I mouse over the

[issue32630] Migrate decimal to use PEP 567 context variables

Yury Selivanov added the comment: > However, I could not find any tests for the added feature (safe > use with async) though. We would be adding a new feature without > tests. This is no problem, I can add a few async/await tests. > I'm getting a large slowdown: >

[issue32630] Migrate decimal to use PEP 567 context variables

Stefan Krah added the comment: Tests - I ran some of my own tests (not even close to all), they seem fine. However, I could not find any tests for the added feature (safe use with async) though. We would be adding a new feature without tests. Performance ---

[issue32670] Enforce PEP 479—StopIteration and generators—in Python 3.7 by default

Change by Yury Selivanov : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue32670] Enforce PEP 479—StopIteration and generators—in Python 3.7 by default

Yury Selivanov added the comment: New changeset 43c47fe09640c579462978ec16f81295f5857cde by Yury Selivanov in branch 'master': bpo-32670: Enforce PEP 479. (#5327) https://github.com/python/cpython/commit/43c47fe09640c579462978ec16f81295f5857cde --

[issue32545] Unable to install Python 3.7.0a4 on Windows 10 - Error 0x80070643: Failed to install MSI package.

Matt Ripley added the comment: Hi Terry, Thanks for getting back to me on this. I tried all of the suggestions on the link that you sent to me and the python installer still will not install for me. I get the same errors. I can install the conda python distributions.

[issue32571] Speed up and clean up getting optional attributes in C code

Yury Selivanov added the comment: Closing this one now. Thanks Serhiy and Inada-san! -- nosy: +yselivanov resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue32662] Implement Server.serve_forever and corresponding APIs

Yury Selivanov added the comment: I think these failures were on the previous version of the test; i've fixed it last night and it shouldn't timeout this way anymore. -- ___ Python tracker

[issue18533] Avoid error from repr() of recursive dictview

Terry J. Reedy added the comment: Thank you for checking and merging. In the future, we will be reminded to change '#' to 'GH-' in the title. I only found out a couple of days ago that this is desired. David, in msg193651, and Raymond, in msg193779, already suggested

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

Yury Selivanov added the comment: > OK. I hope both of you keep an eye on this and actively try to test and break it! That's the plan! -- ___ Python tracker

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

Guido van Rossum added the comment: OK. I hope both of you keep an eye on this and actively try to test and break it! -- ___ Python tracker

[issue18533] Avoid error from repr() of recursive dictview

Change by Roundup Robot : -- pull_requests: +5196 ___ Python tracker ___

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

Change by Yury Selivanov : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

Yury Selivanov added the comment: Nathaniel's latest PR looks good and fixes the crash. FWIW losing some warnings during interpreter shutdown isn't exactly a new problem, therefore I think there's nothing wrong with the new API here. For instance, I've seen asyncio's

[issue32591] Deprecate sys.set_coroutine_wrapper and replace it with more focused API(s)

Yury Selivanov added the comment: New changeset dba976b8a28d6e5daa66ef31a6a7c694a9193f6a by Yury Selivanov (Nathaniel J. Smith) in branch 'master': bpo-32591: fix abort in _PyErr_WarnUnawaitedCoroutine during shutdown (#5337)

[issue32493] UUID Module - FreeBSD build failure

Change by Barry A. Warsaw : -- nosy: +barry ___ Python tracker ___ ___ Python-bugs-list

[issue32579] UUID module fix, uuid1 python module function

Change by Ned Deily : -- nosy: +barry stage: -> patch review ___ Python tracker ___ ___

[issue32206] Run modules with pdb

Jason R. Coombs added the comment: Thanks for creating this functionality. I'm so excited to use it that I've published a backport [https://pypi.org/project/backports.pdb], which seems to work for at least the basic use cases on Python 2.7+. --

[issue32640] Python 2.7 str.join documentation is incorrect

Mariatta Wijaya added the comment: Thanks! -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue32640] Python 2.7 str.join documentation is incorrect

Mariatta Wijaya added the comment: New changeset f5e8f71fe339fe5e47c4d24ac2e5177fa3c02922 by Mariatta in branch '2.7': [2.7] bpo-32640: Clarify the behavior of str.join and unicode object (GH-5333)

Doubt in line_profiler documentation

​I'm having trouble understanding something in the ​documentation of https://github.com/rkern/line_profiler The definition for the time column says - "Time: The total amount of time spent executing the line in the timer's units. In the header information before the tables, you will see a line

[issue25330] Docs for pkgutil.get_data inconsistent with semantics

Paul Ganssle added the comment: I'm not sure if this warrants a separate issue, but I also notice this in the documentation: > If the package cannot be located or loaded, or it uses a loader which does > not support get_data, then None is returned. In particular, the

[issue32677] Add.isascii() to str, bytes and bytearray

Change by INADA Naoki : -- title: Add str.isascii() -> Add.isascii() to str, bytes and bytearray ___ Python tracker ___

[issue3177] Add shutil.open

Change by Matej Cepl : -- nosy: +mcepl ___ Python tracker ___ ___ Python-bugs-list mailing

[issue32206] Run modules with pdb

Change by Jason R. Coombs : -- nosy: +jason.coombs ___ Python tracker ___ ___

[issue32513] dataclasses: make it easier to use user-supplied special methods

Guido van Rossum added the comment: Can you also clarify that this *never* looks at whether the method is implemented in a superclass? And clarify what happens with the __hash__ = None that's automatically created when you define __eq__. Please also repeat the default value

[issue32513] dataclasses: make it easier to use user-supplied special methods

Eric V. Smith added the comment: I'm working on a better formulation of the __hash__ issue, so don't invest much time in this until I post my next message. I'm going to re-organize the various tables to make them easier to read. --

[issue32680] smtplib SMTP instances missing a default sock attribute

Stéphane Wirtel added the comment: You are right, but... if there is an exception in the connect method before the init of self.sock, the sock attribute will not exist and in this case, and only in this case, there will be an exception in the close method. your code will

[issue32681] Fix uninitialized variable in os_dup2_impl

Change by Stéphane Wirtel : -- versions: +Python 3.7 ___ Python tracker ___ ___

  1   2   >