On 01/26/15 at 05:30pm, Simon Horman wrote: > On Mon, Jan 26, 2015 at 04:40:49PM +0900, Simon Horman wrote: > > * Although somewhat cure the approach of setting the next field to NULL > > s/cure/crude/ > > > seems far less dangerous than trying to update the list infrastructure > > to handle this case.
list_moved() not handling the list_empty() case is somewhat rude. Why not just handle that special case and call list_init() in list_moved() instead? I realize it's currently documented to fail in list_moved() but I don't see any side effect in handling this properly because any caller doing so right now would be buggy. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
