RFR: 8293779: redundant checking in AESCrypt.makeSessionKey() method [v2]

[Xue-Lei Andrew Fan]

> Hi,
> 
> Please review this simple code cleanup.
> 
> The following checking for key in the makeSessionKey() method is redundant as it the same checking has been performance before calling the method.
> 
> 
>         if (k == null) {
>             throw new InvalidKeyException("Empty key");
>         }
>         if (!isKeySizeValid(k.length)) {
>              throw new InvalidKeyException("Invalid AES key length: " +
>                                            k.length + " bytes");
>         }
> 
> 
> No new regression test, simple cleanup.
> 
> Thanks,
> Xuelei

Xue-Lei Andrew Fan has updated the pull request incrementally with two additional commits since the last revision:

 - remove unused throws in comment
 - remove unused throws

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/10263/files
  - new: https://git.openjdk.org/jdk/pull/10263/files/fa427fd3..d915ca1b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=10263&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10263&range=00-01

  Stats: 3 lines in 1 file changed: 0 ins; 2 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/10263.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10263/head:pull/10263

PR: https://git.openjdk.org/jdk/pull/10263