typo on the source code for connectAdvanced?

[hkjpotato]

Hi, in the source of connectAdvanced line 255 it seems there is a typo in the comment

//Doing it this way vs having a permanent componentDidMount that does
// a boolean check every time avoids an extra method call most of the time, resulting
 // in some perf boost.

How to do a boolean check in the 'componentDidMount' for the succesive update since it will only be called once in the lifecycle? Could you explain how to use a permanent componentDidMount method that does the same thing like the current temporal 'componentDidUpdate' approach?

Thank you.

[jimbolla]

That was probably supposed to be componentDidUpdate.

[timdorr]

Yeah, I'll go change it.

[hkjpotato]

Thank you. It makes sense now. I guess the logic here is that only when an update triggered by its onStateChange needs to do the notifyNestedSubs. If we implement a permanent componentDidUpdate then if it is triggered by a parent's re-render it should have a boolean to stop it from doing the notification.