most of the changes were done trought Katie's namefsck script which
convertes forward class declarations to include directives, however
other fixes here and there were needed as well as some questionable
changes to Q_DECLARE_TYPEINFO() macro calls because they most likely
have to do the namespacing themselfs (QT_BEGIN/END_NAMESPACE, and
probably will be in Katie) meaning that some of the changes may be
temporary and reverted later.
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
the module has some use cases, like avoiding linkage to the
kmediaplayer library which links to MPV, which links to FFmpeg.
and since the module is loaded on demand there should be no change
in terms of resources usage untils it is used, in which case it
will use (roughly) the resources of single MPV instance until
kded4 quits.
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
now when the slider is being dragged the player does not continuesly seek,
it does it only when the slider is released to avoid bad noise during the
quick seeking.
also, the controls will not be hidden now while any of the controls is
being pressed by the user, it's still not perfect because once the user
releases it the controls will be hidden and then (most likely) shown again
(unless no mouse was used for the interactive, e.g. tab for selection and
then space for interaction) but that will require even more connections
to the controls so I'm leaving it as is for now.
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
WId should be unsigned long for Q_WS_X11 (see qwindowdefs.h from Qt4) but
MPV requiers that the type is intptr_t (see options.rst from MPV) yet none
of them works but quintptr does - it's magic.
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
neither the compiler would complain (QVariant can hold custom types via
runtime registration) nor runtime warning about invalid QVariant was issued
for this (because it was casted to something, not sure what but it was the
wrong type)
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
this improves on https://github.com/fluxer/kdelibs/commit/8e9028c4 because
it takes the initial state (so to say) into account (m_visible was not set
on initialization)
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
it is needed in Okular and in KNotify I think so with the overload method
the possibility for MPV API break to affect those application will be
reduced, KAbstractPlayer is ment to be abstraction after all
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
