-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Escape throws an error when drawing area with only one point #5941
Comments
I think those are changes I made some time last year to check if an area self intersects. It probably just needs an if() statement to avoid the instance when there is only 1 node (i.e. nodes.length-2 resolves to -1 and hence entity gets set to undefined) Just a bit busy atm, but can have a look at it later |
@maxgrossman what happens if you inspect the node immediately after All the nodes should probably be there in that case, but in your screenshot, |
@quincylvania Out of interest, Presumeably, the underlying |
Yes you should always |
Is it worth forcing it through in the childNodes method and _clone-ing it before returning? |
I don't understand what you're asking. |
As in: Line 107 in c09171d
|
It's extremely hot code, so I'd rather not introduce an extra |
Possibly related: #5911
The text was updated successfully, but these errors were encountered: