Skip to content

Conversation

@CertainLach
Copy link
Owner

@CertainLach CertainLach commented Jun 5, 2021

Two bellow issues were fixed by forking rust-gc into https://github.com/CertainLach/jrsonnet-gc

It is currently not possible to implement recursive type, so all tracing
is noop here

Signed-off-by: Yaroslav Bolyukin <[email protected]>
AST can't have cycles, so tracing is noop here

Signed-off-by: Yaroslav Bolyukin <[email protected]>
Interned string is backed by Rc<str>, there is nothing to trace here

Signed-off-by: Yaroslav Bolyukin <[email protected]>
Some manual Trace/Finalize implementations can be replaced with derives
with Manishearth/rust-gc#116 getting merged

Signed-off-by: Yaroslav Bolyukin <[email protected]>
It will build, but some methods is broken for now

Signed-off-by: Yaroslav Bolyukin <[email protected]>
@CertainLach CertainLach requested a review from JarvisCraft June 5, 2021 19:08
@CertainLach CertainLach added the enhancement New feature or request label Jun 5, 2021
@CertainLach CertainLach self-assigned this Jun 5, 2021
@CertainLach CertainLach added this to the v0.4.0 milestone Jun 5, 2021
Signed-off-by: Yaroslav Bolyukin <[email protected]>
@CertainLach CertainLach requested a review from JarvisCraft June 12, 2021 12:43
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Property __intrinsic_namespace__ shouldn't be handled by anyone

Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
@CertainLach CertainLach merged commit 8db09e1 into master Jul 4, 2021
@CertainLach CertainLach deleted the gc-v2 branch July 4, 2021 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants