fuzzy icon indicating copy to clipboard operation
fuzzy copied to clipboard

soundex breaks deepcopy somehow

Open jaraco opened this issue 13 years ago • 1 comments

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!

  • Bitbucket: https://bitbucket.org/yougov/fuzzy/issue/4

jaraco avatar May 01 '12 19:05 jaraco

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.

jaraco avatar Jul 02 '14 12:07 jaraco