Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Str.split is not tail recursive #4100

Closed
vicuna opened this issue Sep 4, 2006 · 1 comment
Closed

Str.split is not tail recursive #4100

vicuna opened this issue Sep 4, 2006 · 1 comment
Labels
feature-wish otherlibs

Comments

@vicuna
Copy link

vicuna commented Sep 4, 2006

Original bug ID: 4100
Reporter: pvernaza
Status: acknowledged (set by @damiendoligez on 2006-09-11T09:20:58Z)
Resolution: open
Priority: normal
Severity: feature
Version: 3.09.2
Category: otherlibs

Bug description

It would be useful if Str.split were documented as not being tail recursive, since it can lead to mysterious failures when parsing large strings. Furthermore, perhaps a tail-recursive Str.rev_split could be added?
@xavierleroy
Copy link
Contributor

This seems to have been fixed in commit 7e9f606 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-wish otherlibs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xavierleroy @vicuna