coveralls
commented
1 year ago Pull Request Test Coverage Report for Build 4281966818
- 15 of 15 (100.0%) changed or added relevant lines in 1 file are covered.
- 38 unchanged lines in 3 files lost coverage.
- Overall coverage decreased (-0.05%) to 55.222%
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| connmgr/connmanager.go | 2 | 86.62% | ||
| wire/msgaddrv2.go | 16 | 51.72% | ||
| wire/netaddressv2.go | 20 | 74.45% | ||
| <!-- | Total: | 38 | --> |
| Totals | |
|---|---|
| Change from base Build 4163994991: | -0.05% |
| Covered Lines: | 26619 |
| Relevant Lines: | 48204 |
The default script builder allocates 500 bytes of memory for each script. That is quite large for most applications and therefore wastes a lot of memory when only small scripts are created. To avoid breaking backward compatibility, we add a new functional option to the NewScriptBuilder function that allows the user to specify the initial allocation size.
cc @Roasbeef, @Crypt-iQ.