Skip to content

graphics/microwindows: nanoxcalc fix and upstream update to fix warnings - #3770

Open
ppisa wants to merge 1 commit into
apache:masterfrom
ppisa:fix-microwindows-warnings
Open

graphics/microwindows: nanoxcalc fix and upstream update to fix warnings#3770
ppisa wants to merge 1 commit into
apache:masterfrom
ppisa:fix-microwindows-warnings

Conversation

@ppisa

@ppisa ppisa commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

graphics/microwindows: nanoxcalc fix and upstream update to fix warnings

This update introduces the following changes in mw:

  • Fixed all idetified -Werror warnings in CI.
  • Fixed the issue where the backspace key was not working in QEMU

Impact

Make inclusion of Microwindows configuration and documentation closer to pass test.

Testing

Tested on qemu-intel64 target for now

This update introduces the following changes in mw:
- Fixed all idetified -Werror warnings in CI.
- Fixed the issue where the backspace key was not working in QEMU

Signed-off-by: Pavel Pisa <ppisa@pikron.com>
@ppisa

ppisa commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

I see no way how to resolve

error: Mixed case identifier found

because these identifiers are mandated by Nano-X API and has equivalents in X11.

See the file XtoNX.h for standard X11 identifiers mapping to Nano-X ones

https://github.com/ghaerr/microwindows/blob/master/src/nx11/XtoNX.h

which enables to build X11 applications against Nano-X.

XOpenDisplay maps to GrOpen
XCreateSimpleWindow to GrNewWindow
etc.

@ppisa

ppisa commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

@acassis @michallenc Please, can you approve this pull. It should resolve all compiler warnings in Microwindows code. I have resolved all style issues in nxcalc.c except error: Mixed case identifier found but that problem and lot of others is already in nuttx-apps accepted version so this update reduces even these style problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants