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
gnrc_sixlowpan_frag: remove redundant PID member #11015
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise looks good
@@ -89,10 +89,9 @@ typedef struct { | |||
*/ | |||
typedef struct { | |||
gnrc_pktsnip_t *pkt; /**< Pointer to the IPv6 packet to be fragmented */ | |||
size_t datagram_size; /**< Length of just the (uncompressed) IPv6 packet to be fragmented */ | |||
uint16_t datagram_size; /**< Length of just the (uncompressed) IPv6 packet to be fragmented */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unrelated, or at least a second commit?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Split out in its own commit.
The interface ID is actually also provided in `pkt` so providing it in the `gnrc_sixlowpan_msg_frag_t` struct is redundant.
It can't get bigger, so it doesn't make sense to waste another 2 byte on it.
a261a2d
to
0954fab
Compare
Murdock's also happy :-) |
Since I'm about to open a PR with another modification to |
(see #11021) |
Contribution description
The interface ID is actually also provided in
pkt
so providing it in thegnrc_sixlowpan_msg_frag_t
struct is redundant.Testing procedure
examples/gnrc_networking
on a 6Lo-compatible board (e.g.iotlab-m3
,samr21-xpro
, etc.) should still be able to ping with large payloads (e.g. >100) to a node flashed withexamples/gnrc_border_router
(link-local addresses is enough).Issues/PRs references
None.