[PR #568] [MERGED] [bugs] cleanup some codes for global variables and clang-tidy specs #994

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

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/568
Author: @kvedala
Created: 7/13/2020
Status: Merged
Merged: 7/16/2020
Merged by: @kvedala

Base: masterHead: cleanup


📝 Commits (4)

  • d19a3a7 cleanup some codes for global variables and clang-tidy specs
  • 9642e10 allow subfolders in project_euler folder
  • f58916f fix dynamic array allocations
  • 39cdc7b fix possible memory leak

📊 Changes

6 files changed (+88 additions, -27 deletions)

View changed files

📝 .github/workflows/awesome_workflow.yml (+1 -1)
📝 project_euler/problem_13/sol1.c (+37 -14)
📝 project_euler/problem_23/sol1.c (+14 -2)
📝 project_euler/problem_23/sol2.c (+14 -2)
📝 project_euler/problem_25/sol1.c (+6 -0)
📝 project_euler/problem_401/sol1.c (+16 -8)

📄 Description

Description of Change

Fixed some files for errors and warnings reported by clang-tidy.

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/568 **Author:** [@kvedala](https://github.com/kvedala) **Created:** 7/13/2020 **Status:** ✅ Merged **Merged:** 7/16/2020 **Merged by:** [@kvedala](https://github.com/kvedala) **Base:** `master` ← **Head:** `cleanup` --- ### 📝 Commits (4) - [`d19a3a7`](https://github.com/TheAlgorithms/C/commit/d19a3a7bc889ac7103aeca38d9a241d73bbf0360) cleanup some codes for global variables and clang-tidy specs - [`9642e10`](https://github.com/TheAlgorithms/C/commit/9642e1068d1a5448a414304e8472741aee14df9c) allow subfolders in project_euler folder - [`f58916f`](https://github.com/TheAlgorithms/C/commit/f58916f8f51087ca644dd5703bad2b2b6777626d) fix dynamic array allocations - [`39cdc7b`](https://github.com/TheAlgorithms/C/commit/39cdc7b10800ee0a31f6a6e6b98870bdb0f29fdb) fix possible memory leak ### 📊 Changes **6 files changed** (+88 additions, -27 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/awesome_workflow.yml` (+1 -1) 📝 `project_euler/problem_13/sol1.c` (+37 -14) 📝 `project_euler/problem_23/sol1.c` (+14 -2) 📝 `project_euler/problem_23/sol2.c` (+14 -2) 📝 `project_euler/problem_25/sol1.c` (+6 -0) 📝 `project_euler/problem_401/sol1.c` (+16 -8) </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 --> Fixed some files for errors and warnings reported by clang-tidy. #### References <!-- Add any reference to previous pull-request or issue --> #### 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 --> <a href="https://gitpod.io/#https://github.com/TheAlgorithms/C/pull/568"><img src="https://gitpod.io/api/apps/github/pbs/github.com/kvedala/C.git/f58916f8f51087ca644dd5703bad2b2b6777626d.svg" /></a> --- <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:41 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#994