b4n commented on this pull request.
if (vStringLength (name) > 0) { + tagEntryInfo *parent = getEntryOfNestingLevel (nl);
It's incorrect, because I stated in my commend above, we still have to call `getNestingLevel()` because it's that function that calls `nestingLevelsPop()` when appropriate. Just removing the call removes this side effect, resulting in never removing nest levels.
This is pretty much exactly what I tried to warn about in the comment above when sating that *"have a tendency to remove [unused variables]"*, suggesting it's not so simple here. What we can do is call the function without storing its value, and possible rename it to be clearer.