Csound › Csound - Dev

One for the new parser

Classic

List

Threaded

11 messages Options

Andres Cabrera

One for the new parser

Hi,

This line crashes the new parser:

acarr table3 aphscarr + (amod*kmodindex) + (amod2 *kmodindex2, gisine, 1, 0, 1

Of course there's a parenthesis missing, but still it is crashing. I may not have the latest git version, so please ignore if it has been fixed.

Which makes me think... The parser has been extensively tested for compatibility, but maybe it needs some more testing with wrong input?

Cheers,

Andrés

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Csound-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/csound-devel

Remove Ads

Tito Latini

Re: One for the new parser

Recently I have fixed a rounding problem with table{,i,3}
but if you attach the csd file, I can check it

On Sun, Feb 05, 2012 at 09:57:55PM +0000, Andres Cabrera wrote:

> Hi,
>
>
> This line crashes the new parser:
>
>
> acarr table3 aphscarr + (amod*kmodindex) + (amod2 *kmodindex2, gisine, 1,
> 0, 1
>
>
> Of course there's a parenthesis missing, but still it is crashing. I may
> not have the latest git version, so please ignore if it has been fixed.
>
>
> Which makes me think... The parser has been extensively tested for
> compatibility, but maybe it needs some more testing with wrong input?
>
>
> Cheers,
>
> Andrés

Tito Latini

Re: One for the new parser

In reply to this post by Andres Cabrera

I had understood badly; it's a problem
with the csp_orc_sa_globals_find function.

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Csound-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/csound-devel

Tito Latini

Re: One for the new parser

In reply to this post by Andres Cabrera

The bug is fixed without these rules in yacc

| '(' expr error
| '(' error

two possibilities:
1) remove the rules
2) rewrite the error action

It's very simple but AFK from now. If tomorrow I won't see the fix,
I'll try to write a better error rule.

tito

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Csound-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/csound-devel

Victor Lazzarini

Re: One for the new parser

So we need to wait for these fixes before freezing.

Victor
On 5 Feb 2012, at 22:58, Tito Latini wrote:

> The bug is fixed without these rules in yacc
>
> | '(' expr error
> | '(' error
>
> two possibilities:
> 1) remove the rules
> 2) rewrite the error action
>
> It's very simple but AFK from now. If tomorrow I won't see the fix,
> I'll try to write a better error rule.
>
> tito
>
> ------------------------------------------------------------------------------
> Try before you buy = See our experts in action!
> The most comprehensive online learning library for Microsoft developers
> is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
> Metro Style Apps, more. Free future releases when you subscribe now!
> http://p.sf.net/sfu/learndevnow-dev2
> _______________________________________________
> Csound-devel mailing list
> [hidden email]
> https://lists.sourceforge.net/lists/listinfo/csound-devel

Dr Victor Lazzarini
Senior Lecturer
Dept. of Music
NUI Maynooth Ireland
tel.: +353 1 708 3545
Victor dot Lazzarini AT nuim dot ie

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Csound-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/csound-devel

Tito Latini

Re: One for the new parser

bug fixed, closed the parenthesis

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Csound-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/csound-devel

Andres Cabrera

Re: One for the new parser

Fantastic! Thanks Tito.

Cheers,
Andrés

On Mon, Feb 6, 2012 at 9:23 AM, Tito Latini <[hidden email]> wrote:

bug fixed, closed the parenthesis

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Csound-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/csound-devel

Tito Latini

Re: One for the new parser

I have a fix for avoid segfault with unbalanced defmacro (soon)

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Csound-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/csound-devel

jpff

Re: One for the new parser

Today is planned release day -- GIT was tagged yesterday and today i just
bumbed frpm 5.16.0 to 5.16.1 because of the latest change.
I will be preparing src files soon, followed by SuSE versions
==John ff

> I have a fix for avoid segfault with unbalanced defmacro (soon)
>
>
>

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Csound-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/csound-devel

Tito Latini

Re: One for the new parser

can I apply the fix for malformed defmacros?

On Mon, Feb 06, 2012 at 11:10:50AM -0000, [hidden email] wrote:

> Today is planned release day -- GIT was tagged yesterday and today i just
> bumbed frpm 5.16.0 to 5.16.1 because of the latest change.
> I will be preparing src files soon, followed by SuSE versions
> ==John ff
>
> > I have a fix for avoid segfault with unbalanced defmacro (soon)
> >
> >
> >
>
>
> ------------------------------------------------------------------------------
> Try before you buy = See our experts in action!
> The most comprehensive online learning library for Microsoft developers
> is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
> Metro Style Apps, more. Free future releases when you subscribe now!
> http://p.sf.net/sfu/learndevnow-dev2
> _______________________________________________
> Csound-devel mailing list
> [hidden email]
> https://lists.sourceforge.net/lists/listinfo/csound-devel

Tito Latini

Re: One for the new parser

In reply to this post by jpff

done; it's the last change for today

------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Csound-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/csound-devel