Committed the patch then.

2013/8/28 Vinay Agrawal <[email protected]>:
> +1
>
>
> On Wed, Aug 28, 2013 at 2:41 PM, Kasper Sørensen <
> [email protected]> wrote:
>
>> Hi all,
>>
>> I was debugging some code and ran into a legacy thing in the
>> DefaultRow where we have a deprecated field, "_items". The role of
>> this field has been taken over by the "_header" field which has
>> another type etc., but for backwards compatible deserialization
>> purposes the old "_items" field is still there.
>>
>> So I did a bit of investigation on how to get rid of the field
>> entirely and simply do some conversion on the fly while deserializing
>> legacy objects. And that seems to work nicely actually. I've created a
>> patch and posted it as a Gist here:
>>
>> https://gist.github.com/kaspersorensen/6363895
>>
>> Any feedback is welcome. And if you agree, can we then add this to the
>> code?
>>
>> Kasper
>>

Reply via email to