indentation is screwed up here as well -- This code has also been taken directly from the buffer, and there is no context explaining what  everything means.

Questions: Why are we testing WEBVTT_FAILED()? What (specific webvtt_status values) can webvtt_string_putc() return? These are the important things, we don't need the rest of it.