Skip to content

Conversation

kormat
Copy link
Contributor

@kormat kormat commented Oct 16, 2018

This change is Reviewable

Copy link
Contributor

@juagargi juagargi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm: We will patch SCIONLab with almost identical code, only the tolerance being different

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link
Contributor

@juagargi juagargi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r2.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link
Contributor

@shitz shitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 1 of 1 files at r1, 1 of 1 files at r2.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@kormat kormat merged commit db20361 into scionproto:master Oct 16, 2018
@kormat kormat deleted the path_store_grace branch October 16, 2018 13:34
juagargi pushed a commit to juagargi/scion that referenced this pull request Oct 17, 2018
juagargi pushed a commit to juagargi/scion that referenced this pull request Oct 18, 2018
juagargi pushed a commit to juagargi/scion that referenced this pull request Oct 18, 2018
matzf added a commit to matzf/scion that referenced this pull request Nov 1, 2018
If a PCB arrives with a timestamp from the future (which is accepted
due to the changes in scionproto#1990), it may result in a zero delay_time value.
The delay_time value is used in a division in the fidelity computation,
which will trigger an exception.
matzf added a commit to matzf/scion that referenced this pull request Nov 5, 2018
If a PCB arrives with a timestamp from the future (which is accepted
due to the changes in scionproto#1990), it may result in a zero delay_time value.
The delay_time value is used in a division in the fidelity computation,
which will trigger an exception.
lukedirtwalker pushed a commit that referenced this pull request Nov 5, 2018
If a PCB arrives with a timestamp from the future (which is accepted
due to the changes in #1990), it may result in a zero delay_time value.
The delay_time value is used in a division in the fidelity computation,
which will trigger an exception.
matzf added a commit to matzf/scion that referenced this pull request Nov 5, 2018
If a PCB arrives with a timestamp from the future (which is accepted
due to the changes in scionproto#1990), it may result in a zero delay_time value.
The delay_time value is used in a division in the fidelity computation,
which will trigger an exception.
juagargi added a commit to juagargi/scion that referenced this pull request Sep 10, 2019
Antl had a problem with ARM 32:
antlr/antlr4#1858

So picking the commit of antlr where the PR scionproto#1990 was merged.

Also casting a constant to uint64 (see default type of an untyped constant
https://golang.org/ref/spec#Constants ).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants