-
-
Notifications
You must be signed in to change notification settings - Fork 234
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
Clarified a bit more documentation, fixed a lot of typos and removed some redundant codes. #311
Conversation
* Added BOX86_SAFEMMAP env. var. | ||
* Reworked dynarec memory manager | ||
* Added a few opcodes | ||
* Fixes on some float to int x86 convertions opcodes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it is x87 actually, not x86.
x86 doesn't handle float, only x87 or SSE does.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have changed it. Should be fine.
CHANGELOG.md
Outdated
* FTL 1.6.9 still have corrupted music, but older 1.5.13 seems fine (different set of libs) | ||
* Added a few wrapped libraries (like libz or some other x11 related library). | ||
* For trace enabled build, Trace can be enabled only after a certain amount of opcodes (still, a debugger would be better). | ||
* Some fixes in a few opcodes, and implemented x86 rounding (SuperMeatBoy behaves better now). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still x87 here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have changed it. Should be fine now.
USAGE.md
Outdated
* 0 : default. Don't log `dlsym` error | ||
* 1 : Log dlsym error | ||
Enables/Disables the logging of `dlsym` errors. | ||
* 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'msorry but no, I really don't want 2 bullet points per options.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have changed it, please let me know your feedback as I am unsure if I fixed it or not.
sync fork with box86
I also changed the copyright year.