Only TIME based, not Frame based, yet.

Almost unused subtitle. No idea though if FPS or 1/10th of sec, Annodex and MPlayer disagree...

Parsing is fine but timing is wrong since it must be multiplied by FPS.

If it's really wanted, functions should de inlined

I wonder why we need this (in libvlc-control) by the way...?

It was not exported from libvlc anyhow.

Turns out it was only used once in the whole tree.

malloc(0) can legally be NULL (or not unique).

I don't have the courage to replace all the occurences in the code.

Also sizeof(byte_t)^Wsizeof(uint8_t) can only ever be 1, if it is
defined. It's not defined, if char is more than 8 bits, and uint8_t is
not provided, but then VLC will not compile in the first place.

by using VLC_ADD_PLUGIN instead of VLC_ADD_BUILTINS.

Please check for potential regressions.