Remove unnecessary semaphore on Windows

scratchfoundation / scratch-link

Device interoperability layer for Windows and MacOS

BSD 3-Clause "New" or "Revised" License

102 stars 83 forks source link

Remove unnecessary semaphore on Windows #156

Closed terado0618 closed 4 years ago

terado0618 commented 4 years ago

Resolves

Resolves #152

Proposed Changes

Remove the semaphore for the WebSocket session

Reason for Changes

According to a comment of Fleck's maintainer at https://github.com/statianzo/Fleck/issues/191, Fleck is thread safe. This means that the semaphore for the WebSocket session can be removed.

This PR does not significantly improve performance but better than nothing.