- Inkscape Devel - Inkscape Mailman

Win32 build broken
by Bob Jamison 03 Oct '08

03 Oct '08

Somewhere in the NR->Geom migration, the win32 build was broken. I could fix it tomorrow, but probably the best person would be the one modified the .h file, and knows what the mapping should be. bob ===================== Make error line 293: problem compiling: src/ui/dialog/filedialogimpl-win32.cpp: In member function 'bool Inkscape::UI::Dialog::FileOpenDialogImplWin32::set_svg_ preview()': src/ui/dialog/filedialogimpl-win32.cpp:872: error: no matching function for call to 'sp_item_invoke_bbox(SPItem*, boost::optional<NR::Rect>&, Geom::Matrix, bool )' src/sp-item.h:216: note: candidates are: void sp_item_invoke_bbox(const SPItem*, boost::optional<Geom::D2<Geom::Interval> >&, const Geom::Matrix&, unsigned int, SPItem::BBoxType) src/sp-item.h:217: note: void sp_item_invoke_bbox(const SPItem*, NRRect*, const Geom::Matrix&, unsigned int, SPItem::BBoxType)

2 1

LPE Spiro stress test
by unknown＠example.com 03 Oct '08

03 Oct '08

Hi all, I think I just fixed the long standing bug where LPE Spiro crashed inkscape with a non-continuity exception. (we caught that exception, but okay...) I need some stress testing now! I already did some tests and am no longer able to crash inkscape. Give it a go and let me know if you can crash inkscape with lpe spiro now. Cheers, Johan

3 3

Re: [Inkscape-devel] [PATCH] terminate called after throwing an instance of std::length_error
by Ted Gould 02 Oct '08

02 Oct '08

On Thu, 2008-10-02 at 14:58 +0200, Adrian Knoth wrote: > On Wed, Oct 01, 2008 at 02:51:32PM -0700, Josh Andler wrote: > > turn out to be a duplicate report issue. It appears that the issue in > > the reports is actually library related, as I can not reproduce it with > > a newer distribution (Ubuntu Intrepid) and the 0.46 release. > > I'm running Debian Unstable, which is probably as new as Intrepid, but > I'm facing the bug... This is likely to be that the poppler API changed. We've added a fix for that, and detection, in SVN but not released a patch for 0.46 for it. It's been on my TODO list for a little while as this is also broken with the 0.46 packages in Intrepid and any other distro that is shipping an updated version of poppler. --Ted

2 2

Re: [Inkscape-devel] [PATCH] terminate called after throwing an instance of std::length_error
by Josh Andler 01 Oct '08

01 Oct '08

