Thanks. I think it's a win on the whole too but I'm never sure whether the syntax is too ugly, even though I've done it this way before elsewhere.
On Fri, Jul 27, 2012 at 01:33:59PM -0700, Ethan Jackson wrote: > Awesome, this is a huge win. Thanks a lot. > > Ethan > > On Fri, Jul 20, 2012 at 4:25 PM, Ben Pfaff <b...@nicira.com> wrote: > > Casts are sometimes necessary. One common reason that they are necessary > > is for discarding a "const" qualifier. However, this can impede > > maintenance: if the type of the expression being cast changes, then the > > presence of the cast can hide a necessary change in the code that does the > > cast. Using CONST_CAST, instead of a bare cast, makes these changes > > visible. > > > > Inspired by my own work elsewhere: > > http://git.savannah.gnu.org/cgit/pspp.git/tree/src/libpspp/cast.h#n80 > > > > Signed-off-by: Ben Pfaff <b...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
