[issue5337] Scanner class in re module undocumented

Raymond Hettinger added the comment: Looking back at the original thread, good reasons were expresses for not documenting this code but just leaving it in as an example. -- dependencies: -Regexp 2.7 (modifications to current re 2.2.2) resolution: -> rejected status: open -> closed __

[issue5337] Scanner class in re module undocumented

Raymond Hettinger added the comment: This is very old code, without a mature API. Will take a look at competing recipes to see if this is still the best way of doing this. Maybe it should return a generator instead of a list. Perhaps there should be some checking for re's that don't combine w

[issue5337] Scanner class in re module undocumented

Changes by Raymond Hettinger : -- assignee: effbot -> rhettinger ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue5337] Scanner class in re module undocumented

Jeroen Ruigrok van der Werven added the comment: OK, clear, then I'll see what I can do. -- ___ Python tracker ___ ___ Python-bugs-lis

[issue5337] Scanner class in re module undocumented

Michael Foord added the comment: Whether it was intended to be exposed or not it is known and used - and therefore we can't change the API without going through the usual deprecation process. As it is used and useful it should be documented. -- ___ P

[issue5337] Scanner class in re module undocumented

Jeroen Ruigrok van der Werven added the comment: So far effbot hasn't said anything about this. So Frederik, was it intended to be exposed or not? -- nosy: +asmodai ___ Python tracker __

[issue5337] Scanner class in re module undocumented

Matthew Barnett added the comment: One of the limitations is that it identifies what matched by using capture groups, so if the expressions provided contain captures then it gets confused! :-) I handled that by 1) rejecting named captures and 2) changing unnamed captures into non-captures. ---

[issue5337] Scanner class in re module undocumented

Raymond Hettinger added the comment: Whether this should be exposed is up to effbot. It's his code. He knows its limitations and he made the original decision to leave it undocumented. -- assignee: benjamin.peterson -> effbot ___ Python tracker

[issue5337] Scanner class in re module undocumented

Georg Brandl added the comment: OK, so we'll wait for that. -- dependencies: +Regexp 2.7 (modifications to current re 2.2.2) ___ Python tracker ___ __

[issue5337] Scanner class in re module undocumented

Matthew Barnett added the comment: FYI, I did tidy up the class and add a 'scaniter' method when I was working on issue #2636; it might yet see the light of day if it gets the go ahead! -- nosy: +mrabarnett ___ Python tracker

[issue5337] Scanner class in re module undocumented

Benjamin Peterson added the comment: I'm sure people are already rely on the intimate details of this class, so why not? -- ___ Python tracker ___ ___

[issue5337] Scanner class in re module undocumented

Georg Brandl added the comment: The class is commented as being "experimental", and the interface probably could use improvement (it's a bit awkward to subclass Scanner), but since it's been around for so long, I guess enough people will be using it that we have to keep it that way and document

[issue5337] Scanner class in re module undocumented

Raymond Hettinger added the comment: Any opinions on whether this was intended to be exposed? -- assignee: georg.brandl -> effbot nosy: +effbot, rhettinger ___ Python tracker ___

[issue5337] Scanner class in re module undocumented

New submission from Michael Foord : There is a useful Scanner class in the re module which is undocumented. See: http://mail.python.org/pipermail/python-dev/2003-April/035075.html http://www.evanfosmark.com/2009/02/sexy-lexing-with-python/ -- assignee: georg.brandl components: Document