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.

Sign up for GitHub

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

Jump to bottom

nativenet has still problems #148

Closed
mehlis opened this issue Aug 15, 2013 · 4 comments
Closed

nativenet has still problems #148

mehlis opened this issue Aug 15, 2013 · 4 comments
Assignees
@LudwigKnuepfer
Milestone
Release 2013.08

Comments

@mehlis
Copy link
Contributor

mehlis commented Aug 15, 2013

without this:

https://github.com/mehlis/RIOT/commit/ac2fdc10b991953c19bee9c611bae230e820a4e7

PAYLOAD_SIZE is 0

I'm not that deep in nativenet, but warning was that the msg buffer has size 0...

please test this patch! and current master with: -W -Wextra -pedantic
@ghost ghost assigned LudwigKnuepfer Aug 15, 2013
@OlegHahm
Copy link
Member

I fixed that during the merge of #123: 701b817

@mehlis
Copy link
Contributor Author

mehlis commented Aug 15, 2013

ok I'll rebase onto master

@mehlis
Copy link
Contributor Author

mehlis commented Aug 15, 2013

jep it's fixed in your commit

@mehlis mehlis closed this as completed Aug 15, 2013
@OlegHahm
Copy link
Member

Sorry, for introducing this bug fix through the back door, but as you know: the release is coming...

eduazocar pushed a commit to eduazocar/RIOT that referenced this issue Jul 1, 2022
@luisan00
Merge pull request RIOT-OS#148 from luisan00/feature/nvs_uint32
0d2220b 
wifi-subsys: add support for storing uin32 integers in the NVS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LudwigKnuepfer LudwigKnuepfer

Labels
None yet
Projects
None yet
Milestone
Release 2013.08
Development

No branches or pull requests
3 participants
@mehlis @OlegHahm @LudwigKnuepfer