Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

warning: no previous prototype for 'recvfrom_packets' #11146

Closed
Keitagit-kun opened this issue May 18, 2023 · 1 comment
Closed

warning: no previous prototype for 'recvfrom_packets' #11146

Keitagit-kun opened this issue May 18, 2023 · 1 comment
Labels

Comments

@Keitagit-kun
Copy link

I did this

I tried to compile static libcurl v8.1.0
It compiled and seems to work OK, but at compile time I get warning:

[ 95%] Building C object lib/CMakeFiles/libcurl.dir/vquic/curl_msh3.c.obj
[ 96%] Building C object lib/CMakeFiles/libcurl.dir/vquic/curl_ngtcp2.c.obj
[ 96%] Building C object lib/CMakeFiles/libcurl.dir/vquic/curl_quiche.c.obj
[ 97%] Building C object lib/CMakeFiles/libcurl.dir/vquic/vquic.c.obj
C:\Temp\curl-8.1.0-OpenSSL3\curl-8.1.0\lib\vquic\vquic.c:434:10: warning: no previous prototype for 'recvfrom_packets' [-Wmissing-prototypes]
 CURLcode recvfrom_packets(struct Curl_cfilter *cf,
          ^~~~~~~~~~~~~~~~
[ 98%] Building C object lib/CMakeFiles/libcurl.dir/vssh/libssh.c.obj
[ 98%] Building C object lib/CMakeFiles/libcurl.dir/vssh/libssh2.c.obj
[ 99%] Building C object lib/CMakeFiles/libcurl.dir/vssh/wolfssh.c.obj
[100%] Linking C static library libcurl.a
[100%] Built target libcurl

I expected the following

no warning

curl/libcurl version

The C compiler identification is GNU 7.2.0
Check for working C compiler: C:/Program Files/mingw-w64/x86_64-7.2.0-win32-seh-rt_v5-rev1/mingw64/bin/gcc.exe
Check for working C compiler: C:/Program Files/mingw-w64/x86_64-7.2.0-win32-seh-rt_v5-rev1/mingw64/bin/gcc.exe -- works
Detecting C compiler ABI info
Detecting C compiler ABI info - done
Detecting C compile features
Detecting C compile features - done
curl version=[8.1.0]
Picky compiler options: -W -Wall -pedantic -Wbad-function-cast -Wconversion -Winline -Wmissing-declarations -Wmissing-prototypes -Wnested-externs -Wno-long-long -Wno-multichar -Wpointer-arith -Wshadow -Wsign-compare -Wundef -Wunused -Wwrite-strings -Wcast-align -Wdeclaration-after-statement -Wempty-body -Wendif-labels -Wfloat-equal -Wignored-qualifiers -Wno-format-nonliteral -Wno-sign-conversion -Wno-system-headers -Wstrict-prototypes -Wtype-limits -Wvla -Wmissing-parameter-type -Wold-style-declaration -Wstrict-aliasing=3 -Wno-pedantic-ms-format -Wformat=2 -Warray-bounds=2 -ftree-vrp -Wduplicated-cond -Wnull-dereference -fdelete-null-pointer-checks -Wshift-negative-value -Wshift-overflow=2 -Walloc-zero -Wduplicated-branches -Wformat-overflow=2 -Wformat-truncation=1 -Wrestrict -Wdouble-promotion -Wunused-const-variable
Found OpenSSL: C:/Temp/curl-8.1.0-OpenSSL3/deps/lib/libcrypto.dll.a  
Performing Curl Test HAVE_WIN32_WINNT
Performing Curl Test HAVE_WIN32_WINNT - Success
Found _WIN32_WINNT=0x502
LTO supported and enabled
Enabled features: SSL IPv6 unixsockets libz brotli zstd AsynchDNS Largefile alt-svc HSTS NTLM HTTP2 HTTP3 HTTPS-proxy unicode threadsafe
Enabled protocols: DICT FILE FTP FTPS GOPHER GOPHERS HTTP HTTPS IMAP IMAPS LDAP LDAPS MQTT POP3 POP3S RTSP SMB SMBS SMTP SMTPS TELNET TFTP
Enabled SSL backends: OpenSSL
Configuring done
@vszakats
Copy link
Member

Can confirm.

@bagder bagder added the build label May 18, 2023
bagder added a commit that referenced this issue May 18, 2023
warning: no previous prototype for 'recvfrom_packets'

Reported-by: Keitagit-kun on github
Fixes #11146
@bagder bagder closed this as completed in 5b4bcc6 May 19, 2023
bch pushed a commit to bch/curl that referenced this issue Jul 19, 2023
warning: no previous prototype for 'recvfrom_packets'

Reported-by: Keitagit-kun on github
Fixes curl#11146
Closes curl#11148
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

3 participants