It may be work looking into the documentation a little to understand the purpose of the PdxWrapper.
On Thu, Sep 21, 2017 at 11:37 AM David Kimura <dkim...@pivotal.io> wrote: > Is using PdxWrapper any different than if user added their type into the > serialization registry? If not, then do we really want to provide 2 ways > to do the same thing? > > Thanks, > David > > On Thu, Sep 21, 2017 at 10:24 AM, Michael William Dodge <mdo...@pivotal.io > > > wrote: > > > +1 for type safety > > > > Sarge > > > > > On 21 Sep, 2017, at 10:21, Mark Hanson <mhan...@pivotal.io> wrote: > > > > > > Here is a link to my branch in my fork that has the changes on it. > > > > > > https://github.com/mhansonp/geode-native/tree/wip/templatePdxWrapper > > > > > > Thanks, > > > Mark > > > > > > On Thu, Sep 21, 2017 at 10:19 AM, Mark Hanson <mhan...@pivotal.io> > > wrote: > > > > > >> Hi All, > > >> > > >> In reviewing the PdxWrapper class it, it seemed like it would be a > good > > >> move to make this a template class. This will allow better type > checking > > >> anytime we use it. > > >> > > >> An example of what is being planned is to change from > > >> > > >> MyClass object; > > >> PdxWrapper((void *) object,...) > > >> to PdxWrapper<MyClass>(object, ....) > > >> > > >> I have a chunk of code moved over that seems to show it is possible, > > >> though there are some bugs that need to be addressed, so it is not a > > done > > >> deal. > > >> > > >> What do people think? > > >> > > >> Thanks, > > >> Mark > > >> > > > > >
