Mailing-List: contact users-help@qpid.apache.org; run by ezmlm
Precedence: bulk
Reply-To: users@qpid.apache.org
Message-ID: <1478896897.5242.1.camel@redhat.com>
Subject: Re: Heads-up: Splitting proton-c into multiple libraries [Was:
 Proton's road ahead]
From: Andrew Stitcher <astitcher@redhat.com>
To: users@qpid.apache.org
Date: Fri, 11 Nov 2016 15:41:37 -0500
In-Reply-To: <514603100.8750643.1478895913546.JavaMail.zimbra@redhat.com>
References: 
	<CANVjZ_S1o7eXSTHrHYZSD8iorUGB6GCdv1DjytTCrPMmjT-jmA@mail.gmail.com>
	 <1478882776.5789.113.camel@redhat.com>
	 <CANVjZ_QsA9NfTf_jVqMtu-kc=RzaCt3-ysh_XstWCgXx5-L4vg@mail.gmail.com>
	 <514603100.8750643.1478895913546.JavaMail.zimbra@redhat.com>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 8bit
archived-at: Fri, 11 Nov 2016 20:42:13 -0000

We could certainly copy the URL parser functionality into the C++
binding but as Ken notes it is used in the reactor so we'd have to
leave it there too (although we can make it private).

As for the parser for the pn_data_t I think that can be completely
removed as I'm not aware of any use of it all.

Andrew


On Fri, 2016-11-11 at 15:25 -0500, Ken Giusti wrote:
> IIRC C reactor also uses the url parser.
> 
> 
> ----- Original Message -----
> > 
> > From: "Justin Ross" <justin.ross@gmail.com>
> > To: users@qpid.apache.org
> > Sent: Friday, November 11, 2016 3:02:40 PM
> > Subject: Re: Heads-up: Splitting proton-c into multiple libraries
> > [Was: Proton's road ahead]
> > 
> > On Fri, Nov 11, 2016 at 8:46 AM, Andrew Stitcher <astitcher@redhat.
> > com>
> > wrote:
> > > 
> > > 
> > > Extra (potential) library: All source in directory
> > > - extra
> > > 
> > 
> > Could we avoid having this extra library if we copied the URL
> > parser to the
> > C++ binding?  You noted elsewhere that the only other thing in
> > here, the
> > data parser, isn't currently used.
> > 
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@qpid.apache.org
For additional commands, e-mail: users-help@qpid.apache.org