Clipboard: code cleaning proposal
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Inkscape |
Fix Released
|
Undecided
|
Codain |
Bug Description
Here is not a bug but a proposal after a discussion on the developper mailinglist.
After a deep overview, I've found 4 functions in ClipboardManage
Finally I will keep one of those 4 function in place (_copyNode) because moving it in the Node class creates two much warnings for the moment.
I will post a patch wich will does following things :
Function void ClipboardManage
will become sp_import_
(TODO: look for a merge with file_import() in same file)
Function void ClipboardManage
will become SPDocument:
(no existing function found)
Function SPCSSAttr *ClipboardManag
will become SPCSSAttr *sp_repr_
(no existing function found)
Related branches
Changed in inkscape: | |
assignee: | nobody → Romain (romain2boss) |
tags: | added: clipboard |
Changed in inkscape: | |
status: | Fix Committed → Fix Released |
Thanks for the patch, Romain.
Committed in trunk revision 11153.
If you wish to work further on the merge of sp_import_document, feel free to reopen this report and add the patch.