annotations: extend source position information for properties

[bmx666]

merge_nodes, defined in livetree.c, uses srcpos_extend only for nodes.

[dgibson]

merge_nodes, defined in livetree.c, uses srcpos_extend only for nodes.

Sorry it's taken me forever to look at this. However, this behaviour is intentional. Because the new property value entirely overwrites the old property value, I didn't think it made sense to include the now obsolete location in the annotations.

[bmx666]

Sorry @dgibson but then for nodes it's look a little bit strange, because you can remove whole node and re-create same node in different dts, BUT obsolete location will be left.

[dgibson]

Sorry @dgibson but then for nodes it's look a little bit strange, because you can remove whole node and re-create same node in different dts, BUT obsolete location will be left.

That's a fair point. We should probably remove the old node srcpos information when we a /delete-node/ directive is processed.

[bmx666]

@dgibson I think after that a fair point, you understood I wanted to trace history path of property/node/label from first dts up to the latest when I asked about #62

[dgibson]

@bmx666, obviously I haven't looked at this in ages. Sorry.

If you still want to pursue this, can you make a case for the behaviour change you'd like to see, then we can go from there.

[bmx666]

Hi @dgibson
Sync with main branch without any conflicts.
Nothing change since my last comment -> #67 (comment)