Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removing Wasted Bytes #81

Open
colin-axner opened this issue Dec 3, 2018 · 2 comments
Open

Removing Wasted Bytes #81

colin-axner opened this issue Dec 3, 2018 · 2 comments
Labels
enhancement New feature or request V2 next iteration of the contract
Milestone

Comments

@colin-axner
Copy link
Member

colin-axner commented Dec 3, 2018

I think a future optimization to be considered is removing the requirement for 65 zero bytes to be passed in for the signature of a 1 input transaction. It would add a bit more logic to the rootchain, but reduces the block size of the sidechain (allowing more txs in a block).

@colin-axner colin-axner added the enhancement New feature or request label Dec 3, 2018
@ValarDragon
Copy link

65 bytes of zeros cost 260 gas. Its not fully clear to me, but looking at the yellow paper codes it appears that the if statement gas is signifcantly smaller.

Also removing these 0 bytes is a significant improvement to tendermint block size, which should be a noticeable improvement to speed of consensus.

@colin-axner colin-axner added the V2 next iteration of the contract label Feb 28, 2019
@colin-axner
Copy link
Member Author

Remove padding of fixed length txBytes, The average tx will currently consistent of 250-500 unnecessary zero bytes. This will slow down processing on sidechain/increase storage.

@colin-axner colin-axner added this to the Version 2 milestone Feb 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request V2 next iteration of the contract
Development

No branches or pull requests

2 participants