dtc
dtc copied to clipboard
dgibson
annotations: extend source position information for properties
merge_nodes, defined in livetree.c, uses srcpos_extend only for nodes.
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.
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.
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.
@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
@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.
Hi @dgibson Sync with main branch without any conflicts. Nothing change since my last comment -> https://github.com/dgibson/dtc/pull/67#issuecomment-1200152301