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

Flow plugin reorder streams - fix check to determine if reorder is necessary #602

Closed
wants to merge 10 commits into from

Conversation

Marnalas
Copy link
Contributor

@Marnalas Marnalas commented Feb 25, 2024

Hi there,

This is my proposal to fix #601.

@HaveAGitGat
Copy link
Owner

Would it not be simpler to just move const originalStreams = JSON.stringify(streams); down 4 lines rather than these other changes or am I missing something?

@Marnalas
Copy link
Contributor Author

Marnalas commented May 8, 2024

You are perfectly right. Thank you for taking the time.

One thing to note though is comparing the whole stream array to see if some items were switched is probably overkill. That's why I made an attempt with performance in mind. That being said, not sure the trade off is worth it. I'll let you be the judge of that ;)

@HaveAGitGat
Copy link
Owner

Yeah think it's fine doing it that way in this case as comparison is only being done once so the overhead isn't very much.

Added here ty:
#629

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.

[BUG] Flow plugin Reorder streams always reorder even if not needed
2 participants