Skip to content

Streaming UDP API changes (Issue 439) #19

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

Already on GitHub? Sign in to your account

Closed
wants to merge 4 commits into from

Conversation

amcewen
Copy link
Contributor

@amcewen amcewen commented Jan 13, 2011

Changes to the UDP class so that it derives from Stream. Also addition of an IPAddress class to make passing round IP addresses easier.

The "operator uint8_t*()" cast needs to be removed in the long run (it makes it possible for people to end up with pointers to the contents of IPAddress objects that no longer exist) but it's needed in the short-term to allow IPAddress objects to be passed to functions like Ethernet.begin(). I'll update Ethernet, Client and Server to use IPAddress when the DHCP code is added, and remove it then.

…dPacket and readPacket calls have been removed, and replaced with Stream-derived alternatives which provide more commonality with other communications classes and to allow both buffered and full-packet-at-a-time uses. Also includes the introduction of an IPAddress class to make passing them around easier (and require fewer pointers to be exposed)
…d (rather than byte-based) operations don't require a cast. As requested by Tom Igoe. Part of the fix to issue 439.
…hernetClass which takes just a MAC address and gets the rest of its configuration information via DHCP. Examples updated to use the IPAddress class and some have been changed to get their config via DHCP.
@damellis
Copy link
Contributor

Applied: f43c091

@damellis damellis closed this Mar 26, 2011
ricklon added a commit to ricklon/Arduino that referenced this pull request May 18, 2011
tbowmo pushed a commit to tbowmo/Arduino that referenced this pull request Jul 14, 2016
Added MQTTGateway version 0.1b
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