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

Multiline translation show as ML_NOTFOUND #1069

Open
justgo97 opened this issue Jan 5, 2023 · 2 comments
Open

Multiline translation show as ML_NOTFOUND #1069

justgo97 opened this issue Jan 5, 2023 · 2 comments

Comments

@justgo97
Copy link
Contributor

justgo97 commented Jan 5, 2023

Help us help you

  • [+ ] I have checked that my issue doesn't exist yet.
  • [+ ] I have tried my absolute best to reduce the problem-space and have provided the absolute smallest test-case possible.
  • [+ ] I can always reproduce the issue with the provided description below.

Environment

  • Operating System version:
  • Game/AppID (with version if applicable):
  • Current AMX Mod X version: Lastest master
  • Current Metamod version: lastest
  • [+] I have updated AMX Mod X to the latest version and it still happens.
  • [+] I have updated AMX Mod X to the latest snapshot and it still happens.
  • [+] I have updated Metamod to the latest version and it still happens.

Description

Multiline translation show as ML_NOTFOUND, this issue didn't exist back in amx 1.8.2 so something must have changed in the code that handle lang files to cause this issue

Problematic Code (or Steps to Reproduce)

Just load any lang file with a multi line and try to read it

example here: https://forums.alliedmods.net/showthread.php?t=324405

Logs

  • Please attach in separate files: game output, library logs, kernel logs, and any other supporting information.
  • In case of a crash, please attach minidump or dump analyze output.
@SmileYzn
Copy link
Contributor

SmileYzn commented Jan 8, 2023

Check If text file is in same codepage of any default amxx translation file.

Like UTF8 without BOM

@djearthquake
Copy link

djearthquake commented May 16, 2023

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

No branches or pull requests

3 participants