Commit b41e8b34 authored by Maurelian's avatar Maurelian Committed by GitHub

More precise comments on batch context validity (#345)

parent af06777b
...@@ -937,7 +937,8 @@ contract OVM_CanonicalTransactionChain is iOVM_CanonicalTransactionChain, Lib_Ad ...@@ -937,7 +937,8 @@ contract OVM_CanonicalTransactionChain is iOVM_CanonicalTransactionChain, Lib_Ad
internal internal
view view
{ {
// If there are existing elements, this batch must come later. // If there are existing elements, this batch must have the same context
// or a later timestamp and block number.
if (getTotalElements() > 0) { if (getTotalElements() > 0) {
(,, uint40 lastTimestamp, uint40 lastBlockNumber) = _getBatchExtraData(); (,, uint40 lastTimestamp, uint40 lastBlockNumber) = _getBatchExtraData();
...@@ -1018,7 +1019,7 @@ contract OVM_CanonicalTransactionChain is iOVM_CanonicalTransactionChain, Lib_Ad ...@@ -1018,7 +1019,7 @@ contract OVM_CanonicalTransactionChain is iOVM_CanonicalTransactionChain, Lib_Ad
internal internal
view view
{ {
// All sequencer transactions' times must increase from the previous ones. // All sequencer transactions' times must be greater than or equal to the previous ones.
require( require(
_nextContext.timestamp >= _prevContext.timestamp, _nextContext.timestamp >= _prevContext.timestamp,
"Context timestamp values must monotonically increase." "Context timestamp values must monotonically increase."
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment