On Mon, Apr 20, 2015 at 2:21 PM, Thomas Graf <[email protected]> wrote:
> 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. > Yeah, using CI would be ideal~ Thx, I'll send out patches soon, Alex Wang, _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
