I don't believe so...
> +#define change_parent(p) list_move_tail(&(p)->sibling,&(parent)->children)
On the above line change_parent takes one argument.
> + change_parent(p, p->parent);
Here, it takes two.
Otherwise it looks good, and removes a couple of assignments. Not sure
how useful it is though, is this done a lot?
-- Brad Heilbrun - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/