starred/C
1
0
Fork 0
mirror of https://github.com/TheAlgorithms/C.git synced 2026-02-14 13:54:36 +00:00
Code Issues 1.4k Packages Projects Releases Wiki Activity

[PR #554] [bugs & docs] lots of documentation and bug fixes #983

New Issue
Open
opened 2026-01-29 15:16:36 +00:00 by claunia · 0 comments
claunia commented 2026-01-29 15:16:36 +00:00
Owner
Copy Link

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

State: closed
Merged: Yes

Description of Change

  • Sudoku file used multiple global variables with ambiguous names and had no documentation.
    These issues have been fixed. Moreover, extensive documentation and debug messages have been added.
  • More documentation and LGTM alerts fixes for:
    • fibonacci_fast
    • project euler problem 19
    • merge_sort
      • also fixed dynamic memory de-allocation

References

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/554 **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 --> * Sudoku file used multiple global variables with ambiguous names and had no documentation. These issues have been fixed. Moreover, extensive documentation and debug messages have been added. * More documentation and LGTM alerts fixes for: * fibonacci_fast * project euler problem 19 * merge_sort * also fixed dynamic memory de-allocation #### References <!-- Add any reference to previous pull-request or issue --> * Part of efforts for #550 * original commit for sudoku: 97f8c4a782d80f1afb0a7a0881b11c98968ae57f #### 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) - [x] 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:36 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
dont-close
enhancement
good first issue
hacktoberfest
hacktoberfest
help wanted
invalid
not-an-algorithm
out of date
Practice coding
pull-request
Mirrored from GitHub Pull Request
 question
Stale
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claunia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#983
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?