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

application.setSystemMenu() crashes app on Windows (indentation problem) #998

Open
scsmash3r opened this issue Oct 31, 2024 · 0 comments
Open
Assignees
Labels
api An issue, task, or discussion related to public runtime APIs bug Something isn't working runtime An issue, task, or discussion related to the runtime core windows

Comments

@scsmash3r
Copy link

There is this function in docs: https://socketsupply.co/apis/#application_setsystemmenuoptions
And there is an example code, that will not work under Windows, when copy-pasted.
The culprit is: example is missing a TAB char at the end of the string in menu variable, thus the app crashes because, probably, can't find a delimiter or a "exit-cycle" symbol?

setSystemMenu_app_crash_windows_indentation.mp4

What OS are you using (uname -a, or Windows version)?

Windows 10 22H2 (64bit)

What version Socket Runtime are you using?

0.6.0-rc.5 (d5637b5) (installed from npm @socketsupply/[email protected])

What programming language are you using (C/C++/Go/Rust)?

rustc 1.82.0 (f6e511eec 2024-10-15)

What did you expect to see and what you saw instead?

Expecting the example to work properly and also expecting to see a proper guide about how to set up a menu with a correct structure. It seems that Windows crashes if TAB symbol is missing before the ` (closing backtick). So the issue under Windows platform is stumbling into proper indentation of a string inside the variable itself.

More context in Discord thread

@jwerle jwerle self-assigned this Nov 15, 2024
@jwerle jwerle added windows api An issue, task, or discussion related to public runtime APIs runtime An issue, task, or discussion related to the runtime core bug Something isn't working labels Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api An issue, task, or discussion related to public runtime APIs bug Something isn't working runtime An issue, task, or discussion related to the runtime core windows
Projects
None yet
Development

No branches or pull requests

2 participants