Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Big endian fixes #718

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

Big endian fixes #718

wants to merge 2 commits into from

Conversation

Zeex
Copy link
Contributor

@Zeex Zeex commented Oct 27, 2022

What this PR does / why we need it:

Just a couple of small fixes for compiling and running on big-endian systems e.g. MIPS.

One commit addresses a compilation error in amxdbg, another one adds a missing piece of code in amx_Init() to initialize the libraries table similar to the other tables.

Which issue(s) this PR fixes:

Fixes #

What kind of pull this is:

  • A Bug Fix
  • A New Feature
  • Some repository meta (documentation, etc)
  • Other

Additional Documentation:

Zeex added 2 commits October 27, 2022 03:35
Not doing this causes the VM to crash on startup inside amx_Init() when
extracting the names of linked libraries from the name table.
@Zeex Zeex requested a review from a team as a code owner October 27, 2022 19:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant