[issue34789] doc xml.sax.make_parser expects a list not just any sequence

Change by Andrés Delfino : -- pull_requests: +8978 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34789] doc xml.sax.make_parser expects a list not just any sequence

Fred L. Drake, Jr. added the comment: The existing PR can be re-targeted to merge to a maintenance branch (I'd be inclined to merge manually, myself, but will have to check the current devguide to make sure that's still allowed). A new PR can be made for the non-documentation fix for

[issue34789] doc xml.sax.make_parser expects a list not just any sequence

Andrés Delfino added the comment: Admittedly, my Git-fu is not strong, but since the PR was opened against master, and given that the PR as is won't be merged to master, can it still be "backported" to 3.7, 3.6, and 2.7? Shouldn't I update the PR so it can be merged to master (that won't be

[issue34789] doc xml.sax.make_parser expects a list not just any sequence

Fred L. Drake, Jr. added the comment: It probably makes more sense to keep that PR for the maintenance branches, and create a new branch / PR to land on master. -- ___ Python tracker

[issue34789] doc xml.sax.make_parser expects a list not just any sequence

Andrés Delfino added the comment: I think it makes sense for make_parser to accept any iterable too. I'll update the PR when I have the testcase ready. -- ___ Python tracker

[issue34789] doc xml.sax.make_parser expects a list not just any sequence

Fred L. Drake, Jr. added the comment: I'm just going to presume this issue has been around a long time, but I think that's a pretty safe presumption. Accepting a general sequence instead of only a list would reasonable, and I'd support a fix that caused the code to accept a general sequence

[issue34789] doc xml.sax.make_parser expects a list not just any sequence

Change by Andrés Delfino : -- keywords: +patch pull_requests: +8944 stage: -> patch review ___ Python tracker ___ ___

[issue34789] doc xml.sax.make_parser expects a list not just any sequence

New submission from Andrés Delfino : Doc says: "If parser_list is provided, it must be a *sequence* of strings which name modules that have a function named create_parser()" but code concatenas parser_list with an existing list: for parser_name in parser_list + default_parser_list: Fix this