> 1519 ~Lens();
>
> I believe this should be virtual now.
Right. I wonder why the compiler didn't complain. It usually does in these cases. Fixed.
> 1538 template<class RowAdaptor>
> 1539 Model<RowAdaptor>::Model()
>
> The missing model.SetGetterFunction(...) could be a very nasty surprise in the
> future.
Ah, good catch. I created a new Init() method that gets called from the constructors so they can share the setup code. (my favorite hate-pattern. why oh why can't we chain constructors in C++?!?!?!11). Anyway; fixed.
> 1519 ~Lens();
>
> I believe this should be virtual now.
Right. I wonder why the compiler didn't complain. It usually does in these cases. Fixed.
> 1538 template<class RowAdaptor> r>::Model( ) Function( ...) could be a very nasty surprise in the
> 1539 Model<RowAdapto
>
> The missing model.SetGetter
> future.
Ah, good catch. I created a new Init() method that gets called from the constructors so they can share the setup code. (my favorite hate-pattern. why oh why can't we chain constructors in C++?!?!?!11). Anyway; fixed.