Csound › Csound - Dev

text format

Classic

List

Threaded

9 messages Options

Tito Latini

text format

I have noticed that the text format where the lines finish
only with CR (\r) doesn't work well with the preprocessor
in the new parser, because

"\r" { }

and

while ((c = input(yyscanner)) != '\n'); // for skip

However the old parser and the score (sread.c) consider the eol with only CR.

We have 2 possibilities:

1 - only the text formats that finish with CR LF or LF are correct
2 - to correct the preprocessor

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

Remove Ads

Victor Lazzarini

Re: text format

Tito,

it looks like we need option 2).

Regards

Victor
On 27 Jan 2012, at 10:38, Tito Latini wrote:

> I have noticed that the text format where the lines finish
> only with CR (\r) doesn't work well with the preprocessor
> in the new parser, because
>
> "\r" { }
>
> and
>
> while ((c = input(yyscanner)) != '\n'); // for skip
>
> However the old parser and the score (sread.c) consider the eol with only CR.
>
> We have 2 possibilities:
>
> 1 - only the text formats that finish with CR LF or LF are correct
> 2 - to correct the preprocessor
>
> 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

Andres Cabrera

Re: text format

Hi,

I agree with Victor. There are many old orchestras produced in
MacCsound that end in \r, so it's worth keeping compatibility with
those.

Cheers,
Andrés

On Fri, Jan 27, 2012 at 11:10 AM, Victor Lazzarini
<[hidden email]> wrote:

> Tito,
>
> it looks like we need option 2).
>
> Regards
>
> Victor
> On 27 Jan 2012, at 10:38, Tito Latini wrote:
>
>> I have noticed that the text format where the lines finish
>> only with CR (\r) doesn't work well with the preprocessor
>> in the new parser, because
>>
>> "\r" { }
>>
>> and
>>
>> while ((c = input(yyscanner)) != '\n'); // for skip
>>
>> However the old parser and the score (sread.c) consider the eol with only CR.
>>
>> We have 2 possibilities:
>>
>> 1 - only the text formats that finish with CR LF or LF are correct
>> 2 - to correct the preprocessor
>>
>> 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

------------------------------------------------------------------------------
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: text format

I guess so -- strange who no one seemed to care when I made these changes,
but now it is assumed.

> Hi,
>
> I agree with Victor. There are many old orchestras produced in
> MacCsound that end in \r, so it's worth keeping compatibility with
> those.
>
> Cheers,
> AndrÃ©s
>
> On Fri, Jan 27, 2012 at 11:10 AM, Victor Lazzarini
> <[hidden email]> wrote:
>> Tito,
>>
>> it looks like we need option 2).
>>
>> Regards
>>
>> Victor
>> On 27 Jan 2012, at 10:38, Tito Latini wrote:
>>
>>> I have noticed that the text format where the lines finish
>>> only with CR (\r) doesn't work well with the preprocessor
>>> in the new parser, because
>>>
>>> Â "\r" { }
>>>
>>> and
>>>
>>> Â while ((c = input(yyscanner)) != '\n'); // for skip
>>>
>>> However the old parser and the score (sread.c) consider the eol with
>>> only CR.
>>>
>>> We have 2 possibilities:
>>>
>>> Â 1 - only the text formats that finish with CR LF or LF are correct
>>> Â 2 - to correct the preprocessor
>>>
>>> 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
>
> ------------------------------------------------------------------------------
> 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: text format

It seems only a problem with `#include'.

In my simple test, test.csd is CR LF and the included file is CR.
The included file contains

itest1 = 1
itest2 = 2
itest3 = 3

and I get

syntax error ...
>>>itest1 = 1itest2 <<<

I have resolved removing the "r" rule and replacing the "\n" rule with

NEWLINE (\n|\r?)

I want to make some other tests before adding the change.

------------------------------------------------------------------------------
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: text format

In reply to this post by jpff

Sorry, it's the rule

NEWLINE (\n|\r\n?)

------------------------------------------------------------------------------
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: text format

In reply to this post by jpff

I have modified the `comment' function because there was a
segfault with the comments in an included file; now it seems ok

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

jpff

Re: text format

were the changes for \r in the lexer or preprocessor?

> I have modified the `comment' function because there was a
> segfault with the comments in an included file; now it seems ok
>
> 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
>
>
>

Tito Latini

Re: text format

> were the changes for \r in the lexer or preprocessor?

preprocessor (csound_pre.lex); the diff is in git

------------------------------------------------------------------------------
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