On Mon, Aug 6, 2012 at 7:25 AM, Peter Zijlstra <pet...@infradead.org> wrote: > On Thu, 2012-08-02 at 15:34 -0700, Michel Lespinasse wrote: >> +struct rb_augment_callbacks { >> + void (*propagate)(struct rb_node *node, struct rb_node *stop); >> + void (*copy)(struct rb_node *old, struct rb_node *new); >> + void (*rotate)(struct rb_node *old, struct rb_node *new); >> +}; > > Should we make that const pointers? Daniel?
I don't think it would hurt, but note that each function taking this as an argument takes it as a const struct rb_augment_callbacks *, so I doubt the extra consts would help either. -- Michel "Walken" Lespinasse A program is never fully debugged until the last user dies. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/