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.

Sign up for GitHub

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

Jump to bottom

Fix -Wloop-analysis warnings. #50435

Merged
merged 1 commit into from Jun 26, 2021
Merged

Fix -Wloop-analysis warnings. #50435

merged 1 commit into from Jun 26, 2021

Conversation

pkasting
Copy link
Contributor

Bug: chromium:1223264

@pkasting
Fix -Wloop-analysis warnings.
9cf7001 
Bug: chromium:1223264
@google-ml-butler google-ml-butler bot added the size:S CL Change Size: Small label Jun 24, 2021
@google-cla google-cla bot added the cla: yes label Jun 24, 2021
@gbaned gbaned added the comp:lite TF Lite related issues label Jun 24, 2021
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Jun 24, 2021
@gbaned gbaned requested a review from miaout17 June 24, 2021 03:51
@gbaned gbaned self-assigned this Jun 24, 2021
miaout17
miaout17 approved these changes Jun 25, 2021
View reviewed changes
Copy link
Contributor

@miaout17 miaout17 left a comment

Choose a reason for hiding this comment

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

Looks good. It seems we accidentally made a reference to a constant pointer before?

@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Jun 25, 2021
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Jun 25, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jun 25, 2021
@copybara-service copybara-service bot merged commit e7e270d into tensorflow:master Jun 26, 2021
PR Queue automation moved this from Approved by Reviewer to Merged Jun 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miaout17 miaout17 miaout17 approved these changes

Assignees

@gbaned gbaned

Labels
cla: yes comp:lite TF Lite related issues ready to pull PR ready for merge process size:S CL Change Size: Small
Projects
PR Queue
  
Merged
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@pkasting @miaout17 @kokoro-team @gbaned