[vlc-devel] [PATCH 0/5] A 2nd subtitle feature
Jean-Baptiste Kempf
jb at videolan.org
Tue Oct 6 12:35:43 CEST 2015
> This is my first ever contrib to open source,
> so any opinions would be great.
This is way better than the first version, tbh.
But, it would be nice to generalize it.
Moreover, the changes should be re-ordered in logical order: aka, first
core changes, then hotkey.
I don't think you should use a new spu-es variable, but just extend it:
we can have multiple video-tracks at the same time, "video-es", why
would be need multiple spu-es variables?
With my kindest regards,
--
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device
More information about the vlc-devel
mailing list