Hi Adrian! Thank you for the patch. I have taken care of the bug reports as it did turn out to be a duplicate report issue. It appears that the issue in the reports is actually library related, as I can not reproduce it with a newer distribution (Ubuntu Intrepid) and the 0.46 release. I have attached the patch you supplied to the bug report so it can be tested against Ubuntu Hardy to see if it is fixed there (as that generation of distro seems to exhibit the problem). Thanks again, and know that if you want to submit more patches or contribute in any other way, it's always appreciated. :) Cheers, Josh Adrian Knoth wrote: > [Note: I've already sent this yesterday, but the mail didn't make it > through until now, so I resend it with inline patch and without quotes > in the subject] > > [Note2; Yesterday, I tested against NULL, but changed this to 0 in order > to avoid the notion of a pointer.] > > Hi! > > Sorry for not posting to launchpad, I don't want to register all the > time. I'm talking about > > https://bugs.launchpad.net/inkscape/+bug/229412 > > and > > https://bugs.launchpad.net/inkscape/+bug/235383 > > which are probably the same (duplicate). > > I also faced this problem, so I took a closer look. Here's a more > detailed backtrace: > > Starting program: /tmp/inkscape-0.46/src/inkscape /tmp/mon_01_overall.pdf > [Thread debugging using libthread_db enabled] > [New Thread 0x7fe1e17eb6f0 (LWP 12419)] > terminate called after throwing an instance of 'std::length_error' > what(): basic_string::append > > Program received signal SIGABRT, Aborted. > [Switching to Thread 0x7fe1e17eb6f0 (LWP 12419)] > 0x00007fe1d9a26ef5 in raise () from /lib/libc.so.6 > (gdb) bt > #0 0x00007fe1d9a26ef5 in raise () from /lib/libc.so.6 > #1 0x00007fe1d9a28413 in abort () from /lib/libc.so.6 > #2 0x00007fe1da4c6244 in __gnu_cxx::__verbose_terminate_handler () > from /usr/lib/libstdc++.so.6 > #3 0x00007fe1da4c4646 in ?? () from /usr/lib/libstdc++.so.6 > #4 0x00007fe1da4c4673 in std::terminate () from /usr/lib/libstdc++.so.6 > #5 0x00007fe1da4c475a in __cxa_throw () from /usr/lib/libstdc++.so.6 > #6 0x00007fe1da461347 in std::__throw_length_error () > from /usr/lib/libstdc++.so.6 > #7 0x00007fe1da4a3be0 in std::string::append () from /usr/lib/libstdc++.so.6 > #8 0x00007fe1e05dffc2 in Glib::ustring::append (this=0x7fffe9930a20, > src=0x3083 <Address 0x3083 out of bounds>, n=18446744073709551615) > at ustring.cc:494 > #9 0x000000000091f404 in Inkscape::Extension::Internal::SvgBuilder::_flushText > (this=0x3733c00) at extension/internal/pdfinput/svg-builder.cpp:1213 > #10 0x000000000091fb79 in Inkscape::Extension::Internal::SvgBuilder::endTextObject (this=0x3083, state=0x3083) > at extension/internal/pdfinput/svg-builder.cpp:1317 > #11 0x0000000000923b99 in PdfParser::go (this=0x3733dc0, > topLevel=<value optimized out>) > at extension/internal/pdfinput/pdf-parser.cpp:410 > #12 0x0000000000927413 in PdfParser::parse (this=0x3733dc0, > obj=0x7fffe9930da0, topLevel=1) > at extension/internal/pdfinput/pdf-parser.cpp:381 > #13 0x0000000000901653 in Inkscape::Extension::Internal::PdfInput::open ( > this=<value optimized out>, uri=<value optimized out>) > at extension/internal/pdfinput/pdf-input.cpp:704 > #14 0x00000000008de5e1 in Inkscape::Extension::Input::open (this=0x2d58ea0, > uri=0x35caad8 "/tmp/mon_01_overall.pdf") at extension/input.cpp:156 > #15 0x00000000008dd9dd in Inkscape::Extension::open ( > key=<value optimized out>, filename=0x35caad8 "/tmp/mon_01_overall.pdf") > at extension/system.cpp:92 > #16 0x0000000000458899 in sp_file_open (uri=@0x7fffe9930f00, key=0x0, > add_to_recent=true, replace_empty=true) at file.cpp:181 > #17 0x00000000004500ef in sp_main_gui (argc=2, argv=0x7fffe9931088) > at main.cpp:714 > #18 0x0000000000450605 in main (argc=2, argv=0x7fffe9931088) at main.cpp:539 > > > Obviously, frame 9 calls text_buffer.append(&glyph.code, 1). It turned > out that glyph.code[0] eventually is \0, so glyph.code shorter than 1, > hence the exception. > > I haven't digged into the code to find out why glyph.code does not > always contain a sensible value, but I've added a 0 check. > > The attached patch fixes the problem, inkscape can then successfully > import the PDFs in question. > > > Anyway, somebody who's more common with the code should have a look at > this, since my patch only cures the symptoms, not the root cause. > > > HTH > > PS: Somebody should probably also update and merge the bug reports ;) > > > --- src/extension/internal/pdfinput/svg-builder.cpp 2008-03-11 05:20:21.000000000 +0100 > +++ src/extension/internal/pdfinput/svg-builder.cpp.adi 2008-09-30 20:07:18.000000000 +0200 > @@ -1210,7 +1210,9 @@ > last_delta_pos = delta_pos; > > // Append the character to the text buffer > - text_buffer.append((char *)&glyph.code, 1); > + if (0 != glyph.code[0]) { > + text_buffer.append((char *)&glyph.code, 1); > + } > > glyphs_in_a_row++; > i++; > > >

1 0

Cheat Sheet For Converting to 2geom
by Joshua L. Blocher 30 Sep '08

30 Sep '08

For those of you who asked and some who didn't. I give you a copy of my notes on converting from NR:: to Geom::. Please feel free to fill in anything I missed. I will update it with anything new I learn as time permits. http://wiki.inkscape.org/wiki/index.php/CheatSheetForConvertingto2geom Joshua L. Blocher verbalshadow

1 0

current problem with inkscape-svn & poppler-0.8.7
by unknown＠example.com 30 Sep '08

30 Sep '08

