Descriptionhttps://sourceforge.net/p/testlilyissues/issues/5409/
Most of these changes eliminate a conversion from int to vsize
(a.k.a. size_t) by changing the public interface to use vsize.
I understand that whether to use signed or unsigned numbers for indexing is a polarizing issue. I prefer using size_t for consistency with the standard library, but if there are strong opinions the other way, I'm willing to rework this patch to restore int to the public interface and cast to size_t in the implementation.
These warnings were observed when building with g++ 8.0.1 on x86_64.
Patch Set 1 #
|