[ovs-dev] [PATCH] lib: Adapt jsonrpc header file for use in C++.

Ben Pfaff blp at ovn.org
Thu Oct 4 16:28:42 UTC 2018


One point of view is that JSON-RPC is a standard protocol and it's not
OVS's responsibility to implement it for every third-party program.

Another is that we could expose the additional APIs that are needed.

On Wed, Oct 03, 2018 at 07:01:09PM -0700, Yifeng Sun wrote:
> We need a proper method for external code to send queries to db.sock.
> The jsonrpc.h does expose more than what is needed.
> Do you have any better suggestion?
> 
> Thanks,
> Yifeng
> 
> On Wed, Oct 3, 2018 at 6:26 PM Ben Pfaff <blp at ovn.org> wrote:
> 
> > On Wed, Oct 03, 2018 at 04:07:22PM -0700, Yifeng Sun wrote:
> > > This patch exposes jsonrpc.h for inclusion in C++, which will be
> > > used by NSX.
> > >
> > > It added [extern "C"] to the header file and fixed other places
> > > to accommodate this change.
> > >
> > > Signed-off-by: Yifeng Sun <pkusunyifeng at gmail.com>
> >
> > Is this useful by itself?  The header file refers to struct stream,
> > struct pstream, struct reconnect_stats, and struct svec, but these
> > aren't in public headers.
> >


More information about the dev mailing list