I should note to readers that this change is because `errorStr` is a parameter of this function, so there was a name collision, which is bad form. Ideally I think we could have more separation between the two names. Maybe this one should be called `sfxErrorStr` for example.