Strings on parser state stack should not be pointers to strings

If the string is a pointer, then we don't know if we need to delete it or not after releasing it. It's ambiguous.

Due to the fact that a webvtt_string contains in itself only a pointer, there's no real reason to allocate a second pointer for it.

This will help avoid issues like those mentioned in #139.