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

build: Fix build errors #1

Open
wants to merge 2 commits into
base: development
Choose a base branch
from
Open

Conversation

calcitem
Copy link
Contributor

@calcitem calcitem commented May 17, 2023

Fix:
Severity Code Description Project File Line Suppression State Error BC30451 'TudL閜ni' is not declared. It may be inaccessible due to its protection level. Malom3 malom\Malom3\Main.vb 1070 Active

This error is most likely caused by Hungarian language support not installed in Windows. So, replace those Hungarian words that cause compile errors with English.

Fix:
Severity	Code	Description	Project	File	Line	Suppression State
Error	BC30451	'TudL閜ni' is not declared. It may be inaccessible due to its protection level.	Malom3	malom\Malom3\Main.vb	1070	Active
@calcitem calcitem closed this May 17, 2023
@calcitem calcitem reopened this May 17, 2023
@ggevay
Copy link
Owner

ggevay commented May 17, 2023

Thank you for the translations and fixing the build error!

We will adjust the translations a bit during the weekend.

@calcitem
Copy link
Contributor Author

You're welcome! I'm glad I could help with the troubleshooting the build error.

Some changes in this pull request are based on educated guesses regarding the translations. I'm looking forward to seeing your adjustments. It's always beneficial to align more closely with the official implement.

@calcitem calcitem force-pushed the dev branch 2 times, most recently from 3f7f02d to e6ac8f2 Compare May 28, 2023 03:48
@calcitem
Copy link
Contributor Author

calcitem commented Jun 4, 2023

In this project https://github.com/calcitem/Sanmill/tree/malom/src/perfect the comments have mostly been replaced with English. I'll make a PR after testing is done.

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.

2 participants