use isStride1 to avoid convertSizeT if section.getStride() is empty

[skynet2982]

Fixes #1072

[CLAassistant]

All committers have signed the CLA.

[cwardgar]

In general, I don't like merging changes without a test case to cover them. However, this one is small and I can't imagine how it would bite us.

convertSizeT() will only be called when stride1 == false, and that can only be the case when secStride != null && secStride.length > 0. Therefore, calling convertSizeT() is now safe, and should never result in a null return.

[cwardgar]

@DennisHeimbigner Do you agree with my analysis? You are the code owner, so please merge this if you do.

[DennisHeimbigner]

Assuming that
values.getDataAsByteBuffer(ByteOrder.nativeOrder());
works as expected, this should be ok since enum
basetypes are all integer types.