From: Daniel Bevenius Date: Fri, 12 Jul 2024 09:05:21 +0000 (+0200) Subject: llama : suppress unary minus operator warning (#8448) X-Git-Tag: upstream/0.0.4488~1105 X-Git-Url: https://git.djapps.eu/?a=commitdiff_plain;h=f53226245f421bd01b47cce43a47e791de82c636;p=pkg%2Fggml%2Fsources%2Fllama.cpp llama : suppress unary minus operator warning (#8448) This commit updates the _try_copy lambda and moves the unary minus operator to after the cast to int32_t. The motivation for this that currently the following warning is generated on windows: ```console llama.cpp\src\llama.cpp(21147,30): warning C4146: unary minus operator applied to unsigned type, result still unsigned ``` --- diff --git a/src/llama.cpp b/src/llama.cpp index f91ac777..59b76a6d 100644 --- a/src/llama.cpp +++ b/src/llama.cpp @@ -21144,7 +21144,7 @@ int32_t llama_token_to_piece(const struct llama_model * model, llama_token token size--; } if (length < (int32_t)size) { - return (int32_t) -size; + return -(int32_t) size; } memcpy(buf, token, size); return (int32_t) size;