Re: PojoComparator question

2016-05-31 Thread Stephan Ewen
The "compareSerialized" should probably internally always reuse instances, where possible. Since these are never passed into user code or anything, that should be okay to do. On Tue, May 31, 2016 at 11:52 AM, Aljoscha Krettek wrote: > Hi, > I think this is an artifact from

Re: PojoComparator question

2016-05-31 Thread Aljoscha Krettek
Hi, I think this is an artifact from the past. Using the "non-reuse" deserialize seems more correct, especially in the presence of subclasses. Best, Aljoscha On Mon, 30 May 2016 at 19:13 Gábor Horváth wrote: > Hi! > > While I was working on code generation support for

PojoComparator question

2016-05-30 Thread Gábor Horváth
Hi! While I was working on code generation support for PojoComparators, I stumbled upon the compareSerialized method [1]. It first creates two new instances and then it is using the reusing overloads of the serializer. Calling the non-reusing overload would create the instance anyways. Is there a