Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

fix 6 year old bug in ipaddress.h#28

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

Already on GitHub?Sign in to your account

Closed
Rotzbua wants to merge1 commit intoarduino:masterfromRotzbua:fix_extern

Conversation

@Rotzbua
Copy link

This bug causes multiple sram usage. Now only one instance should be in sram.

fixarduino/Arduino#1007

NicoHood, awatterott, Schweigst, and PaoloP74 reacted with thumbs up emoji
@matthijskooijman
Copy link
Collaborator

I wonder if this would be better fixed by making the declaration in the .h fileconstexpr and give it a value there. I believe that, to allow taking the address of the constant, it still needs a definition in the .cpp file as introduced by this PR, but then the definition does not need to assign a value.

Floessie reacted with thumbs up emoji

@facchinm
Copy link
Member

Fixed byarduino/ArduinoCore-API@3dbd7d7 (as soon as AVR core will be based on chainsaw the patch will be applied here as well)

@aentinger
Copy link
Contributor

Closing this issue since initial port of ArduinoAPI has been done and this issue is fixed witharduino/ArduinoCore-API@3dbd7d7 as noted by@facchinm .

@RotzbuaRotzbua deleted the fix_extern branchNovember 3, 2019 21:24
darkk added a commit to darkk/ArduinoCore-avr that referenced this pull requestOct 29, 2025
NTPClient-based project shrunk per `size firmware.elf` - .text 30100 -> 29838: -262 bytes, -0.8% of Uno Flash - .bss    602 ->   542:  -60 bytes, -2.9% of Uno SRAMThat addressesarduino#28,arduino#461,arduino#463 andarduino/Arduino#1007
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Multiple instances of INADDR_NONE [imported]

4 participants

@Rotzbua@matthijskooijman@facchinm@aentinger
[8]ページ先頭
©2009-2025 Movatter.jp