]> git.djapps.eu Git - pkg/ggml/sources/whisper.cpp/commitdiff
whisper : fix compiler warnings in whisper.cpp (#2895)
authorDaniel Bevenius <redacted>
Tue, 18 Mar 2025 12:38:41 +0000 (13:38 +0100)
committerGitHub <redacted>
Tue, 18 Mar 2025 12:38:41 +0000 (13:38 +0100)
This commit fixes compiler warnings in whisper.cpp by changing the type
of the loop index variable from int64_t to size_t.

Currently the following warnings are generated by the compiler:
```console
/whisper.cpp/src/whisper.cpp:209:27: warning: comparison of integers of different signs: 'int64_t' (aka 'long long') and 'size_t' (aka 'unsigned long') [-Wsign-compare]
  209 |     for (int64_t i = 0; i < nels; ++i) {
      |                         ~ ^ ~~~~
/whisper.cpp/src/whisper.cpp:219:27: warning: comparison of integers of different signs: 'int64_t' (aka 'long long') and 'size_t' (aka 'unsigned long') [-Wsign-compare]
  219 |     for (int64_t i = 0; i < nels; ++i) {
      |                         ~ ^ ~~~~
```

src/whisper.cpp

index 986a132cfad7e0c9c9a371ba77515d13f78c101d..2c535cfd05db81aef998f2c761bd680b59a70f47 100644 (file)
@@ -206,7 +206,7 @@ static ggml_tensor * whisper_set_f32(struct ggml_tensor * t, float v) {
     GGML_ASSERT(t->type == GGML_TYPE_F32);
     GGML_ASSERT(ggml_is_contiguous(t));
     size_t nels = ggml_nelements(t);
-    for (int64_t i = 0; i < nels; ++i) {
+    for (size_t i = 0; i < nels; ++i) {
         ((float *) t->data)[i] = v;
     }
     return t;
@@ -216,7 +216,7 @@ static ggml_tensor * whisper_set_i32(struct ggml_tensor * t, int32_t v) {
     GGML_ASSERT(t->type == GGML_TYPE_I32);
     GGML_ASSERT(ggml_is_contiguous(t));
     size_t nels = ggml_nelements(t);
-    for (int64_t i = 0; i < nels; ++i) {
+    for (size_t i = 0; i < nels; ++i) {
         ((int32_t *) t->data)[i] = v;
     }
     return t;