Issue Details (XML | Word | Printable)

Key: CODEC-72
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Unassigned
Reporter: Sebb
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Commons Codec

Soundex and RefinedSoundex issues with character arrays

Created: 06/Jul/08 12:36 PM   Updated: 24/Mar/09 07:19 AM
Return to search
Component/s: None
Affects Version/s: None
Fix Version/s: 1.4

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works soundex.patch 2008-07-06 01:10 PM Sebb 6 kB

Resolution Date: 24/Mar/09 07:19 AM


 Description  « Hide
Soundex and RefinedSoundex both have public US_ENGLISH_MAPPING char arrays. These don't appear to be used externally, so should be made private - or at least package protected.

Also, the class constructors store externally mutable char arrays; it would be better if these were copied.

Patch to follow.



 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
No work has yet been logged on this issue.