-
Notifications
You must be signed in to change notification settings - Fork 3
Closed
Labels
COREcore functionalitiescore functionalitiesDOCImprovements or additions to documentationImprovements or additions to documentationFEATURENew feature or requestNew feature or request
Milestone
Description
Description
In several data model we have *Length
variable to define the length of the next variable in the bytes. @capt4ce bring a good point of wether we need the *Length
or not.
- Proposal:
We should use thelength
only in theTransactionBytes
like
length := len(tx.SenderAddressLength)
buffer.Write(length)
buffer.Write(tx.SenderAddress)
(`[....1,0,0,0, ...]`)
instead of in the proto structure like
Transaction {
SenderAccountAddressLength uint32
SenderAddress string
}
buffer.Write(tx.SenderAddressLength)
buffer.Write(tx.SenderAddress)
Breakdown
- Remove
*length
variable in
Transaction.*AccountAddress
Transaction.TransactionBodyLength
NodeRegistration.NodeAddressLength
Additional Diagram / File
Put additional diagram or file
Metadata
Metadata
Assignees
Labels
COREcore functionalitiescore functionalitiesDOCImprovements or additions to documentationImprovements or additions to documentationFEATURENew feature or requestNew feature or request