Skip to content

Allow for multi-line %hookf usage#117

Open
iCrazeiOS wants to merge 1 commit into
theos:masterfrom
iCrazeiOS:master
Open

Allow for multi-line %hookf usage#117
iCrazeiOS wants to merge 1 commit into
theos:masterfrom
iCrazeiOS:master

Conversation

@iCrazeiOS

Copy link
Copy Markdown

What does this implement/fix? Explain your changes.

Allows for multi-line %hookf usage such as

%hookf(int, MSGetExecutablePath,
		char *buf,
		size_t *size) {
	int ret = %orig(buf, size);
	return ret;
}

Does this close any currently open issues?

No

Any relevant logs, error output, etc?

Without this fix, Logos gives this error while parsing the hookf:
error: missing closing parenthesis

Where has this been tested?

Operating System: macOS

Platform: iOS

@L1ghtmann L1ghtmann left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks for this!

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