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(draw): fix compiler error in lv_draw_sw_transform.c #2 #4612

Merged
merged 4 commits into from Sep 29, 2023

Conversation

C47D
Copy link
Contributor

@C47D C47D commented Sep 28, 2023

Description of the feature or fix

Supersedes #4506

Checkpoints

Be sure the following conventions are followed:

  • Follow the Styling guide
  • Prefer enums instead of macros. If inevitable to use defines export them with LV_EXPORT_CONST_INT(defined_value) right after the define.
  • In function arguments prefer type name[] declaration for array parameters instead of type * name
  • Use typed pointers instead of void * pointers
  • Do not malloc into a static or global variables. Instead declare the variable in lv_global_t structure in lv_global.h and mark the variable with (LV_GLOBAL_DEFAULT()->variable) when it's used. See a detailed description here.
  • Widget constructor must follow the lv_<widget_name>_create(lv_obj_t * parent) pattern.
  • Widget members function must start with lv_<modul_name> and should receive lv_obj_t * as first argument which is a pointer to widget object itself.
  • structs should be used via an API and not modified directly via their elements.
  • struct APIs should follow the widgets' conventions. That is to receive a pointer to the struct as the first argument, and the prefix of the struct name should be used as the prefix of the function name too (e.g. lv_disp_set_default(lv_disp_t * disp))
  • Functions and structs which are not part of the public API must begin with underscore in order to mark them as "private".
  • Arguments must be named in H files too.
  • To register and use callbacks one of the following needs to be followed (see a detailed description here):
    • For both the registration function and the callback pass a pointer to a struct as the first argument. The struct must contain void * user_data field.
    • The last argument of the registration function must be void * user_data and the same user_data needs to be passed as the last argument of the callback.
    • Callback types not following these conventions should end with xcb_t.

Compiler issues error on lv_color_t initializer {0} not yet converted
to new initializer format:  _LV_COLOR_ZERO_INITIALIZER in
argb_and_rgb_aa() function.
PRIX32 next to LV_LOG_WARN() string is only implemented in 'master' branch,
not in 'release/v8.3' branch, causing PRIX32 to be an unrecognized symbol.
Handled by simply adding the sprintf format character 'X' to end of string
simulating original intention in 'master' branch.
docs/CHANGELOG.md Outdated Show resolved Hide resolved
@kisvegabor kisvegabor merged commit 3b2d75b into lvgl:release/v8.3 Sep 29, 2023
17 checks passed
@kisvegabor
Copy link
Member

Thanks, I've just removed the CHANGELOG changes.

@C47D C47D deleted the lv_draw_sw_transform_fix branch September 29, 2023 13:16
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.

None yet

3 participants