➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: concurrent.futures.wait calls len() on an possible iterable
Type: behavior Stage: patch review
Components: Library (Lib) Versions: Python 3.10
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: rohitkg98
Priority: normal Keywords: patch

Created on 2020-10-05 02:02 by rohitkg98, last changed 2022-04-11 14:59 by admin.

Pull Requests
URL Status Linked Edit
PR 22555 open rohitkg98, 2020-10-05 02:35
Messages (1)
msg377991 - (view) Author: Kaushal Rohit (rohitkg98) * Date: 2020-10-05 02:02 
`fs` argument could be an iterable, and calling len on it would raise an Exception.

We are converting `fs` into a set anyways for set difference, and that too twice. we can just put an `fs = set(fs)`.

Let me know if we choose to go with that and I will make a PR ASAP.

Thanks.
History
Date User Action Args
2022-04-11 14:59:36adminsetgithub: 86104
2020-10-05 02:35:35rohitkg98setkeywords: + patch
stage: patch review
pull_requests: + pull_request21553
2020-10-05 02:02:57rohitkg98create