On Mon, Jul 24, 2017 at 03:31:38PM -0700, Shashank Ram wrote: > Previously, the function would take the curNbl and nextNbl > as inputs, and modify the linked list, and merge the input > linked list with the newly generated newNbl list. > > This is confusing for the caller, and the function has > unnecessary logic for merging linked lists that instead > the caller should take care of. This is because the > OvsCreateNewNBLsFromMultipleNBs() is a generic API > that can be used by other functions as well, and its > natural for different callers to have different needs. > > This patch refactors the behavior of OvsCreateNewNBLsFromMultipleNBs > to take in the curNbl and lastNbl, and it returns > a linked list of NBLs and sets the HEAD and TAIL of the > new list obtained from the curNbl. If the caller wants > to chain a new linked list at the HEAD or TAIL, it > can make use of the curNbl and lastNbl to do so.
I applied this to master. Thank you! _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
