Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Batch Amendment #5060
base: develop
Are you sure you want to change the base?
Batch Amendment #5060
Changes from 83 commits
879aba5
46fd68f
c57ace7
91d9909
f37377d
b3136a2
5e919eb
1225a61
304ff3e
aded948
82d4943
eaf7893
ca39807
dd71073
26518a5
3ceb05c
12324f1
e9f045b
51ed234
676b9b6
6c6180d
bd03ea3
c0e00b0
a3e15a1
14eef60
70e9931
c693b8f
418836c
fe09972
a72b070
0a45c16
2a0b9d4
3a9963a
2c3c5c4
a54a570
3ab585b
26ee555
0d5242e
51990ff
ff7fa26
ae5abf5
a78da6a
d3a2554
e099e03
84a69ef
f9fe787
0deea74
9266676
3fff4ca
ecfc64b
096f4bf
122368f
274660b
04c5bde
e4a5c52
659e705
597e7fe
639f8b7
75fd76b
2925a5d
1367e03
f185d38
767a88e
ee03811
9d8368d
b408b80
27d4685
419b14d
6c87e70
a75cf0e
e6021bf
fd0acb7
8a6d6ce
dd77e03
6cb23e3
88c40e4
e4cb784
47120ac
6ed4ac8
52f1ba9
1978fd3
6ffe349
04519e6
8029c30
7e60777
403599b
d18e9b3
afe58ab
aec8268
086e6b0
2ea9fe6
776aabc
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why add these to
STTx
instead of having the code inBatch.cpp
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because they were so similar to the other functionality. I can move them. Is this preferred?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can prob add
tfInnerBatchTxn
intotfUniversal
directly, and check inTransactor::preflight1
iftfInnerBatchTxn
is usable