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

bluedroid: Replace U+0092 with regular apostrophes in avrc_defs.h (IDFGH-2583) #4667

Closed

Conversation

ulfalizer
Copy link

avrc_defs.h has some scattered U+0092 characters in it for apostrophes.
Replace them with standard U+0027 apostrophes.

The U+0092 apostrophes somehow caused invalid UTF-8 in
https://github.com/zephyrproject-rtos/esp-idf, though it seems to be
fine here.

avrc_defs.h has some scattered U+0092 characters in it for apostrophes.
Replace them with standard U+0027 apostrophes.

The U+0092 apostrophes somehow caused invalid UTF-8 in
https://github.com/zephyrproject-rtos/esp-idf, though it seems to be
fine here.

Signed-off-by: Ulf Magnusson <[email protected]>
@github-actions github-actions bot changed the title bluedroid: Replace U+0092 with regular apostrophes in avrc_defs.h bluedroid: Replace U+0092 with regular apostrophes in avrc_defs.h (IDFGH-2583) Jan 25, 2020
ulfalizer added a commit to ulfalizer/esp-idf-1 that referenced this pull request Jan 25, 2020
Corresponding upstream PR (though it's just the weird apostrophes there,
not invalid UTF-8): espressif/esp-idf#4667

Fixes issues while grepping for #defines, in a new test.

Signed-off-by: Ulf Magnusson <[email protected]>
@Alvin1Zhang
Copy link
Collaborator

Thanks for reporting and sorry for the slow turnaround. Sorry that we are now during the holiday of Chinese Lunar New Year. Will update to you when we are back, thanks.

@AbnerFederer
Copy link

Hi, @ulfalizer

Thanks for your code.

But unfortunately, we have fixed the same annotation error in one MR in our internal repo.

Due to the Chinese Spring Festival, the sync. into GitHub is slower than usual.

Thanks again for contributing to this PR, and we will close the PR and notify you when the fix is on GitHub.

Thanks
Abner

@CLAassistant
Copy link

CLA assistant check
All committers have signed the CLA.

@Alvin1Zhang
Copy link
Collaborator

@ulfalizer Thanks for your contribution.

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.

4 participants