On Tue, 6 Mar 2007 00:40:44 -0500 Michael Jennings <[EMAIL PROTECTED]> babbled:
> On Tuesday, 06 March 2007, at 05:06:04 (+0000), > [EMAIL PROTECTED] wrote: > > > Well, that's probably too drastic. It's still really an > > optionally compiled, loadable module, and raster's last mods seem > > to have fixed the issues... (maybe, hopefully). > > Re-enabling the leak to stop the crash is not "fixing the issues." ooh no - i didn't actually fix any leak. i added the close call HOPING it'd fix the leak - it didn't. i left it in as it caused no issues and hoped it might be better - if rsvg were to be fixed. i changed the free call to be g_object_unref as the docs/header claimed it was deprecated. i disabled the close calls now as they cause crashes on other rsvg versions/builds - but it doesn't make me feel any better about the quality of the code for what is not a wierd corner case, but something as simple as "load a file that isn't svg - then go and leak like mad". > > In any case, I think it would be unwise to remove it. > > Again, it is optional and it does provide a useful service. > > Nor is crashing evas apps a "useful service." :-) > > No offense, but I think librsvg is a mistake. SVG as a format is not > ready, and clearly librsvg is even less ready. svg indeed is not ready. for now though it serves as a nice proof of concept vector loader :) > At minimum, SVG support should be disabled by default. Encouraging > people to use a broken implementation of a broken format is...well, > broken. unfortunately - we have issues like fdo menus and icon specs that REQUIRE svg support... :(:(:( big problems :( i could do something like fork of a child process that converts via a pipe - at least if it dies the parent won't, and we will hit a performance snag - but really - for now i'l let it "be" and see if rsvg fixes itself before release of evas 1.0.0 & e0.17 - if not, disabled it shall become (or address this during alpha/beta). > Michael > > -- > Michael Jennings (a.k.a. KainX) http://www.kainx.org/ <[EMAIL PROTECTED]> > n + 1, Inc., http://www.nplus1.net/ Author, Eterm (www.eterm.org) > ----------------------------------------------------------------------- > "Men always want to please women, but these last 15 years, women have > been hard to please. If you want to resist the feminist movement, > the simple way to do it is to give them what they want, and they'll > defeat themselves." -- Jack Nicholson, "Vanity Fair," April 1994 > > ------------------------------------------------------------------------- > Take Surveys. Earn Cash. Influence the Future of IT > Join SourceForge.net's Techsay panel and you'll get the chance to share your > opinions on IT & business topics through brief surveys-and earn cash > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) [EMAIL PROTECTED] 裸好多 Tokyo, Japan (東京 日本) ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
