This is a cosmetic change at this point as _() invokes dgettext(). This
is just a small step toward supporting out-of-tree plugins. These will
need to use their own text domain.

Fix #1590 - patch by jj

Better yet try to parse it directly...

...to help identify possible issues
Signed-off-by: Rémi Denis-Courmont <rem@videolan.org>

Squashed with VLC boolean updates.

(See also http://forum.videolan.org/viewtopic.php?f=7&t=42590).
Signed-off-by: Rémi Denis-Courmont <rem@videolan.org>

Squashed VLC_TRUE fix

Fix decoding of teletext in ps files (the zvbi decoder expects the full PES packet, I haven't been able to test the telx.c decoder yet as I don't have any ps file with teletext subtitles available).

Code cleanup without semantics changes
Resets the reference count when returning a previously allocated picture
Check error path

In the meantime, another module (like the mosaic_bridge hack) could call it.
Instead, increase the reference count to be sure it won't be destroyed.

Some cleaning too.

Could go badly wrong on platforms without atomic 64-bits access

In this case, volatile was both incorrect (volatile does nothing about
thread synchronization) and not required (there is an explicit lock).