[THRIFT-1386] The toString function can consume less CPU and tests

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Won't Fix
• Affects Version/s: 0.8
• Component/s: Java - Compiler
• Patch Info: Patch Available
• Assignee: James E. King III
• Reporter: Verdoïa Laurent
• Created: 08/Oct/11 06:29
• Updated: 03/Jan/19 13:17
• Resolved: 03/Jan/19 13:17

Description

The toString member of generates classes don't need:

1. to test the isFirst value.
The compiler allready know it and can handle the last comma himself

2. to handle special case for nullable object.
StrinBuilder is used for create the resulting string.
This append method always append the string "null" for null objects.
But the result of append(char[] data) with null is not specified; who care. We don't use it!