David Gibson

Results 166 comments of David Gibson

On Mon, Jul 27, 2015 at 08:06:08AM -0700, Maxim Zakharov wrote: > It is indeed a heapsort sorting, though not a classical > implementation, and it can be dubbed just...

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

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

Fix looks good, but I'll need a Signed-off-by line as described in the new `CONTRIBUTING.md` file.

I'm afraid I'm really not sure what you're asking for.

Sorry it's taken me so long to respond. I'd actually completely forgotten about the annotation feature, which is why I was confused. Reviewing properly now.

Ok, let me know when you have somehting ready for review.

If you need it, you need to describe its sematics, and how it differs from `deleted`.

Thanks for the report. Sorry it's taken me so long to look at it. Certainly looks like a bug, I've made an automated test case and I'm investigating now.

Ok, I've applied a fix, it would be great if you can check it.