assets/parser.d.erb

@@ -11,7 +11,7 @@ import std.stdio;
 
 class <%= @classname %>
 {
-    enum : uint
+    enum
     {
 <% @grammar.tokens.each_with_index do |token, index| %>
         TOKEN_<%= token.code_name %> = <%= index %>,
@@ -36,40 +36,23 @@ class <%= @classname %>
 
     static class Decoder
     {
-        struct Result
+        enum
         {
-            enum : ubyte
-            {
-                SUCCESS,
-                EOF,
-                DECODE_ERROR,
-            }
-            ubyte result;
-            alias result this;
-            uint code_point;
-            uint code_point_length;
-
-            static Result success(uint code_point, uint code_point_length)
-            {
-                return Result(SUCCESS, code_point, code_point_length);
-            }
-
-            static Result eof()
-            {
-                return Result(EOF);
-            }
-
-            static Result decode_error()
-            {
-                return Result(DECODE_ERROR);
-            }
+            CODE_POINT_INVALID = 0xFFFFFFFE,
+            CODE_POINT_EOF = 0xFFFFFFFF,
         }
 
-        static Result decode_code_point(string input)
+        struct DecodedCodePoint
+        {
+            uint code_point;
+            uint code_point_length;
+        }
+
+        static DecodedCodePoint decode_code_point(string input)
         {
             if (input.length == 0u)
             {
-                return Result.eof();
+                return DecodedCodePoint(CODE_POINT_EOF, 0u);
             }
             char c = input[0];
             uint code_point;
@@ -109,11 +92,11 @@ class <%= @classname %>
                 }
                 else
                 {
-                    return Result.decode_error();
+                    return DecodedCodePoint(CODE_POINT_INVALID, 0u);
                 }
                 if (input.length <= following_bytes)
                 {
-                    return Result.decode_error();
+                    return DecodedCodePoint(CODE_POINT_INVALID, 0u);
                 }
                 code_point_length = following_bytes + 1u;
                 for (size_t i = 0u; i < following_bytes; i++)
@@ -121,12 +104,12 @@ class <%= @classname %>
                     char b = input[i + 1u];
                     if ((b & 0xC0u) != 0x80u)
                     {
-                        return Result.decode_error();
+                        return DecodedCodePoint(CODE_POINT_INVALID, 0u);
                     }
                     code_point = (code_point << 6u) | (b & 0x3Fu);
                 }
             }
-            return Result.success(code_point, code_point_length);
+            return DecodedCodePoint(code_point, code_point_length);
         }
     }
 
@@ -255,13 +238,13 @@ class <%= @classname %>
             for (;;)
             {
                 auto decoded = Decoder.decode_code_point(m_input[(m_input_position + attempt_match_info.length)..(m_input.length)]);
-                if (decoded == Decoder.Result.DECODE_ERROR)
+                if (decoded.code_point == Decoder.CODE_POINT_INVALID)
                 {
                     lt.token = _TOKEN_DECODE_ERROR;
                     return lt;
                 }
                 bool lex_continue = false;
-                if (decoded != Decoder.Result.EOF)
+                if (decoded.code_point != Decoder.CODE_POINT_EOF)
                 {
                     uint dest = transition(current_state, decoded.code_point);
                     if (dest != cast(uint)-1)

spec/test_d_lexer.d

@@ -8,29 +8,29 @@ int main()
 
 unittest
 {
-    alias Result = Testparser.Decoder.Result;
-    Result result;
+    alias DCP = Testparser.Decoder.DecodedCodePoint;
+    DCP dcp;
 
-    result = Testparser.Decoder.decode_code_point("5");
-    assert(result == Result.success('5', 1u));
+    dcp = Testparser.Decoder.decode_code_point("5");
+    assert(dcp == DCP('5', 1u));
 
-    result = Testparser.Decoder.decode_code_point("");
-    assert(result == Result.eof());
+    dcp = Testparser.Decoder.decode_code_point("");
+    assert(dcp == DCP(Testparser.Decoder.CODE_POINT_EOF, 0u));
 
-    result = Testparser.Decoder.decode_code_point("\xC2\xA9");
-    assert(result == Result.success(0xA9u, 2u));
+    dcp = Testparser.Decoder.decode_code_point("\xC2\xA9");
+    assert(dcp == DCP(0xA9u, 2u));
 
-    result = Testparser.Decoder.decode_code_point("\xf0\x9f\xa7\xa1");
-    assert(result == Result.success(0x1F9E1, 4u));
+    dcp = Testparser.Decoder.decode_code_point("\xf0\x9f\xa7\xa1");
+    assert(dcp == DCP(0x1F9E1, 4u));
 
-    result = Testparser.Decoder.decode_code_point("\xf0\x9f\x27");
-    assert(result == Result.decode_error());
+    dcp = Testparser.Decoder.decode_code_point("\xf0\x9f\x27");
+    assert(dcp == DCP(Testparser.Decoder.CODE_POINT_INVALID, 0u));
 
-    result = Testparser.Decoder.decode_code_point("\xf0\x9f\xa7\xFF");
-    assert(result == Result.decode_error());
+    dcp = Testparser.Decoder.decode_code_point("\xf0\x9f\xa7\xFF");
+    assert(dcp == DCP(Testparser.Decoder.CODE_POINT_INVALID, 0u));
 
-    result = Testparser.Decoder.decode_code_point("\xfe");
-    assert(result == Result.decode_error());
+    dcp = Testparser.Decoder.decode_code_point("\xfe");
+    assert(dcp == DCP(Testparser.Decoder.CODE_POINT_INVALID, 0u));
 }
 
 unittest