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 > >> > >
