[TRINIDAD-2401] Clean up skinning code to avoid use of obsolete synchronized classes

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.1.0-core
• Assignee: Blake Sullivan
• Reporter: Blake Sullivan
• Created: 17/Jul/13 02:51
• Updated: 14/Nov/13 17:50
• Resolved: 14/Nov/13 17:50

Description

The skinning code currently uses several obsolete implementation class--Vector, Stack, HashTable that are implicitly synchronized. None of this synchronization is necessary. Replaces these classes with their modern equivalents. In addition, much of the code is confusing due to functions performing multiple different tasks simultaneously. Break these up into easy to understand pieces that perform a single task.