Skip to content

Correct {infof, failf} format specifiers#2623

Closed
rikardfalkeborn wants to merge 1 commit intocurl:masterfrom
rikardfalkeborn:correct-infof-failf-format-specifiers
Closed

Correct {infof, failf} format specifiers#2623
rikardfalkeborn wants to merge 1 commit intocurl:masterfrom
rikardfalkeborn:correct-infof-failf-format-specifiers

Conversation

@rikardfalkeborn
Copy link
Contributor

This is a followup to #2561 but for infof and failf.

@MarcelRaad
Copy link
Member

%zu is C99 and MSVC has only learned it recently (2013?), unfortunately.

@bagder
Copy link
Member

bagder commented May 31, 2018

%zu is C99

Remember that curl has its own *printf() implementation and it supports %z...

@MarcelRaad
Copy link
Member

Ah, of course, sorry!

@bagder
Copy link
Member

bagder commented May 31, 2018

Thanks!

@bagder bagder closed this in 8ea5d41 May 31, 2018
@rikardfalkeborn rikardfalkeborn deleted the correct-infof-failf-format-specifiers branch May 31, 2018 20:54
@lock lock bot locked as resolved and limited conversation to collaborators Aug 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants