Skip to content

lego1: Add MxCriticalSection implementation#11

Merged
itsmattkc merged 3 commits intoisledecomp:masterfrom
foxtacles:isle-mxcriticalsection
Jun 12, 2023
Merged

lego1: Add MxCriticalSection implementation#11
itsmattkc merged 3 commits intoisledecomp:masterfrom
foxtacles:isle-mxcriticalsection

Conversation

@foxtacles
Copy link
Copy Markdown
Member

This pull request adds the implementation for MxCriticalSection. The assembly output matches the original (manually verified).

In addition to the existing declarations, I've added two member functions (called them Enter and Leave) that seem to fulfill the functionality of locking/entering the mutex/CS.

It's been over 10 years since I've done any reversing/decompilation so I'm a little rusty and picked something easy to start with. It's great to see how far the tooling has come though! Please let me know if something needs changing/fixing.

@itsmattkc
Copy link
Copy Markdown
Contributor

Excellent, looks good to me. Thanks!

@itsmattkc itsmattkc merged commit dec959f into isledecomp:master Jun 12, 2023
@foxtacles foxtacles deleted the isle-mxcriticalsection branch June 14, 2023 08:51
madebr pushed a commit to madebr/isle that referenced this pull request Jun 25, 2024
* Rename LegoEdge::GetOpposingPoint to LegoEdge::CWVertex (isledecomp#977)

* Fix Radio::HandleControl COMPAT_MODE branch (isledecomp#978)

---------

Co-authored-by: DmitriLeon2000 <dmitryaveron2000@gmail.com>
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