[codec] Using US_ENGLISH static in Soundex causes NPE

• Key: CODEC-10
• Type: Bug
• Status: Closed
• Resolution: Fixed
• Priority: Major
• Assignee: Unassigned
• Reporter: Reggie Riser
• Affects Version/s: 1.3
• Fix Version/s: 1.4

Environment

Operating System: Windows XP
Platform: PC

Description

If I generate a soundex like so:
Soundex.US_ENGLISH.soundex( "Williams" )
I get a NullPointerException.

But if I use the following:
new Soundex().soundex( "Williams" )
everything works fine.

I think the order of initialization of statics may be to blame. I'll attach a
test case.

Comments

[Reggie Riser]

Created an attachment (id=17213)
JUnit test case

[Tim O'Brien]

Yes, you are correct, if you move the static variables around, your test cases
pass. I've added you test case to the mix and changed the order of the static
variables. US_ENGLISH now works.

Thanks

(In reply to comment #0)
> If I generate a soundex like so:
> Soundex.US_ENGLISH.soundex( "Williams" )
> I get a NullPointerException.
>
> But if I use the following:
> new Soundex().soundex( "Williams" )
> everything works fine.
>
> I think the order of initialization of statics may be to blame. I'll attach a
> test case.

[Tim O'Brien]

Commit'd fix for issue 37894, the order of the static variable has been changed.
The static US_ENGLISH is now last, test cases added