[PR #552] [bugs/lgtm] code + doc fixes + lgtm corrections #977

Closed
opened 2026-01-29 15:16:34 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/TheAlgorithms/C/pull/552

State: closed
Merged: Yes


Description of Change

  • Fixed misc/cantor_set.c for lgtm errors
  • Fixed searching/binary_search.c for lgtm errors
  • fixed cmake - removed /Za option for MSVC compiler
  • added details section to project_euler/problem_1/sol1.c
  • fix lgtm error for project_euler/problem_401/sol1.c

References

Issue: #550, lgtm alert link

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added tests and example, test must pass
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:

**Original Pull Request:** https://github.com/TheAlgorithms/C/pull/552 **State:** closed **Merged:** Yes --- #### Description of Change <!-- Thank you for your Pull Request. Please provide a description above and review the requirements below. Contributors guide: https://github.com/TheAlgorithms/C-Plus-Plus/CONTRIBUTING.md --> * Fixed `misc/cantor_set.c` for lgtm errors * Fixed `searching/binary_search.c` for lgtm errors * fixed cmake - removed `/Za` option for MSVC compiler * added `details` section to `project_euler/problem_1/sol1.c` * fix lgtm error for `project_euler/problem_401/sol1.c` #### References <!-- Add any reference to previous pull-request or issue --> Issue: #550, [lgtm alert link](https://lgtm.com/projects/g/TheAlgorithms/C/?mode=tree&ruleFocus=2157860312) #### Checklist <!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> - [x] Added description of change - [ ] Added file name matches [File name guidelines](https://github.com/TheAlgorithms/C-Plus-Plus/blob/master/CONTRIBUTING.md#New-File-Name-guidelines) - [ ] Added tests and example, test must pass - [x] Relevant documentation/comments is changed or added - [x] PR title follows semantic [commit guidelines](https://github.com/TheAlgorithms/C-Plus-Plus/blob/master/CONTRIBUTING.md#Commit-Guidelines) - [x] Search previous suggestions before making a new one, as yours may be a duplicate. - [x] I acknowledge that all my contributions will be made under the project's license. Notes: <!-- Please add a one-line description for developers or pull request viewers -->
claunia added the pull-request label 2026-01-29 15:16:34 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#977