deps: suppress warnings in deps/uvwasi/src/uvwasi.c

[HarshithaKP]

Declaration of function 'seekdir' and 'telldir'
Declaration of function 'strlen'
Refs: #18983

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

[cjihrig]

Can you open this against https://github.com/cjihrig/uvwasi first?

[addaleax]

Also, both of these includes are already present. The reason for the warnings is that there are missing feature macros that enable those functions to be declared from these headers, not that the headers aren’t included.

[HarshithaKP]

@cjihrig Sure I will open this in https://github.com/cjihrig/uvwasi

[HarshithaKP]

@addaleax Yes the header files are already included, thank you for the correction.
Minimal reproduce following node.js build flags

$ cat foo.c

#include <stdio.h>
#include <string.h>

int main() {
  char str[128] = "hello";
  strnlen(str, 4);
  return 0;
}

$ cc -D_POSIX_C_SOURCE=200112 -Wextra foo.c

foo.c: In function 'main':
foo.c:6:3: warning: implicit declaration of function 'strnlen' [-Wimplicit-function-declaration]
   strnlen(str, 4);

I am able to circumvent by defining #define _GNU_SOURCE in the source prior to header file inclusion.

$ cc -D_POSIX_C_SOURCE=200112 -Wextra foo.c
$

Is this a right approach?

[addaleax]

I’ll close this PR since there is one upstream now and we’ll get that with the next dependency update – thanks for taking this on, though!