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

fix: palette loading #2722

Merged
merged 2 commits into from
Jul 2, 2022
Merged

fix: palette loading #2722

merged 2 commits into from
Jul 2, 2022

Conversation

rodolforg
Copy link
Contributor

@rodolforg rodolforg commented Jul 2, 2022

Fix #562
Fix #2720

While working on this fix, I found out one problem left to solve:
`synfig::FileSystemNative::ReadStream` seems to not properly work with
its `unget()` method.
It does not affect this fix, though.

Fix synfig#2720
@rodolforg
Copy link
Contributor Author

While working on this fix, I found out one problem left to solve:
synfig::FileSystemNative::ReadStream seems to not properly work with its unget() method.
It does not affect this fix, though.

@ice0
Copy link
Collaborator

ice0 commented Jul 2, 2022

While working on this fix, I found out one problem left to solve: synfig::FileSystemNative::ReadStream seems to not properly work with its unget() method. It does not affect this fix, though.

I have already fixed it, I will do PR tomorrow.

@ice0 ice0 merged commit 081d9ab into synfig:master Jul 2, 2022
@ice0
Copy link
Collaborator

ice0 commented Jul 2, 2022

Merged. Thank you!

@rodolforg rodolforg deleted the fix-562 branch July 2, 2022 17:39
@rodolforg
Copy link
Contributor Author

Backport it?

@ice0
Copy link
Collaborator

ice0 commented Jul 3, 2022

Backport it?

Sure.

Fixed problem with unget() here: #2730

morevnaproject added a commit that referenced this pull request Dec 4, 2022
@rodolforg rodolforg mentioned this pull request Jun 9, 2024
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.

Palette import bug Memory leak on importing saved palette spal
2 participants