Skip to content

Fix for read(buf, len) in WifiClient and Firmware#40

Open
glennkreisel wants to merge 1 commit intoarduino:masterfrom
glennkreisel:master
Open

Fix for read(buf, len) in WifiClient and Firmware#40
glennkreisel wants to merge 1 commit intoarduino:masterfrom
glennkreisel:master

Conversation

@glennkreisel
Copy link

read(buf,len) was not returning the bytes actually waiting to be read.
And it was not returning only the requested bytes but many more.

read(buf,len) was not returning the bytes actually waiting to be read.
And it was not returning only the requested bytes but many more.
@DonKnowlton
Copy link

I have been hoping someone would compile this and I could evaluate it by loading it into wifi shield firmware. I don't know how to compile this to create elf or hex files. This could be a significant fix for wifi shield instability problems.

Can anyone do it?

@glennkreisel
Copy link
Author

Yeah the Arduino WIFI shield has some issues.. That is one of them.
I also could not get a consistent run time with it.  I've switched to LinkSprite and have it running for days on end.

G


From: DonKnowlton notifications@github.com
To: arduino/wifishield wifishield@noreply.github.com
Cc: glennkreisel glennkreisel@yahoo.com
Sent: Wednesday, January 8, 2014 3:33 PM
Subject: Re: [wifishield] Fix for read(buf, len) in WifiClient and Firmware (#40)

I have been hoping someone would compile this and I could evaluate it by loading it into wifi shield firmware. I don't know how to compile this to create elf or hex files. This could be a significant fix for wifi shield instability problems.
Can anyone do it?

Reply to this email directly or view it on GitHub.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants