> > add_child(old_node, new_child); > > } > > > > + old_node->srcpos = srcpos_copy_all(new_node_begin_srcpos); > > This doesn't seem right. Replacing the old position with the new > makes sense for indivudal properties where the whole value is also > replaced. But for nodes we really need to track both locations. > > I think the extra complexity here is why I didn't add this tracking > earlier. I've added a next field to be able to make a list of positions. Here, I append the lists, with the new ones at the front. julia -- To unsubscribe from this list: send the line "unsubscribe devicetree-compiler" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
