Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevents loading twice if using 'y_rconfix' #143

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MatheusAgL
Copy link

Fixes an error for those who use the YSI library

Fixes an error for those who use the YSI library
@Y-Less
Copy link
Member

Y-Less commented Jul 17, 2021

That won't work correctly, the define is only set when you include YSI, which must be after fixes.inc has been included. You'd need to check for the definition of another symbol like a function within y_rconfix. I'm also not sure if this is now needed as I added some extra checks to the code to detect the filterscript correctly (which should work between scripts as well).

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