Skip to content
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

Feature/bluetooth #753

Merged
merged 15 commits into from
Jun 3, 2019
Merged

Feature/bluetooth #753

merged 15 commits into from
Jun 3, 2019

Conversation

Wodann
Copy link
Contributor

@Wodann Wodann commented Mar 19, 2019

Extension of PR #604 that adds the ws2bth.h which is necessary for making a Windows implementation of Bluetooth sockets.

Credits to @Jarcho for all of the initial work.

@Wodann Wodann force-pushed the feature/bluetooth branch 2 times, most recently from 88e2d73 to e4f52e4 Compare March 24, 2019 15:55
@Wodann
Copy link
Contributor Author

Wodann commented Mar 24, 2019

Rebased PR to include latest commits on 0.3 branch. Can you look at this PR @retep998 ?

@Wodann
Copy link
Contributor Author

Wodann commented Apr 17, 2019

Rebased again to incorporate v0.3.7 release. The release of my crate is being blocked by this PR. @retep998 If you don't think it is possible to review this PR in the short-term, I'll just have to publish my forked version of the winapi crate. I've investigated other options, but there is unfortunately no other way..

src/shared/bthdef.rs Outdated Show resolved Hide resolved
src/shared/bthdef.rs Outdated Show resolved Hide resolved
src/shared/bthdef.rs Outdated Show resolved Hide resolved
src/um/bluetoothapis.rs Outdated Show resolved Hide resolved
src/um/bluetoothapis.rs Outdated Show resolved Hide resolved
src/um/bthledef.rs Outdated Show resolved Hide resolved
src/um/ws2bth.rs Outdated Show resolved Hide resolved
src/um/ws2bth.rs Outdated Show resolved Hide resolved
src/um/ws2bth.rs Outdated Show resolved Hide resolved
src/um/ws2bth.rs Outdated Show resolved Hide resolved
@Wodann
Copy link
Contributor Author

Wodann commented Apr 17, 2019

Thank you for your review @retep998. I fixed all of the issues you pointed out, and did the same in places where I found similar issues. I hope the updated PR is up to your standards.

I also rebased the PR on top of your latest commit in the 0.3 branch again to prevent potential merge conflicts.

@retep998 retep998 merged commit 77426a9 into retep998:0.3 Jun 3, 2019
CypherNaught-0x pushed a commit to CypherNaught-0x/io-bluetooth-rs that referenced this pull request Feb 3, 2020
Removed winbluetooth since, as per retep998/winapi-rs#753 it's now integrated in winapi.
The version of winapi was also bumped one minor release to the newest version.
No other changes were neccesary due to precise use imports.
@msiglreith msiglreith mentioned this pull request Apr 7, 2020
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.

2 participants