C target fixes for Antlr 3.5

[Kieran Simpson]

As mentioned in http://www.antlr3.org/pipermail/antlr-interest/2012-September/045368.html there exists a nasty bug in the C target with respect to backtracking and labels (see also the thread at http://www.antlr3.org/pipermail/antlr-interest/2012-March/044216.html).

In the Antlr 3.5 Release Notes (http://www.antlr.org/wiki/display/ANTLR3/ANTLR+3.5+Release+Notes) there is a mention of Sam's fix for the issue "(Sam) Fix labels in synpreds" which provides a starting point for moving the fix over to the C target (http://www.antlr3.org/pipermail/antlr-interest/2012-March/044257.html talks about the Git commit some more).

Since the Antlr mailing list has been moved over to Google Groups I wanted to reraise it as a bug, and politely enquire when a fix might appear?

Cheers,

[Jim Idle]

I thought that someone else had already fixed that? It was a bug in the stringtemplates that affected all targets. 

Sorry that it has yet to be fixed - in my mind it already was fixed. I will see if I can find time this week to change that, but I am travelling pretty much all the time these days, so it is difficult. Anyone that has fixed this is welcome to submit a patch of course.

Jim

--
You received this message because you are subscribed to the Google Groups "antlr-discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to antlr-discussi...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
 
 

[Kieran Simpson]

As far as I'm aware, Sam did fix it for Java (https://github.com/antlr/antlr3/commit/db6adaefb12a1901606f81c9c8931aa07d12949f), and I gave a crack at porting it over to the C target (https://github.com/kierans/antlr3) but I couldn't get it to work.  You're more than welcome to take what I've done to use it as a starting point if it will help you Jim.

Cheers,

[Kieran Simpson]

Jim,
  Has there been any updates on this?

Cheers,

[Mike Lischke]

Kieran,

Unfortunately code changes in the C target seem not to happen anymore. I have 2 merge requests on GitHub for half a year without any sign that somebody would care.

Jim,
  Has there been any updates on this?

Cheers,

On Wednesday, 27 February 2013 10:06:37 UTC+11, Kieran Simpson wrote:

As far as I'm aware, Sam did fix it for Java (https://github.com/antlr/antlr3/commit/db6adaefb12a1901606f81c9c8931aa07d12949f), and I gave a crack at porting it over to the C target (https://github.com/kierans/antlr3) but I couldn't get it to work.  You're more than welcome to take what I've done to use it as a starting point if it will help you Jim.

Cheers,

On Tuesday, 26 February 2013 18:13:58 UTC+11, Jim Idle wrote:

I thought that someone else had already fixed that? It was a bug in the stringtemplates that affected all targets. 

Sorry that it has yet to be fixed - in my mind it already was fixed. I will see if I can find time this week to change that, but I am travelling pretty much all the time these days, so it is difficult. Anyone that has fixed this is welcome to submit a patch of course.

Jim

On Tue, Feb 26, 2013 at 3:00 PM, Kieran Simpson <kiera...@gmail.com> wrote:

As mentioned in http://www.antlr3.org/pipermail/antlr-interest/2012-September/045368.html there exists a nasty bug in the C target with respect to backtracking and labels (see also the thread at http://www.antlr3.org/pipermail/antlr-interest/2012-March/044216.html).

In the Antlr 3.5 Release Notes (http://www.antlr.org/wiki/display/ANTLR3/ANTLR+3.5+Release+Notes) there is a mention of Sam's fix for the issue "(Sam) Fix labels in synpreds" which provides a starting point for moving the fix over to the C target (http://www.antlr3.org/pipermail/antlr-interest/2012-March/044257.html talks about the Git commit some more).

Since the Antlr mailing list has been moved over to Google Groups I wanted to reraise it as a bug, and politely enquire when a fix might appear?

Mike

-- 

www.soft-gems.net

[Terence Parr]

Hi Guys, only Jim can approve these changes and he has been too busy to go through them.

Ter

--
You received this message because you are subscribed to the Google Groups "antlr-discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to antlr-discussi...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
 
 

--

Dictation in use. Please excuse homophones, malapropisms, and nonsense.