It is going to make code messy doing this at all. As I've stated on irc, it's much more readable if it can be treated as a boolean condition

```C
if( find_bytes( ... ) ) {
}
```

vs

```C
if( find_bytes( ... ) == WEBVTT_SUCCESS ) {
}
```
vs
```C
if( !WEBVTT_FAILED( find_bytes( ... ) ) ) {
}
```

My preference is the first option