Fix warning in std::vector<bool> serialize code about magistrate HOT 2 CLOSED

auto i = 0u (aka unsigned i = 0) should do the trick. There should be no need to cast these singed/unsigned errors away unless more subtle hints don't bring the types into "signage compatibility".

Although arguable using size_t i = 0; would also be appropriate (required to be same as collection size since C++17, de facto-true before). In actuality, that overflow case would 'not be expected'.

Mainly, I just don't like to see static casts thrown about without cause :|

from magistrate.

this was changed to std::size_t (#78):
https://github.com/DARMA-tasking/checkpoint/blob/0a4fd01a2bce6044eab62a49c6a33f2025ba9c9b/src/checkpoint/container/vector_serialize.h#L95

closing the issue

from magistrate.