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

chore: fix spelling #6401

Merged
merged 167 commits into from
Jun 28, 2024
Merged

chore: fix spelling #6401

merged 167 commits into from
Jun 28, 2024

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Jun 20, 2024

Description of the feature or fix

Fixes misspellings identified by the check-spelling action.

The misspellings have been reported at https://github.com/jsoref/lvgl/actions/runs/9605104882#summary-26492045362

The action will report that the changes in this PR would make it happy: https://github.com/jsoref/lvgl/actions/runs/9605105200#summary-26492046938

Notes

jsoref added 5 commits June 18, 2024 19:19
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
@jsoref jsoref marked this pull request as draft June 20, 2024 22:24
@@ -352,7 +352,7 @@
#define LV_ATTRIBUTE_FAST_MEM

/*Export integer constant to binding. This macro is used with constants in the form of LV_<CONST> that
*should also appear on LVGL binding API such as Micropython.*/
*should also appear on LVGL binding API such as MicroPython.*/
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

brand

@@ -12,7 +12,7 @@
"vscode": {
// Add the IDs of extensions you want installed when the container is created.
"extensions": [
//"searking.preview-vscode"
//"searKing.preview-vscode"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

docs/Doxyfile Outdated
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't tried to carefully exclude files that are from an upstream. I can drop any files that shouldn't be changed...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've mostly avoided touching translations, but I figured fixing a few brands was ok... Happy to drop if the project prefers to let the localizers deal w/ such things.

docs/ROADMAP.rst Outdated Show resolved Hide resolved
@@ -1187,7 +1187,7 @@ The following features are supported by the decoder:
*) encoding of PNGs, from any raw image to 24- or 32-bit color, or the same color type as the raw image
*) Adam7 interlace and deinterlace for any color type
*) loading the image from harddisk or decoding it from a buffer from other sources than harddisk
*) support for alpha channels, including RGBA color modelete,translucent palettes and color keying
*) support for alpha channels, including RGBA color model,translucent palettes and color keying
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

??

@@ -325,7 +325,7 @@ static void _genSpan(SwRleData* rle, const SwSpan* spans, uint32_t count)
}


static void _horizLine(RleWorker& rw, SwCoord x, SwCoord y, SwCoord area, SwCoord acount)
static void _horizLine(RleWorker& rw, SwCoord x, SwCoord y, SwCoord area, SwCoord aCount)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There appears to be enough camelCasing in this file to justify this change

@@ -862,7 +862,7 @@ bool strokeParseOutline(SwStroke* stroke, const SwOutline& outline)
++pt;
++types;

//emit a signel line_to
//emit a single line_to
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasn't sure if this was signal...

@@ -316,7 +316,7 @@ static int huffext( /* >=0: decoded data, <0: error code */
dc--; /* Decrement number of available bytes */
if(flg) { /* In flag sequence? */
flg = 0; /* Exit flag sequence */
if(*dp != 0) return 0 - (int)JDR_FMT1; /* Err: unexpected flag is detected (may be collapted data) */
if(*dp != 0) return 0 - (int)JDR_FMT1; /* Err: unexpected flag is detected (may be corrupted data) */
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried collapsed and discarded that as a possibility...

@@ -180,8 +180,8 @@ static uint32_t lv_text_get_next_word(const char * txt, const lv_font_t * font,
uint32_t letter = 0; /*Letter at i*/
uint32_t letter_next = 0; /*Letter at i_next*/
int32_t letter_w;
int32_t cur_w = 0; /*Pixel Width of transversed string*/
uint32_t word_len = 0; /*Number of characters in the transversed word*/
int32_t cur_w = 0; /*Pixel Width of traversed string*/
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could be wrong. transverse is a word...

@jsoref jsoref marked this pull request as ready for review June 20, 2024 22:59
jsoref added 21 commits June 20, 2024 19:39
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
jsoref added 10 commits June 20, 2024 19:39
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
@FASTSHIFT FASTSHIFT changed the title Spelling chore: fix spelling Jun 21, 2024
FASTSHIFT
FASTSHIFT previously approved these changes Jun 23, 2024
@liamHowatt
Copy link
Collaborator

liamHowatt commented Jun 26, 2024

Are the fixes in thorvg going to be lost the next time we upgrade it?

Edit: thanks very much for this noble contribution!

@FASTSHIFT
Copy link
Collaborator

Are the fixes in thorvg going to be lost the next time we upgrade it?

@jsoref
Can you also sync this fix with the Thorvg community? So that it can be synced with the next update ;)

@jsoref
Copy link
Contributor Author

jsoref commented Jun 27, 2024

Can you provide me with a link?

@FASTSHIFT
Copy link
Collaborator

Can you provide me with a link?

It's here: https://github.com/thorvg/thorvg.

kisvegabor
kisvegabor previously approved these changes Jun 27, 2024
Copy link
Member

@kisvegabor kisvegabor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing! Thank you very much!

@kisvegabor kisvegabor dismissed stale reviews from FASTSHIFT and themself via e0e8f1a June 27, 2024 10:55
@XuNeo
Copy link
Collaborator

XuNeo commented Jun 27, 2024

Could we add your spelling workflow to lvgl?
https://github.com/jsoref/lvgl/actions/workflows/spelling.yml

@jsoref
Copy link
Contributor Author

jsoref commented Jun 27, 2024

Certainly. I tend to talk people through the configuration and prefer to do it as a separate PR after spelling fixes merge. (There are some choices to make.)

@kisvegabor kisvegabor merged commit df0d36f into lvgl:master Jun 28, 2024
19 checks passed
@kisvegabor
Copy link
Member

Thank you Josh!

@jsoref jsoref deleted the spelling branch June 28, 2024 12:09
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.

6 participants