Skip to content

Conversation

@chiefcll
Copy link
Contributor

Since most nodes are just marked with UpdateType.Children - added a top level

if (updateType > 1) // line#1101

To skip all the other checks.

(rest is just formatting and indent)

@wouterlucas
Copy link
Contributor

I cant review this with all the indent changes, could you leave that as is and only diff the changes you made?

@chiefcll
Copy link
Contributor Author

@wouterlucas - I just added this:

if (updateType > UpdateType.Children) { to line 1101. If you want to make the change directly that might be easier as indenting all the code is what causes this diff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants