static code analysis

Update to 2Geom revision 2396

Just reread the entire diff against trunk. Given the diff size, i must have forgotten things, but hopefully, there are only very few changes of semantics: ->childList is now in the intuitive order (childList()[0] is now firstChild) -> sp_selection_paste_impl is now in the opposite order (change is local to selection-chemistry.cpp, and simplify a few things) -> selection.setReprList now takes the list in the opposite order. It was always the case (the list was always reversed before handing to it) -> a few comparison functions now work "the c++ way": the C way was to return -1 if a<b, 0 if a==b and 1 if a>b, now they return (bool)(a<b)

corrected the casts (hopefully)

Strip ToolFactory

Put a few std::vector<SPitem*>

Just... some... more... lines...

Remove sp_desktop_document and finish cleanup of desktop-handles.h

Purge sp_desktop_controls

Deprecate and remove sp_desktop_selection in favor of SPDesktop::getSelection

Initial removal of box3d outdated GTKish macros.

Move more UI code into ui/

Undo changes in r13391

Clean up some unnecessary pointer usage in livarot

Made constructors of tools use initializer lists.

Moved and renamed some tool-related files.