GraphiteEditor / GraphiteEditor/Graphite

Avoid needing the TaggedValue enum

Aperta
#3,131 0 commenti 0 reazioni 1 assegnatario Rivendicata da @TrueDoctor Vedi su GitHub
Architecture
Lingua principale
Rust
Stelle
27.2k
Fork
1.3k
Merge medio
20h 5m
PR unite (30g)
57

Descrizione

Discussion here: https://discord.com/channels/731730685944922173/860830913390575646/1412529955841245345 where @TrueDoctor wrote:

> We would need a self describing serialization format which does store the type information but if we want arbitrary compound types to work we will always have to do some form of manual deserializaiton
>
> something like this: https://github.com/dtolnay/typetag
>
> … this crates uses a similar link time hack to generate a list of all possible implementations for deserialization as we use for our node definitions
>
> So it should be possible to remove tagged values (or we could at least experiment with that), it's just a question of if that is useful to us rn and if we should prioritize this. And I don't think we should do that since tagged values work just fine for us rn

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.