Gouroutine does not release Because it is not initialized #5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Fix cancel via context and respect context deadline * Remove timeout overwrite and remove go1.7 from travis
* Simplify uuid.Reverse method * Add uuid test * Fix linting issues
This prevents multiple evtPeripheralConnected events from causing a deadlock.
* Add ability to get RxMTU from a connection * Fix race detector warning * Fix compiler error
This can be useful when we need to use specific HCI device ID.
The `rsp[0] == ErrorResponseCode && len(rsp) == 5` case was duplicated. Surrounding code uses `rsp[0] == ErrorResponseCode && len(rsp) != 5` (note "not equal") for the second case, so probably this is probably what we want there as well.
* Made Command Flow Control match the actual behavior specified in the Bluetooth spec (Vol 2, Part E, Section 4.4), old version tended to collect extra allowed commands over time and eventually lock up. * comments on magic numbers * add options for setting scanning and advertising parameters. * Close the socket if device intialization fails. * Fix race condition where new data can be written to a connection while it is being disconnected, and the buffers are leaked from the packet pool forever. * Fix typo in comment. * Prevent HCI send from blocking forever if the device doesn't respond. * clear sent table when a reply is received so unexpected packets don't cause random weirdness. * Avoid receiving invalid command response on device stop. * Update readme (go-ble#12) * Update with disclaimer * update README * update README
- acheiving much more stability with this, since errors sometimes happen when network is busy, but are in no way fatal
This makes it easier to build the project with recent versions of Go -- check it out anywhere and build, vs having to put in GOPATH
hci: add Connect/DisconnectHandler options
linux.hci: fix stop advertise on new connection
…mpfix re-adding log to hci.go
…es-not-implement-ble-DeviceOption Make the *darwin.Device implement the ble.DeviceOption interface
This commit replaces the use of XPC services with the "cbgo" library (CoreBluetooth Go).
This fixes what seems to be a Catalina-specific issue. Pre-Catalina: The DidUpdateState delegate callbacks get called *after* the delegate properties gets assigned. Catalina: DidUpdateState gets called immediately upon construction of the manager objects (CBCentralManager and CBPeripheralManager). Prior to this commit, the code assumed it could assign the delegate properties and then listen for the transition to the powered-on state. This works for older versions of macOS, but with Catalina the code blocks forever, waiting for an event that has already occurred. The fix is to use the following procedure: (for each manager object): 1. Assign the delegate. 2. Check the state. If the state is not "unknown" then the procedure is done. 3. Else, wait up to one second for a state change event.
Currently the check to see if a connection was canceled happens *after* newConn() is called, leaking a goroutine waiting to receive on chInPkt. Move the check up.
fix go vet warning for unkeyed field in composite literal
Mojave / Catalina support
Added support for reading RSSI of a connected peripheral
Don't try to handle canceled connections
Corrected ioctl on mips (go-ble#52)
linux/att: fix duplicated switch case error
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.