On 04/20/15 at 12:28pm, Alex Wang wrote: > Thx a lot for the comments~ > > I think here are my todos: > 1. prefix all public functions in the compat/include with rpl_ + and add > checks > to make sure. > 2. export all rpl_* symbols + add check to make sure all rpl_* are exported.
My concerns were along the same lines as Ben's reply. This resolution seems perfectly fine to me though. I don't really like exporting symbols unless needed but it's probably the lesser evil until we have a solid way of verifying runtime insertion using CI. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
