Comment support #84

Closed
opened 2023-01-12 23:01:02 +00:00 by deavmi · 3 comments
Owner

What is this?

We need to add support for comments, we need to look at the following:

  1. Single line comments
  2. Multi-line comments

I think we should implement this at the lexer level, in the case of a single line comment we want to effectively have it translate to one token:

// Hello world

Should be ["// Hello world"] and not ["//", "Hello", "World"]

### What is this? We need to add support for comments, we need to look at the following: 1. Single line comments 2. Multi-line comments I think we should implement this at the lexer level, in the case of a single line comment we want to effectively have it translate to one token: ``` // Hello world ``` Should be `["// Hello world"]` and **not** `["//", "Hello", "World"]`
deavmi added the
lexer
label 2023-01-12 23:02:17 +00:00
deavmi self-assigned this 2023-01-12 23:02:20 +00:00
deavmi added this to the Lexer project 2023-01-12 23:02:24 +00:00
deavmi added this to the Basics milestone 2023-01-12 23:02:28 +00:00
gusmeyer was assigned by deavmi 2023-07-16 19:44:21 +01:00
deavmi added the due date 2023-07-30 2023-07-16 19:44:59 +01:00
deavmi added reference feature/lexer_improvements 2023-07-16 19:45:04 +01:00
deavmi added a new dependency 2023-07-16 19:45:58 +01:00
deavmi started working 2023-07-20 15:02:32 +01:00
Author
Owner

Added new symbol types for multi-line comments and single line comments @gusmeyer

Added new symbol types for multi-line comments and single line comments @gusmeyer
deavmi added the
parser
label 2023-07-20 15:38:23 +01:00
deavmi stopped working 2023-07-20 15:38:25 +01:00
35 minutes 53 seconds
deavmi started working 2023-07-20 15:56:53 +01:00
deavmi stopped working 2023-07-20 15:59:40 +01:00
2 minutes 48 seconds
deavmi started working 2023-07-20 16:01:39 +01:00
deavmi stopped working 2023-07-20 16:01:42 +01:00
3 seconds
deavmi added spent time 2023-07-20 16:34:34 +01:00
10 minutes
deavmi started working 2023-07-20 16:34:37 +01:00
deavmi stopped working 2023-07-20 16:37:03 +01:00
2 minutes 26 seconds
Author
Owner

Using ArrLexer to test comments before we get @gusmeyer to actually implement comments in the lexer

Using `ArrLexer` to test comments before we get @gusmeyer to actually implement comments in the lexer
Author
Owner

Completed

Completed
deavmi reopened this issue 2023-07-22 20:08:12 +01:00
deavmi removed the
lexer
label 2023-07-22 20:08:20 +01:00
deavmi changed title from Comment support to Comment support in parser 2023-07-22 20:08:28 +01:00
deavmi added
lexer
and removed
parser
labels 2023-07-22 20:08:35 +01:00
deavmi changed title from Comment support in parser to Comment support 2023-07-22 20:08:40 +01:00
deavmi removed their assignment 2023-07-22 20:08:50 +01:00
deavmi self-assigned this 2023-07-22 20:10:54 +01:00
deavmi added the
parser
label 2023-07-22 20:10:57 +01:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Total Time Spent: 51 minutes 10 seconds
deavmi
51 minutes 10 seconds
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

2023-07-30

Reference: tlang/tlang#84
No description provided.