A question about pull #163
maths644311798 opened this issue · 1 comments
maths644311798 commented
In a recent update, you added using ValueType = T; in arg_k.h and changed T GetRequired(const SpiArgKey<T> &key) const { in arg_set.h to
auto GetRequired(const SpiArgKey<T> &key) const ->
typename SpiArgKey<T>::ValueType {
I can not see any problems of the original codes. What’s the advantage of this update?
usafchn commented
@maths644311798 Thank you for your concern. I think that both ways of writing are equivalent, and there is no special intention behind the change here. It is an unintentional change.