Skip to content

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

Closed
ulfalizer wants to merge 1 commit into
espressif:masterfrom
ulfalizer:bluedroid-weird-apostrophes
Closed

ulfalizer wants to merge 1 commit into
espressif:masterfrom
ulfalizer:bluedroid-weird-apostrophes

Conversation

@ulfalizer

Copy link
Copy Markdown

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 <ulfalizer@gmail.com>
@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 <Ulf.Magnusson@nordicsemi.no>
@Alvin1Zhang

Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Alvin1Zhang

Copy link
Copy Markdown
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