I upgrader last week poppler to version 0.8.7 and then get error on inkscape svn compilation, I suppose it's related to poppler upgrade instead of inkscape changes: ../../inkscape/src/extension/internal/pdfinput/pdf-parser.cpp: In member function ‘void PdfParser::doShowText(GooString*)’: ../../inkscape/src/extension/internal/pdfinput/pdf-parser.cpp:2264: error: no matching function for call to ‘GfxFont::getNextChar(char*&, int&, CharCode*, Unicode**, int*, double*, double*, double*, double*)’ /usr/include/poppler/GfxFont.h:204: note: candidates are: virtual int GfxFont::getNextChar(char*, int, CharCode*, Unicode*, int, int*, double*, double*, double*, double*) ../../inkscape/src/extension/internal/pdfinput/pdf-parser.cpp:2317: error: no matching function for call to ‘GfxFont::getNextChar(char*&, int&, CharCode*, Unicode**, int*, double*, double*, double*, double*)’ /usr/include/poppler/GfxFont.h:204: note: candidates are: virtual int GfxFont::getNextChar(char*, int, CharCode*, Unicode*, int, int*, double*, double*, double*, double*) warning: unused variable ‘nSpaces’ Here is the definition of this function in poppler/GfxFont.h: // Get the next char from a string <s> of <len> bytes, returning the // char <code>, its Unicode mapping <u>, its displacement vector // (<dx>, <dy>), and its origin offset vector (<ox>, <oy>). <uSize> // is the number of entries available in <u>, and <uLen> is set to // the number actually used. Returns the number of bytes used by // the char code. virtual int getNextChar(char *s, int len, CharCode *code, Unicode *u, int uSize, int *uLen, double *dx, double *dy, double *ox, double *oy) = 0; It looks like: the Unicode is a ** instead of a * (not sure, I have to look again on k&r & Stroustrup books as I didn't coded for long time) the int uSize is missing in the function call. Regards. Olivier

1 0

lpe vonkoch
by jf barraud 30 Sep '08

30 Sep '08

Hi! I did separate the "reference path" and the "generating path" in lpe-vonkoch. I think it makes the UI more clear, but it changes previous conventions of course (the last segment of the old generating path should be cut/pasted into the new "generating path"). Caution: this might change again. As it is supposed to contain only one segment, the "reference path" could/should be replaced by 2 points (on the other hand, it might be more convenient to be able to move both points at once, copy/paste that segment, etc..., so I'm not sure what's the best here). I also modified the tooltips, and parameter order, trying to follow your remarks. Thank you Alexandre P. and joncruz. Please review it again... Bye, JF.

1 0

Win32 nightly builds don't work since Aug 16
by Alvin Penner 29 Sep '08

29 Sep '08

this is just an attempt to bump the priority of https://bugs.launchpad.net/inkscape/+bug/259237 and http://sourceforge.net/mailarchive/forum.php?thread_name=op.uf6b2ds4a8nfnl%… as far as I can tell, the windows nightly builds have not worked since Aug 16. Using the Aug 21 download, the most recent timestamp that I can see on any files is 08/16/2008 02:49 PM 14,671,360 inkscape.exe part of my reason for posting this is because there is a possibility that I may have caused this, since I did a commit on that day, svn 19670, for the first time, and I'm afraid I may have done it wrong. -- View this message in context: http://www.nabble.com/Win32-nightly-builds-don%27t-work-since-Aug-16-tp1909… Sent from the Inkscape - Dev mailing list archive at Nabble.com.

7 31

website
by Alexandre Prokoudine 28 Sep '08

28 Sep '08

Hi, Could we please have the website fixed? ./publish_web.sh does grab new stuff from SVN, but chnages on website do not appear. That's been happening for last couple of months. Alexandre

4 4

Fwd: NEW: Lots of stuff
by Pajarico 28 Sep '08

28 Sep '08

>> And even with that, it still doesn't recover >> zoom position. 3 works but ` doesn't. Maybe I'm doing something wrong. >> The equivalent menu option "Previous zoom" works, thought. > > Most likely your keyboard just issues some other keycode on pressing > this key. Again, it's only you who can investigate that and propose a > fix. > How can I check this? >> Actually a switch key would work better. Like push 3 for zoom in, push 3 >> again to zoom back. I don't like the idea of keeping Q stroked. > > That wouldn't work. We have much more zoom keys - 1, 2, 3, 4 etc all > have their functions. Not everyone zooms in with the intention to zoom > back again. If you want to zoom to selection _again_ and instead it > zooms you out to some previous zoom, it wouldn't be good usability. Maybe I didn't explain it correctly... I don't intend this to replace the zoom to selection button. Rather this is a new mode which would zoom in when pressed and zoom out when pressed again. The zoom in level would be detrmined by the largest zoom factor used on the last action, and the zoom out level would be determined by the smallest zoom factor used for a determined period of time (to avoid setting a zoom out level accidentally while zooming out with the wheel/zoom tool). The workflow would be like this: 1. You have an object and zoom in and do some fine tuning on the nodes. The zoom in factor is determined (and the zoom position). 2. You zoom back and stare at the changes for a while. In this short period of time the zoom out factor gets determined. 3. Press the key and you go back to zoom in state and do some more tweakings. 4. Press key again and go back to zoom out state. Stare at the editing and go back to zoom in if you wish. Something like this. Would be too cumbersome? > So, a single key which travels back through your history of zooms is a > much better solution. > That's kinda what I'm looking for... implementation details aside. Regards. PS: I'm resending this to inkscape-devel. Sorry bulia.

1 0