Update SerializeAdapter #38

New Issue

mohr · 2020-04-17T17:54:41+02:00

mohr commented

2020-04-17 17:54:41 +02:00

Rename AutoSerializeAdapter to SerializeAdapter, remove previous implementation of SerializeAdapter.

Current code can be adapted by removing the template parameter from the class as the AutoSerializeAdapter is able to infer the type.

Also, move SerializeAdapter_ into a private member class. This is in conflict to the solution proposed in #35 of moving it into its own file but seen as a better alternative as it communicates the intended use ("don't") better to the user.

Rename AutoSerializeAdapter to SerializeAdapter, remove previous implementation of SerializeAdapter. Current code can be adapted by removing the template parameter from the class as the AutoSerializeAdapter is able to infer the type. Also, move SerializeAdapter\_ into a private member class. This is in conflict to the solution proposed in #35 of moving it into its own file but seen as a better alternative as it communicates the intended use ("don't") better to the user.

mohr added the

feature

API Change

labels 2020-04-17 17:54:41 +02:00

muellerr commented

2020-04-20 10:43:51 +02:00

I will adapt #35 to these changes

mohr referenced a pull request that will close this issue

2020-06-30 14:44:32 +02:00

Update SerializeAdapter #122

mohr closed this issue

2020-07-10 12:21:16 +02:00