Skip to content
This repository has been archived by the owner on May 4, 2024. It is now read-only.

[RFC] [abi] implemented a abi for structs using serde-reflection #1055

Open
wants to merge 3 commits into
base: aptos-main
Choose a base branch
from

Conversation

runtian-zhou
Copy link
Member

Motivation

Compile struct layout into serde-reflection format that could be codegened using serde-generate crate.

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

Tests added.

@runtian-zhou runtian-zhou changed the base branch from main to aptos-main May 24, 2023 23:21
@sblackshear
Copy link
Member

Always love seeing more serde-generate integration! But could you help me understand the difference between this and the existing layout generation functionality? https://github.com/move-language/move/blob/main/language/tools/move-bytecode-utils/src/layout.rs is the layout generator, and is exposed via the CLI https://github.com/move-language/move/blob/main/language/tools/move-cli/src/sandbox/commands/generate.rs#L14 with move sandbox generate struct-layouts. This was the PR that added this #652

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants