zcy
/
solvespace
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't regenerate the entire sketch in AddRequest(). 

This is unnecessary and leads to huge slowdowns on large sketches.
E.g. this speeds up import of a certain large DXF from ~60s to ~3s.
pull/66/head
whitequark 2016-10-10 20:33:02 +00:00
parent 0e66eafe5a
commit 2371068d01
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/mouse.cpp
View File

@ -866,7 +866,7 @@ hRequest GraphicsWindow::AddRequest(Request::Type type, bool rememberForUndo) {
// place this request's entities where the mouse is can do so. But // place this request's entities where the mouse is can do so. But
// we mustn't try to solve until reasonable values have been supplied // we mustn't try to solve until reasonable values have been supplied
// for these new parameters, or else we'll get a numerical blowup. // for these new parameters, or else we'll get a numerical blowup.
SS.GenerateAll(SolveSpaceUI::Generate::REGEN); r.Generate(&SK.entity, &SK.param);
SS.MarkGroupDirty(r.group); SS.MarkGroupDirty(r.group);
return r.h; return r.h;
} }