Implicit fallthrough intended?

[r-barnes]

Is the implicit fallthrough at this line intentional, or should it be a break?

[earwig]

It’s intended. This is an error handling case where one of so or sc failed to allocate so we fall through to clean up the closing tag and propagate the error by returning NULL.