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

Implementation of different image formats #9

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ahsand97
Copy link

Hey I think I implemented support to multiple formats using the ImageMagick high level C API, I already tested with .svg, .xpm, .ico, .png, .jpg and extensionless valid image and it works.

Demonstration:
Images used:

Screenshot_20211218_214033

extensionless .svg file
image

.svg file
image

.png file
image

.xpm file
image

.jpg file (almost invisible but is the twitter one)
image

.ico file
image

PS: Also I refactored some code solving some warnings that Intellij CLion was suggesting.

@ahsand97 ahsand97 mentioned this pull request Dec 19, 2021
@xeyownt
Copy link
Owner

xeyownt commented Jan 31, 2022

Hi ahsand97, thanks for the pull request.
I'll give it a try and integrate it if ok.

@ahsand97
Copy link
Author

ahsand97 commented Feb 1, 2022

Hi ahsand97, thanks for the pull request. I'll give it a try and integrate it if ok.

Thanks, I didn't update the dependencies to the snap version cuz I didn't reallt know how to, so I believe the snap doesn't have the MagickWand dependency yet

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