re2c-general Mailing List for re2c scanner generator

Patches item #1386289, was opened at 2005-12-20 17:11
Message generated for change (Settings changed) made by helly
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616202&aid=1386289&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
Resolution: Rejected
Priority: 1
Submitted By: Antal K (antalk)
Assigned to: Nobody/Anonymous (nobody)
Summary: reorganized sources

Initial Comment:
For some time I have been trying to modify re2c to my
needs. Although I did not finish that, for some time I
will not have time to continue. Basically, I
reorganized the sources (more modular and probably
easier to understand), added some comments (mostly for
doxygen) and implemented some changes that I hope
others might find useful, too. Description of changes
is in README.in, 
together with some comments on the directories. Please
take a look at it.

Note on compiling: for production #define NDEBUG to
turn off assert()'s. Otherwise just ./autogen.sh;
./configure; make; make check; sudo make install;
should work.

The resulting program is named re3c-re2c, it is
compatible to re2c (at least to the extent exercised by
make check), although the code emitted is not
identical. (See also -c flag)

make doxy : runs doxygen (HTML output under doxy/)

Unfortunately cannot upload everything:
    Error: Uploaded file must be >20 and <256000 bytes.

Removed doc/, test/*, examples/* to accomodate.
(Kept test/trailing-var.* and examples/cmmap_re.c)



----------------------------------------------------------------------

Comment By: Marcus Börger (helly)
Date: 2005-12-29 20:27

Message:
Logged In: YES 
user_id=271023

If you would provide a patch against head that has all the 
docs i would happily apply them. Rearranging the source 
however doesn't make much sense and you would loose 
history completley.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616202&aid=1386289&group_id=96864

Patches item #1386289, was opened at 2005-12-20 17:11
Message generated for change (Comment added) made by helly
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616202&aid=1386289&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
>Resolution: Rejected
>Priority: 1
Submitted By: Antal K (antalk)
Assigned to: Nobody/Anonymous (nobody)
Summary: reorganized sources

Initial Comment:
For some time I have been trying to modify re2c to my
needs. Although I did not finish that, for some time I
will not have time to continue. Basically, I
reorganized the sources (more modular and probably
easier to understand), added some comments (mostly for
doxygen) and implemented some changes that I hope
others might find useful, too. Description of changes
is in README.in, 
together with some comments on the directories. Please
take a look at it.

Note on compiling: for production #define NDEBUG to
turn off assert()'s. Otherwise just ./autogen.sh;
./configure; make; make check; sudo make install;
should work.

The resulting program is named re3c-re2c, it is
compatible to re2c (at least to the extent exercised by
make check), although the code emitted is not
identical. (See also -c flag)

make doxy : runs doxygen (HTML output under doxy/)

Unfortunately cannot upload everything:
    Error: Uploaded file must be >20 and <256000 bytes.

Removed doc/, test/*, examples/* to accomodate.
(Kept test/trailing-var.* and examples/cmmap_re.c)



----------------------------------------------------------------------

>Comment By: Marcus Börger (helly)
Date: 2005-12-29 20:27

Message:
Logged In: YES 
user_id=271023

If you would provide a patch against head that has all the 
docs i would happily apply them. Rearranging the source 
however doesn't make much sense and you would loose 
history completley.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616202&aid=1386289&group_id=96864

Feature Requests item #1392966, was opened at 2005-12-29 17:48
Message generated for change (Settings changed) made by helly
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616203&aid=1392966&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
>Assigned to: Marcus Börger (helly)
Summary: homepage should tell what the project does

Initial Comment:
The homepage could use a blurb saying what re2c is, like:

 re2c is a great tool for writing fast and flexible
lexers. Unlike
 other such tools, re2c concentrates solely on
generating efficient
 code for matching regular expressions. Not only does
this singleness
 make re2c more suitable for a wider variety of
applications, it
 allows us to generate scanners which approach
hand-crafted ones in
 terms of size and speed.

(from the debian package)

----------------------------------------------------------------------

>Comment By: Marcus Börger (helly)
Date: 2005-12-29 20:20

Message:
Logged In: YES 
user_id=271023

I wrote something....

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616203&aid=1392966&group_id=96864

Feature Requests item #1392966, was opened at 2005-12-29 08:48
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616203&aid=1392966&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: homepage should tell what the project does

Initial Comment:
The homepage could use a blurb saying what re2c is, like:

 re2c is a great tool for writing fast and flexible
lexers. Unlike
 other such tools, re2c concentrates solely on
generating efficient
 code for matching regular expressions. Not only does
this singleness
 make re2c more suitable for a wider variety of
applications, it
 allows us to generate scanners which approach
hand-crafted ones in
 terms of size and speed.

(from the debian package)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616203&aid=1392966&group_id=96864

Feature Requests item #1307467, was opened at 2005-09-29 02:01
Message generated for change (Comment added) made by helly
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616203&aid=1307467&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
Priority: 7
Submitted By: Thomas Rask (thomas_rask)
Assigned to: Marcus Börger (helly)
Summary: Unicode patch for 0.9.7

Initial Comment:
Wanted to enable 16-bit character support as to enable 
construction of unicode scanners.

We didn't find a simple way to support the EBCDIC 
stuff, so we removed it. Further, for some reason the 
use of bit vectors failed in test, so we disabled them as 
well.

We added a -w option to enable the Unicode processing 
as to keep backward compatibility with 8-bit use (the 
stuff mentioned above still disabled though).


Thomas Rask Thomsen


----------------------------------------------------------------------

>Comment By: Marcus Börger (helly)
Date: 2005-12-28 21:29

Message:
Logged In: YES 
user_id=271023

I regenerated the diff, dropped all changes that don't 
belong in it, added the necessary checks that were 
missing, added documentation and so on.

First file generated looks ok. Windows build no longer 
works and still a lot to work on.

----------------------------------------------------------------------

Comment By: Marcus Börger (helly)
Date: 2005-12-28 00:03

Message:
Logged In: YES 
user_id=271023

Could you please provide an updated patch against head?

The restrictions are no problem as long as they stuff 
still works in 8bit mode and one cannot enable -w and one 
of the mentioned ones. But i can handle that if you prefer.

And please provide a unified diff using 'cvs diff -u'.

----------------------------------------------------------------------

Comment By: Marcus Börger (helly)
Date: 2005-10-26 00:13

Message:
Logged In: YES 
user_id=271023

Could you please provide an updated patch against head?
And please provide a unified diff using 'diff -u'.
The restrictions are no problem as long as they stuff still 
works in 8bit mode and one cannot enable -w and one of the 
mentioned ones.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616203&aid=1307467&group_id=96864

Bugs item #1390174, was opened at 2005-12-25 14:38
Message generated for change (Comment added) made by alder
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1390174&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Closed
Resolution: Fixed
Priority: 5
Submitted By: Alexander Demenchuk (alder)
Assigned to: Marcus Börger (helly)
Summary: re2c cannot accept {0,}

Initial Comment:
It actually crashes if this form is ever used.

I attached a patch and a test case for this bug.

With this branch working it might be possible to
express one of the "factor" alternatives in "parser.y" as:

|	primary close
	{
		switch($2){
		case '*':
		$$ = new CloseVOp($1, 0, -1);
		break;
		case '+':
		$$ = new CloseVOp($1, 1, -1);
		break;
		case '?':
		$$ = new CloseVOp($1, 0, 1);
		break;
		}
	}

IMHO, but might be easier to read.

----------------------------------------------------------------------

>Comment By: Alexander Demenchuk (alder)
Date: 2005-12-27 19:30

Message:
Logged In: YES 
user_id=492233

Well, "{0,}" obviously is "*", but making an explicit
substitude in the scanner (a workaround really) still leaves
a "hole" in CloseVOp::compile. IMHO, it might look nicer if
you do fix that method instead. The fix in my patch, I
think, actually makes that method work correctly for all
acceptable parameters.

----------------------------------------------------------------------

Comment By: Marcus Börger (helly)
Date: 2005-12-27 17:59

Message:
Logged In: YES 
user_id=271023

Fix in the scanner.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1390174&group_id=96864

Feature Requests item #1307467, was opened at 2005-09-29 02:01
Message generated for change (Comment added) made by helly
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616203&aid=1307467&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Priority: 7
Submitted By: Thomas Rask (thomas_rask)
>Assigned to: Marcus Börger (helly)
Summary: Unicode patch for 0.9.7

Initial Comment:
Wanted to enable 16-bit character support as to enable 
construction of unicode scanners.

We didn't find a simple way to support the EBCDIC 
stuff, so we removed it. Further, for some reason the 
use of bit vectors failed in test, so we disabled them as 
well.

We added a -w option to enable the Unicode processing 
as to keep backward compatibility with 8-bit use (the 
stuff mentioned above still disabled though).


Thomas Rask Thomsen


----------------------------------------------------------------------

>Comment By: Marcus Börger (helly)
Date: 2005-12-28 00:03

Message:
Logged In: YES 
user_id=271023

Could you please provide an updated patch against head?

The restrictions are no problem as long as they stuff 
still works in 8bit mode and one cannot enable -w and one 
of the mentioned ones. But i can handle that if you prefer.

And please provide a unified diff using 'cvs diff -u'.

----------------------------------------------------------------------

Comment By: Marcus Börger (helly)
Date: 2005-10-26 00:13

Message:
Logged In: YES 
user_id=271023

Could you please provide an updated patch against head?
And please provide a unified diff using 'diff -u'.
The restrictions are no problem as long as they stuff still 
works in 8bit mode and one cannot enable -w and one of the 
mentioned ones.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616203&aid=1307467&group_id=96864

Bugs item #1390174, was opened at 2005-12-25 20:38
Message generated for change (Comment added) made by helly
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1390174&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Alexander Demenchuk (alder)
Assigned to: Marcus Börger (helly)
Summary: re2c cannot accept {0,}

Initial Comment:
It actually crashes if this form is ever used.

I attached a patch and a test case for this bug.

With this branch working it might be possible to
express one of the "factor" alternatives in "parser.y" as:

|	primary close
	{
		switch($2){
		case '*':
		$$ = new CloseVOp($1, 0, -1);
		break;
		case '+':
		$$ = new CloseVOp($1, 1, -1);
		break;
		case '?':
		$$ = new CloseVOp($1, 0, 1);
		break;
		}
	}

IMHO, but might be easier to read.

----------------------------------------------------------------------

>Comment By: Marcus Börger (helly)
Date: 2005-12-27 23:59

Message:
Logged In: YES 
user_id=271023

Fix in the scanner.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1390174&group_id=96864

Bugs item #1390174, was opened at 2005-12-25 20:38
Message generated for change (Settings changed) made by helly
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1390174&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Alexander Demenchuk (alder)
>Assigned to: Marcus Börger (helly)
Summary: re2c cannot accept {0,}

Initial Comment:
It actually crashes if this form is ever used.

I attached a patch and a test case for this bug.

With this branch working it might be possible to
express one of the "factor" alternatives in "parser.y" as:

|	primary close
	{
		switch($2){
		case '*':
		$$ = new CloseVOp($1, 0, -1);
		break;
		case '+':
		$$ = new CloseVOp($1, 1, -1);
		break;
		case '?':
		$$ = new CloseVOp($1, 0, 1);
		break;
		}
	}

IMHO, but might be easier to read.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1390174&group_id=96864

Bugs item #1390174, was opened at 2005-12-25 14:38
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1390174&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Alexander Demenchuk (alder)
Assigned to: Nobody/Anonymous (nobody)
Summary: re2c cannot accept {0,}

Initial Comment:
It actually crashes if this form is ever used.

I attached a patch and a test case for this bug.

With this branch working it might be possible to
express one of the "factor" alternatives in "parser.y" as:

|	primary close
	{
		switch($2){
		case '*':
		$$ = new CloseVOp($1, 0, -1);
		break;
		case '+':
		$$ = new CloseVOp($1, 1, -1);
		break;
		case '?':
		$$ = new CloseVOp($1, 0, 1);
		break;
		}
	}

IMHO, but might be easier to read.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616200&aid=1390174&group_id=96864

Patches item #1386289, was opened at 2005-12-20 17:11
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616202&aid=1386289&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Antal K (antalk)
Assigned to: Nobody/Anonymous (nobody)
Summary: reorganized sources

Initial Comment:
For some time I have been trying to modify re2c to my
needs. Although I did not finish that, for some time I
will not have time to continue. Basically, I
reorganized the sources (more modular and probably
easier to understand), added some comments (mostly for
doxygen) and implemented some changes that I hope
others might find useful, too. Description of changes
is in README.in, 
together with some comments on the directories. Please
take a look at it.

Note on compiling: for production #define NDEBUG to
turn off assert()'s. Otherwise just ./autogen.sh;
./configure; make; make check; sudo make install;
should work.

The resulting program is named re3c-re2c, it is
compatible to re2c (at least to the extent exercised by
make check), although the code emitted is not
identical. (See also -c flag)

make doxy : runs doxygen (HTML output under doxy/)

Unfortunately cannot upload everything:
    Error: Uploaded file must be >20 and <256000 bytes.

Removed doc/, test/*, examples/* to accomodate.
(Kept test/trailing-var.* and examples/cmmap_re.c)



----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616202&aid=1386289&group_id=96864

Patches item #1362806, was opened at 2005-11-21 14:18
Message generated for change (Settings changed) made by helly
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616202&aid=1362806&group_id=96864

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Antal K (antalk)
>Assigned to: Marcus Börger (helly)
Summary: Addition to man on flag -f

Initial Comment:
patch against 
 $Id: re2c.1.in,v 1.25 2005/11/11 07:39:53 helly Exp $

Notes:

a) the discussion under (2.) in the patch is probably too 
   long.  May need to be cut back a little.

b) I am not sure that I discovered all the needed  
   changes. The "The -f option inhibits declaration of 
   yych and yyaccept." part surpised me. It is 
   probably worth to document it.



----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=616202&aid=1362806&group_id=96864