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

Sigma fixes #777

Merged
merged 33 commits into from Dec 2, 2019

Conversation

@levonpetrosyan93
Copy link
Contributor

levonpetrosyan93 commented Dec 2, 2019

No description provided.

levonpetrosyan93 and others added 28 commits Nov 11, 2019
Master update
Sigma fixes
Disabling Sigma
Padding implemented
@reubenyap reubenyap requested review from a-bezrukov and psolstice Dec 2, 2019
@reubenyap reubenyap requested a review from riordant Dec 2, 2019
@lgtm-com

This comment has been minimized.

Copy link

lgtm-com bot commented Dec 2, 2019

This pull request introduces 3 alerts when merging 9549705 into 2a1dad7 - view on LGTM.com

new alerts:

  • 3 for Multiplication result converted to larger type
Andrey
@lgtm-com

This comment has been minimized.

Copy link

lgtm-com bot commented Dec 2, 2019

This pull request introduces 3 alerts when merging 4250aa0 into 2a1dad7 - view on LGTM.com

new alerts:

  • 3 for Multiplication result converted to larger type
@levonpetrosyan93 levonpetrosyan93 force-pushed the sigma_fixes branch from c047c03 to fe61af5 Dec 2, 2019
levonpetrosyan93 and others added 3 commits Dec 2, 2019
Andrey
@levonpetrosyan93 levonpetrosyan93 requested a review from psolstice Dec 2, 2019
@a-bezrukov a-bezrukov merged commit af20f12 into master Dec 2, 2019
4 checks passed
4 checks passed
LGTM analysis: JavaScript No code changes detected
Details
LGTM analysis: C/C++ 3 new alerts
Details
LGTM analysis: Python No new or fixed alerts
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
@a-bezrukov a-bezrukov deleted the sigma_fixes branch Dec 2, 2019
@lgtm-com

This comment has been minimized.

Copy link

lgtm-com bot commented Dec 2, 2019

This pull request introduces 3 alerts when merging 5407b71 into 2a1dad7 - view on LGTM.com

new alerts:

  • 3 for Multiplication result converted to larger type
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.