After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 531059 - Non-incremental loaders can't load from memory
Non-incremental loaders can't load from memory
Status: RESOLVED FIXED
Product: gdk-pixbuf
Classification: Platform
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gdk-pixbuf-maint
gdk-pixbuf-maint
: 469913 (view as bug list)
Depends on:
Blocks: 469913
 
 
Reported: 2008-05-02 13:52 UTC by Bastien Nocera
Modified: 2013-12-19 05:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Bastien Nocera 2008-05-02 13:52:23 UTC 
Currently, there's no way for a loader that doesn't support incremental loading (such as the icns loader, see bug 469913) to load from memory, or remote locations.

gdk-pixbuf should be extended to have a load_stream() member, that would allow loading from GInputStreams (which could be from memory, or remote files).
Comment 1 Denis Pauk 2012-03-25 19:10:33 UTC 
May be we can rewrite file logic to use GIO - and usage GIO interface for file and incremental loading.
Comment 2 William Jon McCann 2012-09-16 16:34:43 UTC 
*** Bug 469913 has been marked as a duplicate of this bug. ***
Comment 3 Rodrigo Silva 2013-05-01 11:01:04 UTC 
Reporter of Bug 629474 posted a patch to add incremental loading for 'icns' files. Can a dev test it and apply it? (or describe why not?)

Maybe the approach he used is general enough to apply to other non-incremental loaders too. Even if it's only for icns, it's better than nothing. It would also close Bug 469912 in eog.

Also, can at least the status of these 3 bugs be updated? They are all confirmed, and possibly duplicates of this one
Comment 4 Rodrigo Silva 2013-05-01 11:09:52 UTC 
This affects not only loading from memory, streams, remote files, but also *local* files when (trying to) use new_from_file_at_size().

So I guess we're stuck not only with local files, but also with only plain new_from_file()
Comment 5 Matthias Clasen 2013-12-19 05:49:02 UTC 
we can load from streams now