style(s2n-quic-core): remove redundant assignment

[jmayclin]

The max_ack_delay transport parameter is loaded multiple times. This commit removes that multiple load.

Description of changes:

The max_ack_delay parameter is currently loaded multiple times. This looks to be redundant, and this commit removes it.

There are no semantic changes expected with this commit. I also wouldn't be surprised if this gets optimized out by the compiler, so I don't expect any performance changes either.

Call-outs:

I have this commit under the style tag, since it isn't necessarily a bug fix. And if the compiler is optimizing this out, then it's not a perf thing either.

Testing:

Passed all unit tests.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[jmayclin]

qns / tshark failing on unrelated configuration error

 CMake Error at cmake/modules/FindPCAP.cmake:212 (message):
  You need libpcap 0.8 or later
Call Stack (most recent call first):
  CMakeLists.txt:1162 (find_package)
  CMakeLists.txt:1211 (ws_find_package)