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

Add additional plugin path checks (3.2) #46389

Merged
merged 1 commit into from
Feb 24, 2021
Merged

Add additional plugin path checks (3.2) #46389

merged 1 commit into from
Feb 24, 2021

Conversation

Shatur
Copy link
Contributor

@Shatur Shatur commented Feb 24, 2021

Need for compatibility after #45316.
Fixes #46383.

@YuriSizov
Copy link
Contributor

It works as expected now. Would it make sense to introduce a new method that sanitizes the input so that you don't repeat the checks manually everywhere?

@Shatur
Copy link
Contributor Author

Shatur commented Feb 24, 2021

@pycbouh, agree, done.

Need for compatibility after #45316.
@YuriSizov
Copy link
Contributor

I thought p_addon.begins_with("res://") check would be a part of that method as well, so it actually sanitizes the value. Otherwise LGTM.

@Shatur
Copy link
Contributor Author

Shatur commented Feb 24, 2021

I thought p_addon.begins_with("res://") check would be a part of that method as well, so it actually sanitizes the value.

I thought about it, but it would require making an extra copy of the String even if the path is the same. I know about COW in String, but this will not zero-cost.
Anyway, let me know if I should change it.

@coppolaemilio
Copy link
Member

Did a build with the most recent changes and I get no error 👍🏻
Thanks for the quick turnaround!

@akien-mga akien-mga merged commit d33ee6f into godotengine:3.2 Feb 24, 2021
@akien-mga
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants