Skip to content

Add CCEGLView::showCursor bindings#1336

Open
chs000-00 wants to merge 2 commits into
geode-sdk:mainfrom
chs000-00:main
Open

Add CCEGLView::showCursor bindings#1336
chs000-00 wants to merge 2 commits into
geode-sdk:mainfrom
chs000-00:main

Conversation

@chs000-00

Copy link
Copy Markdown
Contributor

Adds bindings for CCEGLView::showCursor on windows and mac

@chs000-00

Copy link
Copy Markdown
Contributor Author

oh what the hell how did It fail to pass the checks

@Stazzical

Stazzical commented Jul 3, 2026

Copy link
Copy Markdown

The class has link on Windows and you've put an address for Windows for the function you added, I don't know if that's related to the issue

@chs000-00

Copy link
Copy Markdown
Contributor Author

The windows function does exist and hooks fine, it's the MacOS builds that are failing.

@Stazzical

Copy link
Copy Markdown

The issue is that the address in question for Windows I am already assuming already has its symbols included (as is the case with many other functions and classes on libcocos2d.dll) so having the Windows address in the Broma file won't be helpful

@chs000-00

Copy link
Copy Markdown
Contributor Author

You can't hook the function if it doesn't have symbols in broma. I'm made this PR so I could hook this function in one of my mods, as otherwise it did nothing. I tested with and without the broma symbols, and adding the broma symbol made the function hookable.

@Stazzical

Stazzical commented Jul 4, 2026

Copy link
Copy Markdown

it's not the matter with the symbol itself, it's with the address you set for Windows, it doesn't need that, practically every other cocos function has been doing fine without a Windows address because the symbols do exist for it already inside libcocos2d.dll, so it's not needed. Intel and M1 Mac addresses are fine, or that actually might be your issue here with failing builds.

@chs000-00

Copy link
Copy Markdown
Contributor Author

Yes it does. I just explained why I need the address. I am, unable to hook the function otherwise.

@chs000-00

Copy link
Copy Markdown
Contributor Author

The issue with the build is in fact, iMac and m1.

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