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(rtthread): implement lv_strcat function in rt-thread due to absence of rt_strcat #5920

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

lhdjply
Copy link
Contributor

@lhdjply lhdjply commented Mar 20, 2024

Description of the feature or fix

fix(warning): fix the issue of lv_string_rtthread prompting for the lack of strcat

image

Notes

FASTSHIFT
FASTSHIFT previously approved these changes Mar 20, 2024
@FASTSHIFT FASTSHIFT changed the title fix(warning): fix the issue of lv_string_rtthread prompting for the lack of strcat fix(stdlib/rtthread): fix the issue of lv_string_rtthread prompting for the lack of strcat Mar 20, 2024
@kisvegabor
Copy link
Member

Based on this issue we should have lv_strncat. @liamHowatt will update LVGL to use lv_strncat. I suggest updating this PR after that.

@lhdjply
Copy link
Contributor Author

lhdjply commented Mar 20, 2024

Based on #5897 we should have lv_strncat. @liamHowatt will update LVGL to use lv_strncat. I suggest updating this PR after that.

Should I turn off my pr? Because I thought that when @liamHowatt was updated, the problem I solved with this pr should have been fixed. Or do I add the lv_strncat function directly to this pr? @kisvegabor

Copy link
Collaborator

@XuNeo XuNeo left a comment

Choose a reason for hiding this comment

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

We will have both lv_strcat and lv_strncat right?

src/stdlib/rtthread/lv_string_rtthread.c Outdated Show resolved Hide resolved
@XuNeo XuNeo changed the title fix(stdlib/rtthread): fix the issue of lv_string_rtthread prompting for the lack of strcat fix(rtthread): implement lv_strcat function in rt-thread due to absence of rt_strcat Mar 20, 2024
@liamHowatt
Copy link
Collaborator

Here is the PR #5927

liamHowatt added a commit to liamHowatt/lvgl that referenced this pull request Mar 20, 2024
@lhdjply lhdjply requested a review from FASTSHIFT March 26, 2024 00:02
@kisvegabor kisvegabor merged commit dc0210b into lvgl:master Mar 27, 2024
16 checks passed
@kisvegabor
Copy link
Member

Thank you all for fixing it!

@lhdjply lhdjply deleted the fix/warning branch March 28, 2024 00:01
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.

5 participants