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
crypto/sha1: use math/bits.RotateLeft32 #49051
Conversation
This PR (HEAD: 8aee742) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/356516 to see it. Tip: You can toggle comments from me using the |
8aee742
to
17ea697
Compare
Message from Filippo Valsorda: Patch Set 1: Run-TryBot+1 Code-Review+2 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 1: TryBot-Result-1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
This PR (HEAD: 17ea697) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/356516 to see it. Tip: You can toggle comments from me using the |
Message from Alexander Yastrebov: Patch Set 2: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Filippo Valsorda: Patch Set 3: Patch Set 2 was rebased Copied Votes:
Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Filippo Valsorda: Patch Set 3: Run-TryBot+1 Auto-Submit+1 Code-Review+2 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 3: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 3: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Filippo Valsorda: Patch Set 4: Run-TryBot+1 Auto-Submit+1 Code-Review+2 Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 4: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 4: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 4: TryBot-Result-1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Filippo Valsorda: Patch Set 5: Patch Set 4 was rebased Copied Votes:
Outdated Votes:
Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Filippo Valsorda: Patch Set 5: Run-TryBot+1 Auto-Submit+1 Code-Review+2 Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 5: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Gopher Robot: Patch Set 5: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Heschi Kreinick: Patch Set 5: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Bryan Mills: Patch Set 5: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Message from Daniel Martí: Patch Set 5: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
Updates #31456 Change-Id: I68e0abfb6771c9b1d1bfcbb642db9eb5540f9cab GitHub-Last-Rev: 17ea697 GitHub-Pull-Request: #49051 Reviewed-on: https://go-review.googlesource.com/c/go/+/356516 Auto-Submit: Filippo Valsorda <filippo@golang.org> Reviewed-by: Heschi Kreinick <heschi@google.com> Reviewed-by: Bryan Mills <bcmills@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Filippo Valsorda <filippo@golang.org> Reviewed-by: Filippo Valsorda <filippo@golang.org>
Message from Filippo Valsorda: Patch Set 5: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/356516. |
This PR is being closed because golang.org/cl/356516 has been merged. |
Updates golang#31456 Change-Id: I68e0abfb6771c9b1d1bfcbb642db9eb5540f9cab GitHub-Last-Rev: 17ea697 GitHub-Pull-Request: golang#49051 Reviewed-on: https://go-review.googlesource.com/c/go/+/356516 Auto-Submit: Filippo Valsorda <filippo@golang.org> Reviewed-by: Heschi Kreinick <heschi@google.com> Reviewed-by: Bryan Mills <bcmills@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Filippo Valsorda <filippo@golang.org> Reviewed-by: Filippo Valsorda <filippo@golang.org>
Updates #31456 Change-Id: I68e0abfb6771c9b1d1bfcbb642db9eb5540f9cab GitHub-Last-Rev: 17ea697c5c0bbfdfb1ad91c2c60e22f6efc78b43 GitHub-Pull-Request: golang/go#49051 Reviewed-on: https://go-review.googlesource.com/c/go/+/356516 Auto-Submit: Filippo Valsorda <filippo@golang.org> Reviewed-by: Heschi Kreinick <heschi@google.com> Reviewed-by: Bryan Mills <bcmills@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Filippo Valsorda <filippo@golang.org> Reviewed-by: Filippo Valsorda <filippo@golang.org>
Updates #31456