GraphiteEditor / GraphiteEditor/Graphite

Avoid needing the TaggedValue enum

オープン
#3,131 コメント 0 件 リアクション 0 件 担当者 1 名 @TrueDoctor が担当を希望しています GitHub で見る
Architecture
主要言語
Rust
スター
27.2k
フォーク
1.3k
平均マージ
20時間 5分
マージ済み PR(30日)
57

説明

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

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。