[vlc-devel] [PATCH] FIX: Dshow "none" device open (#2201). Some cleaning

basos g noxelia at gmail.com
Tue Feb 17 13:51:07 CET 2009


2009/2/14 basos g <noxelia at gmail.com>:
> 2009/2/12 basos g <noxelia at gmail.com>:
>> 2009/2/11 Rémi Denis-Courmont <rem at videolan.org>:
>>> Le mercredi 11 février 2009 16:15:48 basos g, vous avez écrit :
>>>> These 2 patches address the issue in dshow as well as some other fixes
>>>>
>>>> -Fix dshow "none" deivice be recognized as none
>>>>  Note that another patch concerning var_optionParse string
>>>>  dequoting should be applied for this to be fixed
>>>>  ( the concept is not to provide quoted strings to dshow options )
>>>>  -Account for the case when BOTH devices could not be opened
>>>>    (file an error, instead of seeming to play)
>>>>  -Cleaned a little
>>>>  -Added dbg and error msgs
>>>>  -Added coments
>>>
>>> I am not against removing quotes there, but we have to be consistent.
>>> Discarding quotes is wrong. There has to be a way to pass a value that does
>>> include quotes.
>>>
>>> --
>>> Rémi Denis-Courmont
>>> http://git.remlab.net/cgi-bin/gitweb.cgi?p=vlc-courmisch.git;a=summary
>>> _______________________________________________
>>> vlc-devel mailing list
>>> To unsubscribe or modify your subscription options:
>>> http://mailman.videolan.org/listinfo/vlc-devel
>>>
>>
>> The point is that for the dshow we don't need quotes.
>> Do we need them elsewhere ?
>> I think quotes mean: take the hole following string including spaces.
>> As long as the delimiting is done correctly i am not sure that we
>> actualy need the quotes inside psz_data.
>>
>> eg.
>>
>> :adev="Create sound blaster" :vdev="vid1" :type=pal etc
>>
>> here psz_value for adev should be
>> Creative sound blaster
>> and not
>> "Creative sound blaster"
>>
>
>
> So whats the point about quotes ? Do we need them ( e.g. is there a
> case that in-string quotes are NEEDED ?
>
> Note that these 2 patches fix a (small) bug and i don't expect that
> you want it to be ignored ?
>

Someone to check this ?
It does not seem to break sthing.

I re-sent the var_optionParse patch with a more clean way (str_dequote
help funtion was added).

NOTE that i could make this patch like str_quote(ing) all vars from
dshow filter.
But i doubt that any other input option user module actually needs
quotes inside psz_data. So i propose to dequote every input string..
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0010-var_OptionParse-was-made-quote-aware.patch
Type: text/x-diff
Size: 2094 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090217/e95668f8/attachment.patch>


More information about the vlc-devel mailing list