fix(param): move some parameters to unsigned

[ghost]

Stacked PRs:

• feat(test): parse as c++ source #568
• chore(build): mpi: set mpicxx, too. #567
• chore(build): add AC_PROG_CXX #566
• feat(rdma): constrain C linkage to init #591
• feat(build): add -Wextra to "picky" compiler flags #588
• fix(platform-aws): fill all platform values #595
• fix(tree): use empty brace initializers for zero-initialization #594
• fix(tracing/nvtx): silence -Wmissing-field-initializer warnings #593
• fix(build): check features before mangling CFLAGS #589
• fix(test): fix typing issues #587
• fix(neuron): remove const from ncclNetPlugin_v{4,5} syms #577
• fix(sendrecv): add missing nccl-headers include #576
• fix(rdma): avoid enum/integral comparison #586
• fix(tree): add fallthrough switch markers #585
• fix(rdma): use COMM_ID_MASK as invalid id #574
• fix(tree): avoid sign comparison issues #575
• ->fix(param): move some parameters to unsigned #571
• feat(param): add uint parameter macro #570
• fix(tuner): fix implicit conversions #573
• fix(tuner): avoid gotos #569
• fix(tree): use decltype instead of typeof for cxx #565
• fix(tree): move declarations to top of function #563

---

fix(param): move some parameters to unsigned

mr_key_size and eager_max_size are positive integers, avoid sign
comparison issues by treating them as such.
Signed-off-by: Nicholas Sielicki nslick@amazon.com

[bwbarrett]

This really should have just been part of #570, but whatever.

[ghost]

Squashed this commit into #570. Thanks.