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

[PR #553] [MERGED] [bugs] fixes for lgtm and improve docs #980

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

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/553
Author: @kvedala
Created: 7/2/2020
Status: ✅ Merged
Merged: 7/2/2020
Merged by: @kvedala

Base: master ← Head: fixes/lgtm

📝 Commits (6)

📊 Changes

4 files changed (+63 additions, -28 deletions)

View changed files

📝 numerical_methods/durand_kerner_roots.c (+6 -6)
📝 project_euler/problem_22/sol1.c (+1 -1)
📝 searching/modified_binary_search.c (+34 -11)
📝 sorting/bead_sort.c (+22 -10)

📄 Description

Description of Change

  • project Euler problem 22 lgtm error fix
  • fix inconsistent use of double and long double in durand_kerner_roots.c
  • add docs & fix lgtm alerts in bead_sort.c

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:

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/TheAlgorithms/C/pull/553 **Author:** [@kvedala](https://github.com/kvedala) **Created:** 7/2/2020 **Status:** ✅ Merged **Merged:** 7/2/2020 **Merged by:** [@kvedala](https://github.com/kvedala) **Base:** `master` ← **Head:** `fixes/lgtm` --- ### 📝 Commits (6) - [`f21f18e`](https://github.com/TheAlgorithms/C/commit/f21f18ef7333a074d5c778bf26fc6e441386da70) make identical datatype - [`cb99925`](https://github.com/TheAlgorithms/C/commit/cb999259cfa68ea19b8e8d765006dd6f85c6bed1) make all doubles as long double - [`2150c35`](https://github.com/TheAlgorithms/C/commit/2150c35846ac213a65b65e92346815d3910c7c99) fix printing long double - [`ec57c8f`](https://github.com/TheAlgorithms/C/commit/ec57c8fa405cb70ac8c82bed9975d0b44f476ca8) docs + fix lgtm alert - [`5daeb8d`](https://github.com/TheAlgorithms/C/commit/5daeb8daaadea630651c7f45354be6e955e3cd6c) fix docs & add link - [`ccd3f66`](https://github.com/TheAlgorithms/C/commit/ccd3f6660870626e2b9d1c679e609e924753306b) fix code - major errors + add docs ### 📊 Changes **4 files changed** (+63 additions, -28 deletions) <details> <summary>View changed files</summary> 📝 `numerical_methods/durand_kerner_roots.c` (+6 -6) 📝 `project_euler/problem_22/sol1.c` (+1 -1) 📝 `searching/modified_binary_search.c` (+34 -11) 📝 `sorting/bead_sort.c` (+22 -10) </details> ### 📄 Description #### 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 --> * project Euler problem 22 lgtm error fix * fix inconsistent use of `double` and `long double` in `durand_kerner_roots.c` * add docs & fix lgtm alerts in `bead_sort.c` #### References <!-- Add any reference to previous pull-request or issue --> * #550 * [project 22 - 1](https://lgtm.com/projects/g/TheAlgorithms/C/snapshot/4d0dbc401d67171d555d494c49a08e72303b5ba7/files/project_euler/problem_22/sol1.c?sort=name&dir=ASC&mode=heatmap#xa0d183509f9bbb35:1) * [durange_kerner_roots](https://lgtm.com/projects/g/TheAlgorithms/C/snapshot/4d0dbc401d67171d555d494c49a08e72303b5ba7/files/numerical_methods/durand_kerner_roots.c?sort=name&dir=ASC&mode=heatmap) * [bead_sort.c](https://lgtm.com/projects/g/TheAlgorithms/C/snapshot/4d0dbc401d67171d555d494c49a08e72303b5ba7/files/sorting/bead_sort.c?sort=name&dir=ASC&mode=heatmap#xdc9736f163b5f58d:1) #### 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 --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
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#980
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?