Use unsigned when iterating through vector indices

A vector can of course never contain a negative number of elements, so naturally size() returns size_t instead of a regular int. This costs us a warning unless we also change the loop counter (since it is only used to index that vector).

This seems to be the only warning for all GCC-compilers on CDash today (strangely it is accepted by Clang).