Re: [PATCH v2] malloc_stats(): Fix `unsigned int` overflow

["Niklas Hambüchen" <mail@nh2.me>]

> This needs a cast to (size_t) as max_mmapped_mem is INTERNAL_SIZE_T,
> which may be smaller than size_t.

Thanks, will fix!

>> +  fprintf (stderr, "max mmap regions = %10d\n", mp_.max_n_mmaps);
> 
> This causes a negative number to be printed when max_n_mmaps increments
> past 2^31, where using "u" defers bad results until 2^32.  So IMHO this
> one is a regression, not a fix.

I'm not sure I understand:
That statement holds for all `int`s, so following this logic, all `int`s should be casted to `unsigned int` first and printed with `%u`, and `%d` would be quite useless.

Doesn't it make more sense to print an `int` with `%d` as normal, so that you can _see_ in `malloc_stats()` when it overflows?

The non-`malloc_stats()` code that uses the `int` will likely also behave strangely if this overflow happens, so it seems weird to pretend the value is of a different type in the malloc-internals inspection function.

(Besides, `max_n_mmaps` sounds like a count of mmaps, not a byte count, so it should be much less likely to overflow.)

Thanks!