Re: [PATCH] Handle doc comment strings in lexer and parser

2021-07-12 Thread Philip Herron
On 12/07/2021 09:32, Mark Wielaard wrote: > Hi Philip, > > On Mon, Jul 12, 2021 at 09:09:09AM +0100, Philip Herron wrote: >> This patch looks good to me. When I tried to apply it to merge it I got >> the following: >> >> ``` >> $ git am  '[PATCH] Handle doc comment strings in lexer and parser.eml'

Re: [PATCH] Handle doc comment strings in lexer and parser

2021-07-12 Thread Mark Wielaard
Hi Philip, On Mon, Jul 12, 2021 at 09:09:09AM +0100, Philip Herron wrote: > This patch looks good to me. When I tried to apply it to merge it I got > the following: > > ``` > $ git am  '[PATCH] Handle doc comment strings in lexer and parser.eml' > Applying: Handle doc comment strings in lexer and

Re: [PATCH] Handle doc comment strings in lexer and parser

2021-07-12 Thread Philip Herron
On 11/07/2021 21:10, Mark Wielaard wrote: > Remove (unused) comment related tokens and replace them with > INNER_DOC_COMMENT and OUTER_DOC_COMMENT tokens, which keep the comment > text as a string. These can be constructed with the new > make_inner_doc_comment and make_outer_doc_comment methods. >

[PATCH] Handle doc comment strings in lexer and parser

2021-07-11 Thread Mark Wielaard
Remove (unused) comment related tokens and replace them with INNER_DOC_COMMENT and OUTER_DOC_COMMENT tokens, which keep the comment text as a string. These can be constructed with the new make_inner_doc_comment and make_outer_doc_comment methods. Make sure to not confuse doc strings with normal co