[ 
http://dev.sourcefabric.org/browse/LS-573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18029#action_18029
 ] 

Matija Kolarić commented on LS-573:
-----------------------------------

I do not believe that separating arguments from variable names makes sense, 
since in functions arguments become variables.
I have no ocaml experience, but REGEX solution seems simple.
In docs it says that this is the label REGEX:
[A-Za-z_][A-Za-z0-9_.']*
I believe it can be changed to
[A-Za-z0-9_]*[A-Za-z_][A-Za-z0-9_.']*
This should provide the backward compatibility and should not interfere with 
floats or integers.
[A-Za-z0-9_.]*[A-Za-z_][A-Za-z0-9_.']* or [A-Za-z0-9_']*[A-Za-z_][A-Za-z0-9_.']*
might be pushing it too far.
I can take a look at the code myself if necessary , please point me to the file.


> Ladspa compatibility issue
> --------------------------
>
>                 Key: LS-573
>                 URL: http://dev.sourcefabric.org/browse/LS-573
>             Project: Liquidsoap
>          Issue Type: Bug
>          Components: Liquidsoap
>    Affects Versions: 1.0 beta
>         Environment: Ubuntu Desktop
>            Reporter: Matija Kolarić
>
> Some LADSPA such as ladspa.mbeq use attribute names that start with digits ( 
> '100hz_gain' ).
> liquidsoap -h ladspa.mbeq shows that attribute names with digits may be used. 
> And they can not.
> Anyway, there is several options I can see:
> 1. allow digits to be used at the beginning of attribute names
> 2. allow some kind of workaround and document it
> 3. disable plugins such as this one (please not)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://dev.sourcefabric.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1
_______________________________________________
Savonet-devl mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/savonet-devl

Répondre à