Skip to content

drivers/encx24j600: Allow empty iolist elements#11325

Merged
kaspar030 merged 1 commit intoRIOT-OS:masterfrom
maribu:encx24j600
Apr 1, 2019
Merged

drivers/encx24j600: Allow empty iolist elements#11325
kaspar030 merged 1 commit intoRIOT-OS:masterfrom
maribu:encx24j600

Conversation

@maribu
Copy link
Copy Markdown
Member

@maribu maribu commented Apr 1, 2019

Contribution description

This commits allows using netdev_driver_t::send() to be called with one or more empty iolist elements. This f1x3s issue #11163

Testing procedure

See #11163

Issues/PRs references

F1x3s #11163

(leet-speak used to prevent auto-close)

@maribu maribu requested a review from kaspar030 April 1, 2019 14:26
@maribu maribu added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Apr 1, 2019
} else {
in = ptr;
}
if (len) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

style nit: why not

if (!len) {
    return;
}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done. (Due to the simplicity of this PR, I just squashed right away.)

This commits allows using netdev_driver_t::send() to be called with one or more
empty iolist elements. This fixes issue
RIOT-OS#11163
Copy link
Copy Markdown
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

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

Untested ACK.

@kaspar030 kaspar030 merged commit 200ce49 into RIOT-OS:master Apr 1, 2019
@danpetry danpetry added this to the Release 2019.04 milestone Apr 11, 2019
@maribu maribu deleted the encx24j600 branch August 20, 2019 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants