Move struct parsers should to rpcclient/ packages #93

Open
opened 2024-05-07 10:04:28 +00:00 by fyrchik · 0 comments

Consider https://git.frostfs.info/TrueCloudLab/frostfs-contract/src/branch/master/frostfsid/client/client.go#L249. parseSubjectExtended is used privately and should be duplicated in the client code (see TrueCloudLab/frostfs-node#1117).

A better approach would be to have sth like ParseSubjectExtended([]stackitem.Item) or UnwrapSubjectExtended(*result.Invoke, error) together with the autogenerated code.
In the bright future, it will surely support generating custom structs.

Consider https://git.frostfs.info/TrueCloudLab/frostfs-contract/src/branch/master/frostfsid/client/client.go#L249. `parseSubjectExtended` is used privately and should be duplicated in the client code (see https://git.frostfs.info/TrueCloudLab/frostfs-node/pulls/1117). A better approach would be to have sth like `ParseSubjectExtended([]stackitem.Item)` or `UnwrapSubjectExtended(*result.Invoke, error)` together with the autogenerated code. In the bright future, it will surely support generating custom structs.
fyrchik changed title from Struct parsers should belong to `rpcclient/` packages to Move struct parsers should to `rpcclient/` packages 2024-05-07 10:04:40 +00:00
fyrchik added the
refactoring
internal
labels 2024-05-07 11:43:27 +00:00
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-contract#93
There is no content yet.