Skip to content

Add some EC files#538

Closed
Ocarinaoftime wants to merge 11 commits intoDarkRTA:masterfrom
Ocarinaoftime:master
Closed

Add some EC files#538
Ocarinaoftime wants to merge 11 commits intoDarkRTA:masterfrom
Ocarinaoftime:master

Conversation

@Ocarinaoftime
Copy link
Copy Markdown
Contributor

@Ocarinaoftime Ocarinaoftime commented Jan 21, 2026

I have started decompiling EC, it's gonna take a minute until it's done lol

I did have to change some of the public/private/protected members of sdk functions just so I could use them in code.

result.h includes all status codes that EC can return for a given function (I think they should all be there)

@Ocarinaoftime
Copy link
Copy Markdown
Contributor Author

I changed objects.json and I created all EC files but not all of them have code in them, so change objects.json how you need.

@Ocarinaoftime
Copy link
Copy Markdown
Contributor Author

OK I think SDK code is back to normal. You don't have to add ec_content files, just csup and the reverted symbols.txt and SDK files

Comment thread src/sdk/ec/log.h

namespace ec {

long lovBufLevel;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should be marked extern.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wdym?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ocarinaoftime
Copy link
Copy Markdown
Contributor Author

I cloned the repo on my VPS, dumped the game, and only added ec_csup files and it successfully compiled, so it's def some of the other files I added

@Ocarinaoftime Ocarinaoftime closed this by deleting the head repository Mar 25, 2026
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