Commit e3965395 authored by JP Dinger's avatar JP Dinger Committed by Jean-Paul Saman

Add isPluginRunning() and getPrivate<>(), and use them instead of the raw code.

Review: Removed debug leftovers that where #if'ed and reverted none exiting p_plugin->getMP() changes.
Signed-off-by: Jean-Paul Saman's avatarJean-Paul Saman <jean-paul.saman@m2x.nl>
parent 5fae4f45
...@@ -95,7 +95,7 @@ RuntimeNPObject::InvokeResult ...@@ -95,7 +95,7 @@ RuntimeNPObject::InvokeResult
LibvlcRootNPObject::getProperty(int index, NPVariant &result) LibvlcRootNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
switch( index ) switch( index )
{ {
...@@ -176,7 +176,7 @@ RuntimeNPObject::InvokeResult LibvlcRootNPObject::invoke(int index, ...@@ -176,7 +176,7 @@ RuntimeNPObject::InvokeResult LibvlcRootNPObject::invoke(int index,
const NPVariant *args, uint32_t argCount, NPVariant &result) const NPVariant *args, uint32_t argCount, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -232,9 +232,9 @@ RuntimeNPObject::InvokeResult ...@@ -232,9 +232,9 @@ RuntimeNPObject::InvokeResult
LibvlcAudioNPObject::getProperty(int index, NPVariant &result) LibvlcAudioNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -283,9 +283,9 @@ RuntimeNPObject::InvokeResult ...@@ -283,9 +283,9 @@ RuntimeNPObject::InvokeResult
LibvlcAudioNPObject::setProperty(int index, const NPVariant &value) LibvlcAudioNPObject::setProperty(int index, const NPVariant &value)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -315,8 +315,7 @@ LibvlcAudioNPObject::setProperty(int index, const NPVariant &value) ...@@ -315,8 +315,7 @@ LibvlcAudioNPObject::setProperty(int index, const NPVariant &value)
libvlc_media_player_t *p_md = libvlc_media_player_t *p_md =
libvlc_playlist_get_media_player(p_plugin->getVLC(), &ex); libvlc_playlist_get_media_player(p_plugin->getVLC(), &ex);
RETURN_ON_EXCEPTION(this,ex); RETURN_ON_EXCEPTION(this,ex);
libvlc_audio_set_track(p_md, libvlc_audio_set_track(p_md, numberValue(value), &ex);
numberValue(value), &ex);
libvlc_media_player_release(p_md); libvlc_media_player_release(p_md);
RETURN_ON_EXCEPTION(this,ex); RETURN_ON_EXCEPTION(this,ex);
return INVOKERESULT_NO_ERROR; return INVOKERESULT_NO_ERROR;
...@@ -354,9 +353,9 @@ LibvlcAudioNPObject::invoke(int index, const NPVariant *args, ...@@ -354,9 +353,9 @@ LibvlcAudioNPObject::invoke(int index, const NPVariant *args,
uint32_t argCount, NPVariant &result) uint32_t argCount, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -409,9 +408,9 @@ RuntimeNPObject::InvokeResult ...@@ -409,9 +408,9 @@ RuntimeNPObject::InvokeResult
LibvlcInputNPObject::getProperty(int index, NPVariant &result) LibvlcInputNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -504,9 +503,9 @@ RuntimeNPObject::InvokeResult ...@@ -504,9 +503,9 @@ RuntimeNPObject::InvokeResult
LibvlcInputNPObject::setProperty(int index, const NPVariant &value) LibvlcInputNPObject::setProperty(int index, const NPVariant &value)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -608,7 +607,7 @@ RuntimeNPObject::InvokeResult ...@@ -608,7 +607,7 @@ RuntimeNPObject::InvokeResult
LibvlcMessageNPObject::getProperty(int index, NPVariant &result) LibvlcMessageNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
switch( index ) switch( index )
{ {
...@@ -711,6 +710,8 @@ LibvlcMessageIteratorNPObject::LibvlcMessageIteratorNPObject(NPP instance, ...@@ -711,6 +710,8 @@ LibvlcMessageIteratorNPObject::LibvlcMessageIteratorNPObject(NPP instance,
RuntimeNPObject(instance, aClass), RuntimeNPObject(instance, aClass),
_p_iter(NULL) _p_iter(NULL)
{ {
// XXX FIXME use _instance or instance in this method?
/* is plugin still running */ /* is plugin still running */
if( instance->pdata ) if( instance->pdata )
{ {
...@@ -744,9 +745,9 @@ RuntimeNPObject::InvokeResult ...@@ -744,9 +745,9 @@ RuntimeNPObject::InvokeResult
LibvlcMessageIteratorNPObject::getProperty(int index, NPVariant &result) LibvlcMessageIteratorNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
switch( index ) switch( index )
{ {
case ID_messageiterator_hasNext: case ID_messageiterator_hasNext:
...@@ -789,9 +790,9 @@ LibvlcMessageIteratorNPObject::invoke(int index, const NPVariant *args, ...@@ -789,9 +790,9 @@ LibvlcMessageIteratorNPObject::invoke(int index, const NPVariant *args,
uint32_t argCount, NPVariant &result) uint32_t argCount, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -850,9 +851,9 @@ RuntimeNPObject::InvokeResult ...@@ -850,9 +851,9 @@ RuntimeNPObject::InvokeResult
LibvlcMessagesNPObject::getProperty(int index, NPVariant &result) LibvlcMessagesNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
switch( index ) switch( index )
{ {
case ID_messages_count: case ID_messages_count:
...@@ -897,9 +898,9 @@ LibvlcMessagesNPObject::invoke(int index, const NPVariant *args, ...@@ -897,9 +898,9 @@ LibvlcMessagesNPObject::invoke(int index, const NPVariant *args,
uint32_t argCount, NPVariant &result) uint32_t argCount, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -972,9 +973,9 @@ RuntimeNPObject::InvokeResult ...@@ -972,9 +973,9 @@ RuntimeNPObject::InvokeResult
LibvlcLogNPObject::getProperty(int index, NPVariant &result) LibvlcLogNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -1016,9 +1017,9 @@ RuntimeNPObject::InvokeResult ...@@ -1016,9 +1017,9 @@ RuntimeNPObject::InvokeResult
LibvlcLogNPObject::setProperty(int index, const NPVariant &value) LibvlcLogNPObject::setProperty(int index, const NPVariant &value)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -1083,9 +1084,9 @@ RuntimeNPObject::InvokeResult ...@@ -1083,9 +1084,9 @@ RuntimeNPObject::InvokeResult
LibvlcPlaylistItemsNPObject::getProperty(int index, NPVariant &result) LibvlcPlaylistItemsNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -1125,9 +1126,9 @@ LibvlcPlaylistItemsNPObject::invoke(int index, const NPVariant *args, ...@@ -1125,9 +1126,9 @@ LibvlcPlaylistItemsNPObject::invoke(int index, const NPVariant *args,
uint32_t argCount, NPVariant &result) uint32_t argCount, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -1191,9 +1192,9 @@ RuntimeNPObject::InvokeResult ...@@ -1191,9 +1192,9 @@ RuntimeNPObject::InvokeResult
LibvlcPlaylistNPObject::getProperty(int index, NPVariant &result) LibvlcPlaylistNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -1267,9 +1268,9 @@ LibvlcPlaylistNPObject::invoke(int index, const NPVariant *args, ...@@ -1267,9 +1268,9 @@ LibvlcPlaylistNPObject::invoke(int index, const NPVariant *args,
uint32_t argCount, NPVariant &result) uint32_t argCount, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -1607,9 +1608,9 @@ RuntimeNPObject::InvokeResult ...@@ -1607,9 +1608,9 @@ RuntimeNPObject::InvokeResult
LibvlcVideoNPObject::getProperty(int index, NPVariant &result) LibvlcVideoNPObject::getProperty(int index, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -1690,9 +1691,9 @@ RuntimeNPObject::InvokeResult ...@@ -1690,9 +1691,9 @@ RuntimeNPObject::InvokeResult
LibvlcVideoNPObject::setProperty(int index, const NPVariant &value) LibvlcVideoNPObject::setProperty(int index, const NPVariant &value)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
...@@ -1816,9 +1817,9 @@ LibvlcVideoNPObject::invoke(int index, const NPVariant *args, ...@@ -1816,9 +1817,9 @@ LibvlcVideoNPObject::invoke(int index, const NPVariant *args,
uint32_t argCount, NPVariant &result) uint32_t argCount, NPVariant &result)
{ {
/* is plugin still running */ /* is plugin still running */
if( _instance->pdata ) if( isPluginRunning() )
{ {
VlcPlugin* p_plugin = reinterpret_cast<VlcPlugin*>(_instance->pdata); VlcPlugin* p_plugin = getPrivate<VlcPlugin>();
libvlc_exception_t ex; libvlc_exception_t ex;
libvlc_exception_init(&ex); libvlc_exception_init(&ex);
......
...@@ -127,6 +127,15 @@ protected: ...@@ -127,6 +127,15 @@ protected:
bool returnInvokeResult(InvokeResult result); bool returnInvokeResult(InvokeResult result);
bool isPluginRunning()
{
return _instance->pdata;
}
template<class T> T *getPrivate()
{
return reinterpret_cast<T *>(_instance->pdata);
}
NPP _instance; NPP _instance;
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment