Why `s1` renamed to `pattern` here? This is not a regex pattern, we're doing string comparison here. I'm not 100% sure that this is an improvement because from the name it looks like a regex pattern while it's actually not.