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

mbsys_simrad3_zero_ss had for i inside of for i #751

Closed
schwehr opened this issue Dec 17, 2019 · 0 comments · Fixed by #754
Closed

mbsys_simrad3_zero_ss had for i inside of for i #751

schwehr opened this issue Dec 17, 2019 · 0 comments · Fixed by #754

Comments

@schwehr
Copy link
Collaborator

schwehr commented Dec 17, 2019

Definitely not going to zero all the things

for (int i = 0; i < MBSYS_SIMRAD3_NUM_PING_STRUCTURES; i++) {
  // ...
  for (int i = 0; i < MBSYS_SIMRAD3_MAXBEAMS; i++) {
    //
  }
  for (int i = 0; i < MBSYS_SIMRAD3_MAXRAWPIXELS; i++) {
    //
  }
  for (int i = 0; i < MBSYS_SIMRAD3_MAXPIXELS; i++) {
     //
  }
}
schwehr added a commit to schwehr/MB-System that referenced this issue Dec 17, 2019
- Comment out unused variables
- Localize variables
- Add TODO about probably bug in alloc and for sure bug in zero (dwcaress#751)
- Add const
schwehr added a commit that referenced this issue Dec 17, 2019
- Comment out unused variables
- Localize variables
- Add TODO about probably bug in alloc and for sure bug in zero (#751)
- Add const
schwehr added a commit to schwehr/MB-System that referenced this issue Dec 18, 2019
schwehr added a commit that referenced this issue Dec 18, 2019
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 a pull request may close this issue.

1 participant