This is based on some work by Martell, and modified by /me
It's not yet perfectly clean, but that will come in the next commits

Signed-off-by: Felix Paul Kühne <fkuehne@videolan.org>

Close #14569

Signed-off-by: Samuel Martin <s.martin49@gmail.com>
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

If old opencv1-style legacy include directory is available, this change
becomes purely cosmetic (maybe will compile a bit faster).
It becomes an FTBFS fix when opencv1-style include directory is missing
(i.e. from opencv-3.0).
Signed-off-by: Samuel Martin <s.martin49@gmail.com>
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

This fixes a regression brought by 9f85beee

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

the subpicture buffer is stored in a picture_t with a d3d_quad_t in place of picture_sys_t
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

If an object is created after the parent is already killed,
ObjectKillChildren() could not have killed the new object. So the new
object needs to inherit the alive flag from its parent instead.

(ref: MSDN KB 287087)

 - do not pass NULL HIML on error,
 - clean up COM on error,
 - fix hr variable shadowing.

Not fixed: leak of the task bar and image list.

The value never changes anyway. This fixes non-interactive inputs.

Do not assume the title number is small; allocate large enough buffer.