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

changing lower case letters to capital #6

Open
jaraco opened this issue Feb 18, 2014 · 1 comment
Open

changing lower case letters to capital #6

jaraco opened this issue Feb 18, 2014 · 1 comment

Comments

@jaraco
Copy link
Contributor

jaraco commented Feb 18, 2014

Originally reported by: Anonymous


I found that calling the soundex() changes the input string to capital. Even creating a deep copy cannot prevent the change.

My current solution is to create a new string then append the letters of input string to the new string one at a time, then use the new string as the input for soundex().


@jaraco
Copy link
Contributor Author

jaraco commented Jul 2, 2014

Original comment by Rohan Anand (Bitbucket: rohan_anand, GitHub: Unknown):


This fork has got it fixed : https://bitbucket.org/dhellmann/fuzzy
Basically deepcopy doesn't work because strings are immutable in python but since the library internally makes a C call, it gets modified in place by C which should have not happened.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant