- Inkscape Devel - Inkscape Mailman

[CAD] Status
by Sebastian Götte 04 Sep '13

04 Sep '13

Hi, this week I ironed out all crashes I could find and fixed the libavoid routing I recently broke. I fixed loading/saving of documents containing connectors and points (yay!) and back-ported the parts of the code that would need C++11 to compile to pre-C++11. Tomorrow I will test more complicated libavoid routing and make a plan for object inclusion/exclusion selection and make a test build to verify everything works on a clean system followed by a few test runs inside valgrind. Remaining TODOs are icons for the toolbar buttons, undo handling, some more advanced point manipulation functions (mainly putting points in groups/taking them out of groups) and, finally, rebasing everything onto upstream. And there is the mystery of the lost cursor, but I think I'll figure that one out. The current code can be found at [0]. Regards, Sebastian [0] https://code.launchpad.net/~h-e-6/inkscape/connector-wip

2 2

[Recolor] Disconnect desktop
by Arshdeep Singh 02 Sep '13

02 Sep '13

Hi all, I needed some help regarding the desktop connection facility. Right now when I have an active selection on the desktop, I have to click inside the Recolor Artwork Dialog Box for the node to be added to the wheel. I am unable to track why this doesn't happen automatically ? Can someone help ? Thanks -- Arshdeep Singh Third Year, Computer Engineering Delhi Technological University Ph: +91-9654115614 https://sites.google.com/site/adsingh1729/home

1 0

[Recolor] Signals, Selection and more
by Arshdeep Singh 02 Sep '13

02 Sep '13

Hi, I have been at work the whole day today. I fixed all crash related issues. Right now, when I select objects on the canvas, a respective node is added onto the recolor wheel. The node is draggable and changes color accordingly. I haven't added the xml related code that shall modify the color property on the canvas. I am trying to fathom if I can somehow use the selection object to keep track of which object was just deleted. Adding the nodes to the wheel is not a problem, deleting it from the wheel is. If I toggle the selection of an object, its node should disappear from the wheel. I am guessing it has something to do with *selectModifiedConn *? Also, if an object in the selection changed its color, its node should be automatically updated. This again has to do with the selection. So far, today has been a very productive day. :) The branch has been updated. -- Arshdeep Singh Third Year, Computer Engineering Delhi Technological University Ph: +91-9654115614 https://sites.google.com/site/adsingh1729/home

2 1

[Units] Document and Object Units
by Matthew Petroff 01 Sep '13

01 Sep '13

In the past week, I have added support for real world document sizes and viewBox based document units. In addition, I have outlined my planned implementation of object specific units. http://www.mpetroff.net/archives/2013/09/01/document-and-object-units/ -Matthew

2 1

[Recolor] Signals: Help needed
by Arshdeep Singh 01 Sep '13

01 Sep '13

Hi all, Today, I fixed all the crashes and got the code working. The nodes are drag-able and nodes are added for every object added in the selection. The working is still buggy. Need to write modified signal call backs and rewire the signals to suit our needs. I have pushed the latest development upto the branch: ~moduli16/inkscape/recolor Regards -- Arshdeep Singh Third Year, Computer Engineering Delhi Technological University Ph: +91-9654115614 https://sites.google.com/site/adsingh1729/home

1 0

Request for Merge: fix for bug #771957
by David R. Bild 01 Sep '13

01 Sep '13

Johan Engelen has "signed-off" in the bugtracker this fix for exporting multilayer pdf+latex output, but we need someone with appropriate privileges to please merge it. Bugtracker: https://bugs.launchpad.net/inkscape/+bug/771957 Fix: https://code.launchpad.net/~drbild/inkscape/multilayer-pdf-latex-output Thanks much, David -- David R. Bild Graduate Student EECS Department University of Michigan 1301 Beal Ave. Ann Arbor, MI 48109

3 9

Re: [Inkscape-devel] Post-merge EMF work
by Krzysztof Kosiński 01 Sep '13

01 Sep '13

2013/8/30 mathog <mathog@...1176...>: > Sure, but since dropping the libUEMF source files out of inkscape isn't > going to be > an option until distros start including it, there is no hurry. OK. For now I'm going to put it in a separate directory, so that it can be easily upgraded from the upstream version. I'll put the symbol font conversion routines in the same directory, since this functionality is closely related to EMF output. I don't know what to do with 'libTERE' yet - I assume you mean the text_reassemble.[hc] files in src/extension/internal. By the way, it seems you haven't added yourself as an author to the EMF output files which you significantly changed. Can I add your name and e-mail there? Regards, Krzysztof

1 1

[CAD] Status
by Sebastian Götte 31 Aug '13

31 Aug '13

Hi, this week I changed the SPPoint selection to use Inkscape's normal selection mechanics instead of the control-point-selection.cpp. This has the advantage that selecting points and connectors at the same time is not quite as bad a hack as it would be otherwise. Doing this I introduced a new signal for selection/deselection in SPObject. I also changed the UI a lot to make it possible to reconnect connectors. SPPoints are now created by shift-clicking into the document while the selection is empty. If the selection is not empty, shift-click toggles the selection of the object under the cursor. Control-click should work as expected with groups. Connectors are terminated by clicking on the end point or somewhere in empty space (in which case a new SPPoint will be created on the clicked location). Sometimes stray points still appear because I have not yet found out how to delete SPPoints without crashing Inkscape. I am working on that. Connector deletion appears to work now. SPPoint deletion still has its issues. After deleting a SPPoint, Inkscape crashes with ominous memory corruption errors. I was unable to find the cause of that so far. I will push a commit later that allows easy reproduction of this. Sebastian

2 3

DialogScript message
by mathog 29 Aug '13

29 Aug '13

Just built branch lp988601 for the first time since Kris merged in changes from Trunk on 8/16. It seems to work fine, but there is a new message at the beginning: Unable to find: DialogScript ** (inkscape:14139): WARNING **: Unknown verb name: DialogScript Is that something old that has been partly removed, or is it something new that has been partly implemented? On the bzr update there was a warning that src/extension/script could not be removed because it was not empty. Related to this message? Thanks, David Mathog mathog@...1176... Manager, Sequence Analysis Facility, Biology Division, Caltech

2 2

[Inskscape-devel] [Recolor] Good News after long
by Arshdeep Singh 29 Aug '13

29 Aug '13

Hi all, I was able to spot the reason for the crashes. It was the type-casting that was causing a pointer nullification. With a few modifications I was able to fix that I got to render the nodes onto the wheel. Its just time to play with signals and xml stuff now. :) Here's a screenshot for interested folks: [image: Inline image 1] -- Arshdeep Singh Third Year, Computer Engineering Delhi Technological University Ph: +91-9654115614 https://sites.google.com/site/adsingh1729/home

1 0

Rethinking revision 12471
by mathog 28 Aug '13

28 Aug '13

I recently became aware of revision 12471, and overall, I am afraid that the way in which it was implemented is detrimental to the readability and future maintenance of the Inkscape code. Presumably the underlying function it implements is to allow changes in the ratio PX/IN, and other units of measurement, and I have no issue with that goal. However, prior to 12471 all of the unit relationships were neatly set up in a set of defines in src/unit-constants.h which defined things like PX_PER_IN which were used elsewhere in the code. These evaluated to static values, and the new ones do not, but that does not negate the usefulness of the define. 12471 removes all of these defines and makes substitutions throughout the code where PX_PER_IN is replaced by Inkscape::Util::Quantity::convert(1, "in", "px") My first criticism is that the new quantity is at first glance, indeterminate. Is it converting IN to PX or vice versa, what does "1" do? My second criticism is that eliminating the defines results in hundreds of substitutions in 12471 that are entirely unnecessary. All that was required was to change the defines in src/unit-constants.h, for instance from #define PX_PER_IN (PT_PER_IN / PT_PER_PX) to #define PX_PER_IN Inkscape::Util::Quantity::convert(1, "in", "px") after which virtually no changes elsewhere in the code are required. By doing that, the existing and relatively clear code, like: p0[X] = (p0[X] * IN_PER_PX * dwDPI); need not become the obfuscated (to my mind) form: p0[X] = (p0[X] * Inkscape::Util::Quantity::convert(1, "px", "in") * dwDPI); Before 12471 PX_PER_IN already evaluated to a complex expression, and the whole point of the define was to let the compiler handle that mess and only present to the programmer a short easily understood term. 12471 is a step backwards in this regard. Regards, David Mathog mathog@...1176... Manager, Sequence Analysis Facility, Biology Division, Caltech

7 22

Fwd: Modifications to RecolorWheelPrivate
by Krzysztof Kosiński 28 Aug '13

28 Aug '13

