Removed useless check

[tkyc]

#2124

[Jeffery-Wasty]

Since ticksSinceMidnight is now only used in one place, can we skip defining a variable? Does SonarQube complain about this? If this suggestion too ugly, we can keep it as you already have it:

        case DATETIME:
            if (8 != decryptedValue.length) {
                MessageFormat form = new MessageFormat(SQLServerException.getErrString("R_NormalizationErrorAE"));
                throw new SQLServerException(form.format(new Object[] {baseSSType}), null, 0, null);
            }

            // SQL datetime is 4 bytes for days since SQL Base Date
            // (January 1, 1900 00:00:00 GMT) and 4 bytes for
            // the number of three hundredths (1/300) of a second since midnight.
            return DDC.convertTemporalToObject(jdbcType, SSType.DATETIME, cal, Util.readInt(decryptedValue, 0),
                    (Util.readInt(decryptedValue, 4) * 10L + 1) / 3, 0);

[tkyc]

Looks like it passed sonarqube. I'm ok with just leaving it as is.