On Tue, Oct 21, 2008 at 11:22:15AM +0200, Dirk Stöcker wrote: > On Tue, 21 Oct 2008, Knut Arne Bjørndal wrote: > >> On Tue, Oct 21, 2008 at 08:49:24AM +0200, Dirk Stöcker wrote: >>> the inkscape stuff in Tahlib.pm runCommand should be moved to Inkscape.pm >>> convert() function, as the current state is useless as it is. >> >> Look at my commit to unstable last night, it's moved to Tileset.pm >> together with the other exception handling. > > Why Tileset.pm? > > Inkscape.pm should be the right place.
Hm, maybe. It's imho such a ugly hack I'm not sure I want it in SVG::Rasterize, which I plan to put on CPAN for general usage. It's not easy to put it in Inkscape.pm either, if I just put it in the constructor it will be run even if inkscape isn't the rasterizer to be used, this is because of how I've used Module::Pluggable, maybe there is a better way to do it that also won't add lots of code for all users of the module but I don't have the time to research this now. -- Knut Arne Bjørndal aka Bob Kåre [EMAIL PROTECTED] [EMAIL PROTECTED]
smime.p7s
Description: S/MIME cryptographic signature
_______________________________________________ Tilesathome mailing list [email protected] http://lists.openstreetmap.org/listinfo/tilesathome
