Resolve "TransactionDocument: replace Vec fields by SmallVec"
Closes #197
Merge request reports
Activity
changed milestone to %v0.4 Full Mirror Node
added C-documents D-easy T-optimization labels
I hope that someone can help me, I don't know to who should I ask...
I almost finished this issue, but there is still a point that I don't know what to do about. Due to the change in the document structure for issuer and signature, all documents are bigger in stack. Because of that modif, 2 fields in the UserDocumentDUBP enum exceed the max difference authorised by clippy in an enum. In that enum, the others fields were already boxed because they are quite big. I tried to box every member, but then an integration test don't pass on gitlab (seems to be only the win64 test, from what I understood). On my machine, every tests passed. And seems that on linux enviro, on gitlab too. Looks like a memory allocation goes bad, like if this small increase in memory usage is too much
I envisage only 2 solutions, and only one seems acceptable for me :
- ignore the clippy warning
- increase the memory size for the win64 test virtual machine (seems bad)
added 1 commit
- f70da4ba - [opt] Replaced vec usage with SmallVec for Issuers, Signatures, Inputs,...
added 1 commit
- f70da4ba - [opt] Replaced vec usage with SmallVec for Issuers, Signatures, Inputs,...