2013/8/27 Arshdeep Singh <moduli16@...400...>: > I added: > new (&(priv->nodes)) std::map<std::string,RecolorWheelNode>(); > to recolor_wheel_init (RecolorWheel *wheel) . > > Honestly I have never seen this style of code, so I really don't understand > what it does. The good news is it does get to run the dialog box of 'Recolor > Artwork' i.e. no error while the dialog is loaded, but the moment I draw a > shape (active selection) the application crashes. This 'style of code' is known as 'placement new' and it calls the constructor of an object with an explicit 'this' pointer. It needs to be used when memory for an object is allocated from a different source than operator new. In normal C++, placement new is rarely used, because objects are typically allocated and constructed at the same time with operator new. However, if you allocate memory from a different source (in this case from GObject), you need to call the constructor yourself. http://en.wikipedia.org/wiki/Placement_syntax > Also my init function looks like this (for a quick browse, if needed): > > static void > recolor_wheel_init (RecolorWheel *wheel) > { > > RecolorWheelPrivate *priv; > > priv = G_TYPE_INSTANCE_GET_PRIVATE (wheel, RECOLOR_TYPE_COLOR_WHEEL, > RecolorWheelPrivate); > > wheel->priv = (RecolorWheelPrivate*)priv; > priv = (RecolorWheelPrivate*)wheel->priv ; What is the point of the last line? Why the casts? They are unnecessary. > What I don't undersytnd is how to initilaize a static object. > RecolorWheelPrivate has other static members like : What exactly do you mean by "static members"? The structure you provided does not define any static members. > gdouble h,s,v don't need explicit initializers. Why so ? Is it that the only > map object requires an explicit initilization. If so, why ? 'gdouble' is a primitive type (a typedef of 'double') which doesn't have any constructors. GObject zeroes memory by default, so if you don't assign to h, s, v they are left as all zeroes, which corresponds to the double value of 0.0. Regards, Krzysztof

4 9

XDG Templates
by Martin Owens 28 Aug '13

28 Aug '13

I was reviewing the templates work by Jan Darowski and it got me thinking about the FreeDesktop.org XDG spec; it has a provision for a templates directory that we should be using instead of (or as well as) ~/.config/inkscape/ which is hidden from the user. This only applies to FreeDesktops/XDG complient desktops. But it would be useful. http://freedesktop.org/wiki/Software/xdg-user-dirs/ Example: XDG_TEMPLATES_DIR="$HOME/Templates" Thoughts?

3 5

Re: [Inkscape-devel] Modifications to RecolorWheelPrivate
by Arshdeep Singh 28 Aug '13

28 Aug '13

Hi, Recap: I made a separate container for RecolorWheel. Initially there was an initialization error that crashed the application. As pointed out by Krzysztof, the std::map<std::string, RecolorWheelNode> had to be explicitly initialized which made the crash go away. The Recolor Dialog box renders perfectly. Invoking the DialogBox in Inkscape was causing a crash (before the explicit initialization of map) . Unfortunately, the application still crashes when I draw a new object on the canvas. Basically the crash happens when I select an object on the canvas. I was debugging the errors in gdb using g_message but here is the tricky part: When I run the inkscape.exe without gdb the application crashes only when the RecolorArtwork Dialog is active and an object on the canvas is selected. But, in gdb the application starts constantly giving the warning: " HEAP: Free heap block a90b8f6 modified at a90b8f6 after it was freed. " As a result the application doesn't get to the point where I can invoke RecolorArtwork and debug it. My latest branch can be found at: https://code.launchpad.net/~moduli16/inkscape/recolor . Regards, Arshdeep On Wed, Aug 28, 2013 at 2:38 AM, Arshdeep Singh <moduli16@...400...> wrote: > By static I meant 'non pointer' variables. Okay so this doesn't seem to be > the problem causing the crash. Gdb ends up showing the same heap error I > started this thread with. Can you spot an error as to why that keeps > happening ? > On Aug 28, 2013 2:25 AM, "Krzysztof Kosiński" <tweenk.pl@...400...> wrote: > >> 2013/8/27 Arshdeep Singh <moduli16@...400...>: >> > I added: >> > new (&(priv->nodes)) std::map<std::string,RecolorWheelNode>(); >> > to recolor_wheel_init (RecolorWheel *wheel) . >> > >> > Honestly I have never seen this style of code, so I really don't >> understand >> > what it does. The good news is it does get to run the dialog box of >> 'Recolor >> > Artwork' i.e. no error while the dialog is loaded, but the moment I >> draw a >> > shape (active selection) the application crashes. >> >> This 'style of code' is known as 'placement new' and it calls the >> constructor of an object with an explicit 'this' pointer. It needs to >> be used when memory for an object is allocated from a different source >> than operator new. >> >> In normal C++, placement new is rarely used, because objects are >> typically allocated and constructed at the same time with operator >> new. However, if you allocate memory from a different source (in this >> case from GObject), you need to call the constructor yourself. >> >> http://en.wikipedia.org/wiki/Placement_syntax >> >> > Also my init function looks like this (for a quick browse, if needed): >> > >> > static void >> > recolor_wheel_init (RecolorWheel *wheel) >> > { >> > >> > RecolorWheelPrivate *priv; >> > >> > priv = G_TYPE_INSTANCE_GET_PRIVATE (wheel, RECOLOR_TYPE_COLOR_WHEEL, >> > RecolorWheelPrivate); >> > >> > wheel->priv = (RecolorWheelPrivate*)priv; >> > priv = (RecolorWheelPrivate*)wheel->priv ; >> >> What is the point of the last line? Why the casts? They are unnecessary. >> >> > What I don't undersytnd is how to initilaize a static object. >> > RecolorWheelPrivate has other static members like : >> >> What exactly do you mean by "static members"? The structure you >> provided does not define any static members. >> >> > gdouble h,s,v don't need explicit initializers. Why so ? Is it that the >> only >> > map object requires an explicit initilization. If so, why ? >> >> 'gdouble' is a primitive type (a typedef of 'double') which doesn't >> have any constructors. GObject zeroes memory by default, so if you >> don't assign to h, s, v they are left as all zeroes, which corresponds >> to the double value of 0.0. >> >> Regards, Krzysztof >> > -- Arshdeep Singh Third Year, Computer Engineering Delhi Technological University Ph: +91-9654115614 https://sites.google.com/site/adsingh1729/home

