Skip to content

Change maxlength to maxlen in sample extension#886

Merged
Headline merged 1 commit intoalliedmodders:masterfrom
SlidyBat:maxlen-fix
Sep 23, 2018
Merged

Change maxlength to maxlen in sample extension#886
Headline merged 1 commit intoalliedmodders:masterfrom
SlidyBat:maxlen-fix

Conversation

@SlidyBat
Copy link
Contributor

@SlidyBat SlidyBat commented Sep 23, 2018

As stated in the SourceHook Development wiki page:

The error and maxlen arguments must be named error and maxlen respectively, or else certain macros will fail

So this simply fixes that in the sample extension to avoid any possible confusion when using those macros.

Copy link
Member

@Headline Headline left a comment

Choose a reason for hiding this comment

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

We don't include mms in the sample here, but we'll take it for good measure.

Thank you!

@Headline Headline merged commit bece744 into alliedmodders:master Sep 23, 2018
@SlidyBat SlidyBat deleted the maxlen-fix branch October 31, 2018 08:02
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