[Xfce-bugs] [Bug 15288] Change image loading backend from GdkPixbuf to abydos
https://bugzilla.xfce.org/show_bug.cgi?id=15288 Magnus Bergman changed: What|Removed |Added Depends on||15874 --- Comment #5 from Magnus Bergman --- I filed a bug (bug 15874) for changing the file loading according to the first approach mentioned above (loading images in a separate thread). I will wait for that bug to be resolved before preparing a new patch for this one. Referenced Bugs: https://bugzilla.xfce.org/show_bug.cgi?id=15874 [Bug 15874] Simply load the image in a thread instead async file reading and progressive loading -- You are receiving this mail because: You are the assignee for the bug. ___ Xfce-bugs mailing list Xfce-bugs@xfce.org https://mail.xfce.org/mailman/listinfo/xfce-bugs
[Xfce-bugs] [Bug 15288] Change image loading backend from GdkPixbuf to abydos
https://bugzilla.xfce.org/show_bug.cgi?id=15288 Jani Välimaa changed: What|Removed |Added CC||wa...@mageia.org -- You are receiving this mail because: You are the assignee for the bug. ___ Xfce-bugs mailing list Xfce-bugs@xfce.org https://mail.xfce.org/mailman/listinfo/xfce-bugs
[Xfce-bugs] [Bug 15288] Change image loading backend from GdkPixbuf to abydos
https://bugzilla.xfce.org/show_bug.cgi?id=15288 --- Comment #4 from Magnus Bergman --- Another possible concern is that abydos cannot be used for image editing (mentioned in bug 5796 and bug 14733). If an image is loaded into memory as a bitmap (cairo image surface), then rotated, flipped and/or cropped, abydos cannot write the changes back to the file (well cairo can write PNG but that's it). IMHO it would be opening a can of worms to add image editing to an image viewer. It could only flaky at best. For some formats it would be virtually impossible to support. And there would be a lot of strange corner cases then it isn't trivial to even know what to ideally do. For example if the user crops a PDF file, should just the current page be cropped or all pages? What if other pages have different orientation or even different sizes, what should happen? It is probable that users only expect it to work on small set of image formats (the ones gdk-pixbuf is able to write), including JPEG. And for a lossy format JPEG it can be suboptimal decode, edit and encode the image again since some changes can be done without re-encoding and suffer further data loss. So I suggest invoking some external editing tools for applying the changes. (But honestly I think users who want to edit images should use a real image editor instead.) It sounds like a good idea to make it a compile time option to use abydos. I could prepare a patch for that. But there is one thing that severely affects the difficulty and complexity of the code: There are two approaches to loading images (both with gdk-pixbuf and abydos), you could either provide the complete content and get the complete image back in one synchronous operation. That could be done in a thread so it doesn't need to be blocking if it's slow for some reason (which might be a good idea anyhow since the current asynchronous code might not be asynchronous enough considering bug 11577). That would be quite simple to implement. The other way is to progressively feed data to the loader as it becomes available and (in cases it's supported) get a gradually loading image to display during the process. Currently Ristretto adds all the complexity to almost support progressive loading, but actually doesn't (nothing is displayed until the image is completely loaded). So that's the intention here? I guess the only use case for progressive loading is very slow internet connections. (But letting non-sandboxed code process potentially malicious data from the internet could be considered a security hazard.) In other cases it's probably preferable to wait until the whole image is available before anything is displayed (which I guess is the reason why the code does what is does today). I could fix the code so progressive loading works. Otherwise I think what progressive loading should be axed in favor of the simpler approach (for improved maintainability). Who has the final say on this? -- You are receiving this mail because: You are the assignee for the bug. ___ Xfce-bugs mailing list Xfce-bugs@xfce.org https://mail.xfce.org/mailman/listinfo/xfce-bugs
[Xfce-bugs] [Bug 15288] Change image loading backend from GdkPixbuf to abydos
https://bugzilla.xfce.org/show_bug.cgi?id=15288 --- Comment #3 from Andre Miranda --- I see, looks very interesting. Although I'm not ristretto's maintainer I can say the main concern is that this library is too recent, so it's not available on distros repositories. Also maintainers may be worried with "what do we do if this lib goes unmaintained?". I think one good approach is to support to abydos as optional, but I don't how much code complexity this would introduce. -- You are receiving this mail because: You are the assignee for the bug. ___ Xfce-bugs mailing list Xfce-bugs@xfce.org https://mail.xfce.org/mailman/listinfo/xfce-bugs
[Xfce-bugs] [Bug 15288] Change image loading backend from GdkPixbuf to abydos
https://bugzilla.xfce.org/show_bug.cgi?id=15288 Igor changed: What|Removed |Added CC||f2...@yandex.ru -- You are receiving this mail because: You are the assignee for the bug. ___ Xfce-bugs mailing list Xfce-bugs@xfce.org https://mail.xfce.org/mailman/listinfo/xfce-bugs
[Xfce-bugs] [Bug 15288] Change image loading backend from GdkPixbuf to abydos
https://bugzilla.xfce.org/show_bug.cgi?id=15288 --- Comment #2 from Magnus Bergman --- I should of course have mentioned that abydos is my attempt to work around the limitations of GdkPixbuf then used for image viewers. It can be found here: http://snisurset.net/code/abydos/ -- You are receiving this mail because: You are the assignee for the bug. ___ Xfce-bugs mailing list Xfce-bugs@xfce.org https://mail.xfce.org/mailman/listinfo/xfce-bugs
[Xfce-bugs] [Bug 15288] Change image loading backend from GdkPixbuf to abydos
https://bugzilla.xfce.org/show_bug.cgi?id=15288 Andre Miranda changed: What|Removed |Added CC||andre...@xfce.org --- Comment #1 from Andre Miranda --- Thanks for the patch, but I can't apply on the current git master. I never heard of this lib abydos, I suppose it's somehow related to cairo (egyption cities), however it seems to be not available in any distribution repository (only in Nix, but it's just a font): https://repology.org/project/abydos/versions Searching on Google also didn't yield any relevant result, can you share a link to the project website? -- You are receiving this mail because: You are the assignee for the bug. ___ Xfce-bugs mailing list Xfce-bugs@xfce.org https://mail.xfce.org/mailman/listinfo/xfce-bugs
