[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH for Dlang support] d: support api.symbol.prefix and api.token

From: Akim Demaille
Subject: Re: [PATCH for Dlang support] d: support api.symbol.prefix and api.token.prefix
Date: Tue, 29 Sep 2020 18:37:06 +0200

> Le 28 sept. 2020 à 14:30, Adela Vais <> a écrit :
> The D skeleton was not properly supporting them.
> * data/skeletons/d.m4, data/skeletons/lalr1.d: Fix it.
> * tests/ Check it.

This commit also made me realize we were using CALC_EOF uselessly in D, EOF is 

commit a7daa63cb79446a62c99de2d9d9f83cac69528d1
Author: Akim Demaille <>
Date:   Mon Sep 28 19:41:32 2020 +0200

    tests: remove useless prefix for EOF in D
    * tests/ (CALC_EOF): Rename as...
    (EOF): this.
    Since there is no risk of a clash with #define EOF here...

diff --git a/TODO b/TODO
index 8d6de590..72af014b 100644
--- a/TODO
+++ b/TODO
@@ -256,6 +256,9 @@ are.  Keep the same variable names.  If you change the 
wording in one place,
 do it in the others too.  In other words: make sure to keep the
 maintenance *simple* by avoiding any gratuitous difference.
+** CI
+Check when gdc and ldc.
 ** Documentation
 Write documentation about D support in doc/bison.texi.  Imitate the Java
 documentation.  You should be more succinct IMHO.
diff --git a/tests/ b/tests/
index ec8723c9..a96fc98a 100644
--- a/tests/
+++ b/tests/
@@ -606,7 +606,7 @@ class CalcLexer(R) : Lexer
     // EOF.
     if (input.empty)
-      return TokenKind.]AT_TOKEN_PREFIX[CALC_EOF;
+      return TokenKind.]AT_TOKEN_PREFIX[EOF;
     // Numbers.
     if (input.front.isNumber)
@@ -666,7 +666,7 @@ m4_define([_AT_DATA_CALC_Y(d)],
 %printer { fprintf (yyo, "%d", $$); } <ival>;
 /* Bison Declarations */
-%token CALC_EOF 0 ]AT_TOKEN_TRANSLATE_IF([_("end of input")], ["end of 
+%token EOF 0 ]AT_TOKEN_TRANSLATE_IF([_("end of input")], ["end of input"])[
 %token <ival> NUM   "number"
 %type  <ival> exp

reply via email to

[Prev in Thread] Current Thread [Next in Thread]