On 10/10/2016 23:29, Jianjun Duan wrote:
> You are right. What we have in VMSTATE now cannot handle a recursive (or
> cyclic as you call it) structure. The idea was to use VMS_LINKED to
> indicate a recursive structure.
Sure, but it's unnecessary. If you didn't have VMS_LINKED, no one would
notice the difference.
Yes, existing VMStateInfos didn't need the new arguments, the new one
does. So you add the new arguments. There's no need to add the flag.
> In this patch is used on a queue. It can
> also be used on list or trees.
> In this regard, VMS_LINKED does represent something.