Am 11.08.2014 15:29, schrieb Brian Paul: > On 08/08/2014 06:15 PM, Roland Scheidegger wrote: >> Am 08.08.2014 23:20, schrieb Brian Paul: >>> Silences MinGW warnings: >>> warning: unknown conversion type character ‘l’ in format [-Wformat] >>> warning: too many arguments for format [-Wformat-extra-args] >>> --- >>> src/mesa/main/bufferobj.c | 33 +++++++++++++++++---------------- >>> src/mesa/main/varray.c | 13 ++++++++----- >>> 2 files changed, 25 insertions(+), 21 deletions(-) >>> >>> diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c >>> index 1dfcda3..d59e63c 100644 >>> --- a/src/mesa/main/bufferobj.c >>> +++ b/src/mesa/main/bufferobj.c >>> @@ -31,6 +31,7 @@ >>> */ >>> >>> #include <stdbool.h> >>> +#include <inttypes.h> /* for PRIu64 macro */ >>> #include "glheader.h" >>> #include "enums.h" >>> #include "hash.h" >>> @@ -2824,8 +2825,8 @@ bind_buffers_check_offset_and_size(struct >>> gl_context *ctx, >>> * value in <offsets> is less than zero (per binding)." >>> */ >>> _mesa_error(ctx, GL_INVALID_VALUE, >>> - "glBindBuffersRange(offsets[%u]=%lld < 0)", >>> - index, (long long int) offsets[index]); >>> + "glBindBuffersRange(offsets[%u]=%" PRIu64 " < 0)", >>> + index, (uint64_t) offsets[index]); >> Shouldn't that be printed with PRId64 instead (casting to int64_t)? Same >> for all others (and same in 4/8). > > I used 'u' since buffer sizes and offsets always positive values in > practice. But if you'd prefer signed I can change that. It doesn't > really matter to me. > > -Brian >
Well I was mostly looking at what the code did before and what the type actually is, but for instance this one will print an error only if the value is below zero, so it imho makes more sense to print the value as a signed value. Roland _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
