Skip to content

Conversation

@sandeepmistry
Copy link
Contributor

As discussed by @matthijskooijman in #65 (comment):

Also, it might be good to let accept() use the same conditional interrupts() call (in a separate commit, of course)?

Also, incorporated feedback from #65 (comment)

To emphasize the non-portability (and reduce confusion for people reading the code), it might be better to even just use __disable_irq() and __enable_irq() directly?

@matthijskooijman
Copy link
Collaborator

Looks good! I would remove the { } from the if for conciseness, but that's a matter of style/taste :-)

@cmaglie
Copy link
Member

cmaglie commented Feb 18, 2016

Rebased and merged.

@cmaglie cmaglie closed this Feb 18, 2016
@cmaglie cmaglie added this to the Release 1.6.4 milestone Feb 18, 2016
@sandeepmistry sandeepmistry deleted the usb-cdc-accept-primask branch April 19, 2016 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants