[THRIFT-1255] Mismatch of method name between JavaME's lib and generated code (compareTo/compareObjects)

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.6.1
• Fix Version/s: 0.7
• Component/s: JavaME - Compiler, JavaME - Library
• Patch Info: Patch Available
• Assignee: Raphaël B.
• Reporter: Raphaël B.
• Created: 02/Aug/11 09:01
• Updated: 02/Aug/11 23:42
• Resolved: 02/Aug/11 22:37

Description

Generated code always calls TBaseHelper.compareTo in implementations of compareTo for the DTO.

In javame's lib, there is a static utility method called TBaseHelper.compareTo for anything (int, short, byte, ...) but Object, which has TBaseHelper.compareObjects instead.

When a structure's field is a structure itself into thrift's contract, the generated code tries to call TBaseHelper.compareTo instead of TBaseHelper.compareObjects.

It's much easier to fix this in the library itself by making names uniform.

Attached patch fixes this issue by renaming compareObjects into compareTo.