activex: setting default aspect ratio is legal
authorAnthony Loiseau <thannoy@actech-innovation.com>
Fri, 19 Jun 2009 15:09:45 +0000 (17:09 +0200)
committerRémi Denis-Courmont <remi@remlab.net>
Mon, 29 Jun 2009 15:40:40 +0000 (18:40 +0300)
Empty parameter means "default/guess".

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>
projects/activex/vlccontrol2.cpp

index c0de083..8fcacd1 100644 (file)
@@ -2076,9 +2076,6 @@ STDMETHODIMP VLCVideo::put_aspectRatio(BSTR aspect)
     if( NULL == aspect )
         return E_POINTER;
 
-    if( 0 == SysStringLen(aspect) )
-        return E_INVALIDARG;
-
     libvlc_media_player_t *p_md;
     HRESULT hr = _p_instance->getMD(&p_md);
     if( SUCCEEDED(hr) )