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

Fix String Compatibility Issues in BleMouse.cpp Library #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sirfragles
Copy link

This commit addresses the compatibility issues between std::string and Arduino String types in the BleMouse library. Modifications have been made to BleMouse.cpp to ensure that std::string is properly converted to Arduino String where necessary. These changes ensure that the library functions correctly with Arduino IDE Version 2.2.1 and the developer version of the Arduino-ESP32 library (Arduino Alpha 2 Release v3.0.0 based on ESP-IDF v5.1). The library has been tested and verified to work as expected with this setup.

This commit addresses the compatibility issues between std::string and Arduino String types in the BleMouse library. Modifications have been made to `BleMouse.cpp` to ensure that std::string is properly converted to Arduino String where necessary. These changes ensure that the library functions correctly with Arduino IDE Version 2.2.1 and the developer version of the Arduino-ESP32 library (Arduino Alpha 2 Release v3.0.0 based on ESP-IDF v5.1). The library has been tested and verified to work as expected with this setup.
@mfloto
Copy link

mfloto commented Jun 11, 2024

Can confirm. This works in the Arduino IDE:

Version: 2.3.2
CLI Version: 0.35.3

And Arduino core for the ESP32 by espressif version 3.0.1
I have tested it on the seeed studio esp32c3.

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