search

yougov / fuzzy

MIT License
50 stars 11 forks source link

soundex breaks deepcopy somehow #4

Open jaraco opened 12 years ago

jaraco commented 12 years ago

Originally reported by: Anonymous

running Soundex on a string changes the original string to uppercase. That's all well and good, but interestingly, it also changes a deep copy of the original string! that seems pretty wrong...

#!python

>>>x = "blabla"
>>>y = copy.deepcopy(x)
>>>sndex = fuzzy.soundex(32)
>>>print sndex(x)
B4140000000000000000000000000000
>>>print x
BLABLA
>>>print y
BLABLA
>>>#running soundex on x changes deep copy y!
jaraco commented 10 years ago

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.