1 2

OpenMP support for Clang
by Samuel Chase 27 Aug '13

27 Aug '13

Hello Inkscape Developers, I just came across this: http://clang-omp.github.io/ . I guess once this makes it into Clang, we'll be able to create fully featured (i.e. OpenMP support enabled) Inkscape builds with clang++. Haven't tested anything so far, but this is still an interesting development... Samuel

1 0

Fix for #771957: interleaved text and graphics in pdf+latex output
by David R. Bild 27 Aug '13

27 Aug '13

The current implementation of pdf+latex renders all text objects (LaTeX) above all graphical objects (PDF). I needed to export a figure with text interleaved with graphics---essentially bug #771957---so I implemented a simple fix for it. I'd like to see this (or, more likely, a modified version) merged for 0.49, but I'm not sure who maintains the pdf+latex renderer. Please let me know how to help. The code is at https://code.launchpad.net/~drbild/inkscape/multilayer-pdf-latex-output -- David -- David R. Bild Graduate Student EECS Department University of Michigan 1301 Beal Ave. Ann Arbor, MI 48109

1 0

Fwd: Modifications to RecolorWheelPrivate
by Arshdeep Singh 26 Aug '13

26 Aug '13

---------- Forwarded message ---------- From: Arshdeep Singh <moduli16@...400...> Date: Mon, Aug 26, 2013 at 1:22 PM Subject: Modifications to RecolorWheelPrivate To: Ted Gould <ted@...11...>, Krzysztof Kosiński <tweenk.pl@...400...> Hi, I added a map object by the name 'nodes' to RecolorWheelPrivate: std::map<std::string,RecolorWheelNode> nodes; I then added this snippet for painting recolor nodes from the map: for (std::map<std::string,RecolorWheelNode>::iterator iter = priv->nodes.begin(); iter != priv->nodes.end(); ++iter) { dist = (*iter).second._color[1] * outer ; xx = center_x + cos ((*iter).second._color[0] * 2.0 * G_PI) * dist; yy = center_y - sin ((*iter).second._color[0] * 2.0 * G_PI) * dist; (*iter).second.x = (gfloat)xx; (*iter).second.y = (gfloat)yy; r = (*iter).second._color[0]; g = (*iter).second._color[1]; b = (*iter).second._color[2]; hsv_to_rgb (&r, &g, &b); } I added this snippet to performUpdate() in the recolor-artwork-widget.cpp : void RecolorArtworkWidget::performUpdate() { if ( update || !desktop ) { return; } if ( dragId ) { // local change; do nothing, but reset the flag g_source_remove(dragId); dragId = 0; return; } update = true; Inkscape::Selection *selection = sp_desktop_selection(desktop); GSList const *items = NULL; //int selObj = 0; if ( selection ) { items = selection->itemList(); //selObj = g_slist_length (const_cast<GSList*>(items)); RecolorWheel* wheel = (RecolorWheel*) (((RecolorWheelSelector*)(rsel))->getWheel()) ; RecolorWheelNode temp; for (GSList const *i = items; i != NULL; i = i->next) { SPObject *obj=reinterpret_cast<SPObject *>(i->data); Inkscape::XML::Node* obj_repr = obj->getRepr(); SPCSSAttr* obj_css = sp_repr_css_attr( obj_repr , "style" ); guint32 rgb32 = sp_svg_read_color( sp_repr_css_property( obj_css, "fill", "#ababab") , 0xF0F8FF ); SPColor color = SPColor (rgb32); float rgb[3] , hsv[3]; sp_color_get_rgb_floatv (&color, rgb); sp_color_rgb_to_hsv_floatv (hsv , temp._color[0] , temp._color[1] , temp._color[2] ); add_node_to_recolor_wheel (wheel, obj->getId() , temp ); } } update = false; } There seems to be some runtime error. Opening the RecolorArtwork Dialogbox crashes inkscape. Running gdb shows the warning that "HEAP: Free heap block a90b8f6 modified at a90b8f6 after it was freed" . Can't figure out whats making the application to crash. I have uploaded this branch too. It is the latest revision. -- Arshdeep Singh Third Year, Computer Engineering Delhi Technological University Ph: +91-9654115614 https://sites.google.com/site/adsingh1729/home -- Arshdeep Singh Third Year, Computer Engineering Delhi Technological University Ph: +91-9654115614 https://sites.google.com/site/adsingh1729/home

1 0

[Units] Analysis of SVG Units
by Matthew Petroff 25 Aug '13

25 Aug '13

Sorry for my recent lack of updates. I was out of the country without a computer for a little over a week, was further delayed by a flight cancellation, and just got back. I had meant to post this update prior to leaving but didn't quite finish writing it in time. I have carefully read the SVG specification with regards to units, coming up with examples and a plan for implementation of real world units. http://www.mpetroff.net/archives/2013/08/24/analysis-of-svg-units/ -Matthew

2 1

[Recolor] Progress Report
by Arshdeep Singh 24 Aug '13

24 Aug '13

Hello all, First of all apologies for being out of touch for so long. There was a placement drive in my college (which took place in October every year but was pre-poned to August this year) and hence I was busy studying. I resumed work on the project from 17th August. In a previous discussion with Ted and Krzysztof , it was concluded that making an all-together separate ColorSelector class for RecolorWheel would be better and much more convenient to code. I was busy doing that this week. Now what remains is to hookup the signals for the recolor wheel. The class nesting of Paint-Selector, GimpWheel etc has me caught a bit right now. I'll figure those out soon. Thanks. -- Arshdeep Singh Third Year, Computer Engineering Delhi Technological University Ph: +91-9654115614 https://sites.google.com/site/adsingh1729/home

1 0

Image question
by mathog 23 Aug '13

23 Aug '13

On import of a 10 x 10 color 3 color bitmap from EMF (intrinsically "sharp") it is converted to the following SVG (sorry about the wrap): <image y="374.99722" x="404.997" xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAoAAAAKCAIAAAACUFjqAAAAUElEQVQYlW2OwQ3AMAgDrxJjwP6DAXO0j1QJQfiFMGfzvBBKKG7DIAGgjIrfni4SQPwsrHJLmy4ZZdVsyKulhm/utBT6ytHbzv74uhDvXJV9Tw0bLHsWOIMAAAAASUVORK5CYII=" height="14.999889" width="14.999889" transform="matrix(1,0,0,1,0,0)" id="image4597" /> When viewed in Inkscape on either Linux or Windows there is what looks like a compression artifact, 1 pixel wide, all the way around the square. However, when viewed in Seamonkey, Firefox, or Opera it is sharp, with the colors just as expected from the original bitmap. (Opera is best for this, because it can zoom in further than the other two browsers.) What accounts for this? Is there some switch one can throw in Inkscape to disable whatever it is doing (anti-aliasing, maybe?) to cause this effect? Thanks, David Mathog mathog@...1176... Manager, Sequence Analysis Facility, Biology Division, Caltech

1 0

Re: [Inkscape-devel] [CAD] Status
by Tavmjong Bah 23 Aug '13

23 Aug '13

Hi Sebastian, On Sat, 2013-08-17 at 00:13 +0200, Sebastian Götte wrote: > Hi, > This week I mainly squashed some bugs. The current code [0] still requires -std=c++11 in the CXXFLAGS. > Connector creation is working properly now, without stray points left over afterwards. Points can be selected in groups which can be transformed (scale, shear, rotate, translate etc.). Simple orthogonal routing and straight routing work. Libavoid routing works as far as there are no objects to be avoided (I did not yet test it with objects to be avoided). Multiple connectors can be connected to the same endpoint. > I tried the code but wasn't able to actually draw any connectors. Since then I've upgraded to Fedora 19. Now I cannot compile connector-context.cpp. (It compiled under Fedora 18 I think.) I get the following error: CXX connector-context.o In file included from /usr/include/glibmm-2.4/glibmm.h:89:0, from /usr/include/gdkmm-2.4/gdkmm/pixbuf.h:9, from ./ui/tool/control-point.h:13, from ./ui/tool/selectable-control-point.h:12, from ui/tool/connector-point-control.h:12, from connector-context.cpp:97: /usr/include/glibmm-2.4/glibmm/threads.h:203:11: error: field ‘gobject_’ has incomplete type GThread gobject_; ^ /usr/include/glibmm-2.4/glibmm/threads.h: In member function ‘GThread* Glib::Threads::Thread::gobj()’: /usr/include/glibmm-2.4/glibmm/threads.h:199:29: error: ‘gobject_’ was not declared in this scope GThread* gobj() { return &gobject_; } ^ /usr/include/glibmm-2.4/glibmm/threads.h: In member function ‘const GThread* Glib::Threads::Thread::gobj() const’: /usr/include/glibmm-2.4/glibmm/threads.h:200:41: error: ‘gobject_’ was not declared in this scope const GThread* gobj() const { return &gobject_; } ^ I've noticed that pixbuf.h is included in several other Inkcape source files which do compile. I am not going to have time to investigate further until I get back from vacation after next week. Tav

2 1

Solving bug #166371 - SAX parser
by Krzysztof Kosiński 21 Aug '13

21 Aug '13

Hello After fixing the XXE vulnerability, Inkscape can no longer read SVGs that use entities in URLs. Notable cases include SVGs saved from Adobe Illustrator. There does not seem to be a way to replace these entities without reintroducing the XXE vulnerability. https://bugs.launchpad.net/inkscape/+bug/166371 I just wanted to point out that the best way to solve this would be to use libxml2's SAX parser or the newer XmlTextReader parser to directly construct the tree of Inkscape::XML::Node's, instead of postprocessing the XML tree obtained from the DOM-like parser. With this approach, we could intelligently substitute only string entities and ignore entities that point to URLs or external files. An additional bonus would be a reduction in memory use and load time, since the libxml2 DOM tree would not be created. http://www.xmlsoft.org/xmlreader.html Regards, Krzysztof

3 3

Re: [Inkscape-devel] Inkscape Wiki Account
by Drini Cami 21 Aug '13

21 Aug '13

Hello, I would like an account on the Inkscape wiki. I’ve been learning how to write extensions, and would like to expand the information available. Right now, I’d like to add screenshots of the different types of INX parameters. Thank you, Drini

2 1

EMF import on Linux
by Krzysztof Kosiński 21 Aug '13

21 Aug '13

Hello I know that there is working EMF import on Linux, and you can paste things from e.g. ChemDraw. Is it possible to make this code also work on Linux, using e.g. libemf, or would it require writing a completely new input filter? Regards, Krzysztof

2 4

Re: [Inkscape-devel] Unit testing
by unknown＠example.com 20 Aug '13

20 Aug '13

Quoting Josh Andler <scislac@...400...>: > If it's already in use in 2geom and liked, I'd vote for it just > because of that alone. > > Cheers, > Josh Yes, Google Test and Boost Test are close enough that the use by 2geom might be a strong deciding factor. To summarize some of the differences: Google Test: * does not use exceptions. Among other things this means that if a function is called that might result in a test failing, the calling test will need to add an extra line to check status afterwards. * intended to be copied into the using project's source tree * needs the additional 'Google Mock' library for certain things such as comparing collections Boost Test: * uses exceptions to terminate tests * use is with pre-built libs/source that are not copied in-tree (which is an external dependency on that Boost library) * does not have an academic/artificial distinction between 'Mock' objects and general fakes/stubs So as long as it looks good to switch to Google Test, I'd suggest copying in Google Mock to our source, since it is a superset and contains a copy of the full Google Test sources. We should keep the actual gmock use to a minimum (even Google themselves warn against Mock overuse), but the collections support alone is extremely useful.

1 0