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

Some words (actually non-words) causes panic #2

Closed
john5000 opened this issue Aug 21, 2015 · 1 comment
Closed

Some words (actually non-words) causes panic #2

john5000 opened this issue Aug 21, 2015 · 1 comment
Assignees

Comments

@john5000
Copy link

Stem("aed") causes an error (a panic).
Seems to be from step1b where rs[len(rs)-2] is used as an index, but len(rs)=1.
From my testing, it only happens with a vowel followed by a word ending that would be stripped, like "aed" or "aing" or "eing", etc.
Of course, those are not really words! But in real life those can show up in text anyway.
Note: despite that, porter2 is an excellent product and is very fast.

@zhenjl
Copy link
Collaborator

zhenjl commented Aug 21, 2015

ack..will look..

@zhenjl zhenjl self-assigned this Aug 21, 2015
@john5000 john5000 changed the title Stem some words (actually non-words) causes panic Some words (actually non-words) causes panic Aug 24, 2015
@zhenjl zhenjl closed this as completed in 56e4718 Aug 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants