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. Paolo > 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.