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

nxglib: Remove warning "Assuming 24-bit color is not packed" #478

Merged
merged 1 commit into from
Mar 7, 2020

Conversation

xiaoxiang781216
Copy link
Contributor

Signed-off-by: Xiang Xiao xiaoxiang@xiaomi.com

Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com>
@acassis acassis changed the base branch from master to pr478 March 7, 2020 19:09
@acassis acassis merged commit 5ef75c7 into apache:pr478 Mar 7, 2020
@patacongo
Copy link
Contributor

patacongo commented Mar 7, 2020

There is an issue with the graphics/nxglib/nxglib_fillrun.h header file.

It is not really a header file. It is a "template" of a C file that is included with some definitions to make a complete C file. So it will NOT pass the tests for a header file.

See:

graphics/nxglib/lcd/nxglib_fillrectangle.c
graphics/nxglib/lcd/nxglib_filltrapezoid.c

It is not clear how this case should be handled. I believe we either have change the file name extension (but then it won't be checked at all) or change the extension to .c (which is misleading too).

See Issue: #482

cwespressif pushed a commit to cwespressif/incubator-nuttx that referenced this pull request Apr 3, 2024
…50-udp-send-issue

[T3PW] ESP32s3 t3pw-dev fix lan9250 UDP send dropping 0 or crashing
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