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. >

Re: [PATCH] Remove HIR MacroItem and other hir macro forward declarations

2021-07-12 Thread Philip Herron
On 11/07/2021 22:39, Mark Wielaard wrote: > Almost all HIR Macro related trees were already removed by > https://github.com/Rust-GCC/gccrs/pull/492 > > But there was still one MacroItem class left in rust-hir.h and several > (unused) forward declarations in various other hir .h files. Remove > them

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 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'

New contributor tasks

2021-07-12 Thread Mark Wielaard
On Mon, Jul 12, 2021 at 11:06:01AM +0100, Philip Herron wrote: > Great work once again. I am aiming to spend some time towards the end of > the week to add more tickets and info for new contributors to get > involved, which I will post the interesting ones onto the mailing list > as well. I think i