Skip to content

Conversation

Lotterleben
Copy link
Member

First attempt at trying to explain the relationship between conn, applications and network stacks a bit better. This is very much WIP and currently mainly exists for @authmillenon to shout at :)

@Lotterleben Lotterleben added the Area: doc Area: Documentation label Jan 19, 2016
@Lotterleben Lotterleben added the State: WIP State: The PR is still work-in-progress and its code is not in its final presentable form yet label Jan 19, 2016
* implementation keeps them compatible to each other. For example, an implementation might allow
* a UDP receive function to be called on a raw IPv6 connection object and even choose to do so
* for valid reasons (e.g. code size), but this definition does not impose this on the
* implementation.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand this explanation ^^. Can you elaborate?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's from the old documentation, I didn't write it ^^ I understood it as a warning that you can't rely on the implementations being completely interchangeable (for example, their definition of conn_udp may be different)?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes. When I find the time I will propose a better explanation :-D

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If my interpretation was correct I can also propose one for you to judge ;)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that there might be no relation between the different connection types.
For simplicity and modularity this API doesn't put any restriction of the actual
implementation of the type. For example, one implementation might choose
to have all connection types have a common base class or use the raw IPv6
connection type to send e.g. UDP packets, while others will keep them
completely seperate from each other.

@OlegHahm OlegHahm mentioned this pull request Jan 22, 2016
24 tasks
@miri64
Copy link
Member

miri64 commented Jan 26, 2016

@Lotterleben what is there still to do (since this PR is marked WIP)?

@Lotterleben
Copy link
Member Author

There's probably a boatload of info missing that I just haven't learned yet, but yeah, It doesn't make sense to stall a PR for "maybe one day"s, does it? Do you have something important in mind that I should describe before we can declare this "good enough for now"? :D

@miri64
Copy link
Member

miri64 commented Jan 26, 2016

No, I would also declare it as "good enough for now". If there is more you can always add a new PR ;-).

@miri64 miri64 removed the State: WIP State: The PR is still work-in-progress and its code is not in its final presentable form yet label Jan 26, 2016
@miri64
Copy link
Member

miri64 commented Jan 26, 2016

ACK & go, when Travis is happy

@miri64 miri64 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jan 26, 2016
@miri64
Copy link
Member

miri64 commented Jan 26, 2016

Ah wait stop, please squash first :-)

@miri64 miri64 added the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Jan 26, 2016
@Lotterleben
Copy link
Member Author

No worries, would've thought that was implied anyways ^^

@Lotterleben Lotterleben removed the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Jan 26, 2016
miri64 added a commit that referenced this pull request Jan 26, 2016
conn: add more detailed documentation (WIP)
@miri64 miri64 merged commit a23b9c3 into RIOT-OS:master Jan 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: doc Area: Documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants