[Question]: Unused value in the QueryAsync<T>

[asylkhan-azat]

In the SqlMapper.Async Line 451 and
Line 1323 what is the purpose of calling Nullable.GetUnderlyingType, if it is not used afterwards?

[asylkhan-azat]

And, if it's just a mistake, is it ok, if I open a PR for that? It looks like, it should be passed to the GetValue method below it's call

[mgravell]

It is a valid question. No idea, without digging. Possibly it should replace the effective-type a few lines lower, but: I'd need to investigate.

[asylkhan-azat]

I've read through the code and seems like it just could be safely removed. The GetValue already contains the same code to check the underlying type of the nullable to decide which default value to return.

The only theoretical difference in the observable behavior could have been the case, when the passed effectiveType is a null, which will throw an exception before the first call to ReadAsync. And if it's removed, the same exception would be thrown in the GetValue method