alloy-rs / alloy-rs/core

[Refactor] Redesign APIs to prevent users from wanting to keep JSON abi data

Đang mở
#670 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Rust
Star
967
Fork
274
Merge trung bình
1 ngày 21 giờ
Pull request đã merge (30 ngày)
15

Mô tả

JSON ABIs are an anti-pattern because they have fallible resolution steps before every encoding

we would prefer the user resolve an entire `DynInterface` at once, and then reference parts of it

hmmm. I think that if users are keeping the json representation in memory and passing it around, that's generally an anti-pattern, right? I see these traits as convenience functions for one-off things and to patch functionality holes, not as core API. `Dyn____` is our in-memory representation of ABI information, and greedily resolving a `JsonAbi` into a `DynAbi` is the longterm officially-recommended path

_Originally posted by @prestwich in https://github.com/alloy-rs/core/pull/632#discussion_r1627614889_

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.