Define lwIP's s32/u32 to int by d-a-v · Pull Request #8560 · esp8266/Arduino (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation0 Commits9 Checks0 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

d-a-v

s32/u32 were previously defined as long,
but long can be 64 bits in host mode,
so this commit reduces valgrind complaints and improves coherency.

There is no need to update lwIP's binaries in this commit.

@d-a-v

s32/u32 were previously defined as long, but long can be 64 bits in host mode, so this commit reduces valgrind complaints and increase coherency.

@d-a-v

@d-a-v

@d-a-v

@d-a-v d-a-v mentioned this pull request

May 11, 2022

@mcspr mcspr mentioned this pull request

May 14, 2022

@d-a-v

@d-a-v

hasenradball pushed a commit to hasenradball/Arduino that referenced this pull request

Nov 18, 2024

s32/u32 were previously defined as long, but long can be 64 bits in host mode, so this commit reduces valgrind complaints and increase coherency.

Co-authored-by: Max Prokhorov prokhorov.max@outlook.com

2 participants

@d-a-v @mcspr