Re: Review Request 37217: Switch from handle to concrete class or reference in C++ client

2015-08-12 Thread Alan Conway
> On Aug. 12, 2015, 3:20 p.m., Alan Conway wrote: > > I want to clarify my thoughts for discussion, I'm working on putting this > > into code. > > > > DESIGN PRINCIPLES: > > > > 1. C++ is not python. In C++ it is normal to provide "raw" alloc/free style > > pairs, esp. when wrapping C. Automa

Re: Review Request 37217: Switch from handle to concrete class or reference in C++ client

2015-08-12 Thread Alan Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37217/#review95090 --- I want to clarify my thoughts for discussion, I'm working on putting

Re: Review Request 37217: Switch from handle to concrete class or reference in C++ client

2015-08-11 Thread Alan Conway
> On Aug. 7, 2015, 2:54 p.m., Alan Conway wrote: > > I would like to try something simpler (sorry!) > > > > There are 2 things I'd like to separate: > > > > 1. Clean, simple, no-value-add access to proton types. > > 2. Separate convenience classes to make proton easier to use. > > > > For 1. t

Re: Review Request 37217: Switch from handle to concrete class or reference in C++ client

2015-08-07 Thread Cliff Jansen
> On Aug. 7, 2015, 2:54 p.m., Alan Conway wrote: > > I would like to try something simpler (sorry!) > > > > There are 2 things I'd like to separate: > > > > 1. Clean, simple, no-value-add access to proton types. > > 2. Separate convenience classes to make proton easier to use. > > > > For 1. t

Re: Review Request 37217: Switch from handle to concrete class or reference in C++ client

2015-08-07 Thread Alan Conway
> On Aug. 7, 2015, 2:54 p.m., Alan Conway wrote: > > I would like to try something simpler (sorry!) > > > > There are 2 things I'd like to separate: > > > > 1. Clean, simple, no-value-add access to proton types. > > 2. Separate convenience classes to make proton easier to use. > > > > For 1. t

Re: Review Request 37217: Switch from handle to concrete class or reference in C++ client

2015-08-07 Thread Alan Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37217/#review94542 --- I would like to try something simpler (sorry!) There are 2 things I

Review Request 37217: Switch from handle to concrete class or reference in C++ client

2015-08-07 Thread Cliff Jansen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37217/ --- Review request for qpid, Alan Conway and Andrew Stitcher. Bugs: PROTON-865