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 113531 - Galeon importer should use SAX instead of DOM
Galeon importer should use SAX instead of DOM
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: Bookmarks
0.x
Other other
: Normal normal
: ---
Assigned To: Marco Pesenti Gritti
Marco Pesenti Gritti
Depends on:
Blocks:
 
 
Reported: 2003-05-22 14:36 UTC by Ross Burton
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Ross Burton 2003-05-22 14:36:00 UTC 
This is sort-of related to #108585...

Please use SAX instead of DOM when imported Galeon bookmarks. My bookmark
collection is ~2500 bookmarks, and when ephy tries to load this the kernel
kills it after it has used my 512M of RAM and 512M of swap.

If I find the time I may be able to rewrite the imported using SAX myself,
as I did write some of the original XBEL importer for Galeon... :)
Comment 1 Marco Pesenti Gritti 2003-05-22 15:32:08 UTC 
Uhm we checked in a fix for this some time ago, before 0.6.1. The guy
that provided the patch had 2000 bookmarks, so I think it should work.
It's prolly just a bug somewhere. DOM is slower than SAX but it should
have no problems importing 2500 bookmarks.
Comment 2 Ross Burton 2003-05-22 15:33:45 UTC 
Aah, Debian Sid only has 0.6.0 at the moment.  I'll wait for Edd to
update it and close this when I can import my bookmarks.

Sorry for not checking.
Comment 3 Ross Burton 2003-06-02 10:16:32 UTC 
Right, running 0.6.1 now.

Imported 2500 bookmarks in 2 seconds -- lovely :)