[PR #580] [MERGED] [enhancement] formatted and added Hash directory to cmake #1011

New Issue

claunia · 2026-01-29T15:16:50Z

claunia commented

2026-01-29 15:16:50 +00:00

📋 Pull Request Information

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

Base: master ← Head: hash

📝 Commits (10+)

63a605e added hash folder to CMAKE build
138400e split sdbm code from hash.c to independent program
af6a872 update readme file
d6dc6df docs + vartype fix
9c5dae4 split djb2 code from hash.c to independent program
0bd33dc fix function reference
ec3488d split xor8 code from hash.c to independent program
3674997 split adler32 code from hash.c to independent program
b2abbfe remove additional author
c3b44aa split crc32 code from hash.c to independent program

📊 Changes

13 files changed (+295 additions, -157 deletions)

View changed files

📝 .clang-tidy (+1 -1)
📝 CMakeLists.txt (+1 -0)
📝 DIRECTORY.md (+5 -3)
➕ hash/CMakeLists.txt (+20 -0)
📝 hash/README.md (+1 -2)
➖ hash/hash.c (+0 -82)
➖ hash/hash.h (+0 -49)
➕ hash/hash_adler32.c (+54 -0)
➕ hash/hash_crc32.c (+62 -0)
➕ hash/hash_djb2.c (+50 -0)
➕ hash/hash_sdbm.c (+50 -0)
➕ hash/hash_xor8.c (+51 -0)
➖ hash/test_program.c (+0 -20)

📄 Description

Description of Change

Programs in hash directory were not documented and present as one file. This PR splits the individual functions to individual files and enables CMAKE on the repo.

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/580 **Author:** [@kvedala](https://github.com/kvedala) **Created:** 7/22/2020 **Status:** ✅ Merged **Merged:** 7/29/2020 **Merged by:** [@kvedala](https://github.com/kvedala) **Base:** `master` ← **Head:** `hash` --- ### 📝 Commits (10+) - [`63a605e`](https://github.com/TheAlgorithms/C/commit/63a605e1663da5534c73b4869120852e30cfeed9) added hash folder to CMAKE build - [`138400e`](https://github.com/TheAlgorithms/C/commit/138400e5587b4a716957157779ecd0b3092f9634) split sdbm code from hash.c to independent program - [`af6a872`](https://github.com/TheAlgorithms/C/commit/af6a87235a45d4cb6410c79b7a2384be5dba9f9f) update readme file - [`d6dc6df`](https://github.com/TheAlgorithms/C/commit/d6dc6df33dc666158ab47a44704d452088a6f90b) docs + vartype fix - [`9c5dae4`](https://github.com/TheAlgorithms/C/commit/9c5dae413f3c5e0029f77de57c3d242b7205fe8e) split djb2 code from hash.c to independent program - [`0bd33dc`](https://github.com/TheAlgorithms/C/commit/0bd33dcedcf323886b96aff3f1cf38eb2a32dbe5) fix function reference - [`ec3488d`](https://github.com/TheAlgorithms/C/commit/ec3488deba5badadbb55ac608a05abc4c5a474e3) split xor8 code from hash.c to independent program - [`3674997`](https://github.com/TheAlgorithms/C/commit/367499799e0672999eecfc6a3ae4900defcd90f6) split adler32 code from hash.c to independent program - [`b2abbfe`](https://github.com/TheAlgorithms/C/commit/b2abbfe8ebfa66f730c1f697c478d5856d18935a) remove additional author - [`c3b44aa`](https://github.com/TheAlgorithms/C/commit/c3b44aaaa86237ad012226029098d1abba2ab6a7) split crc32 code from hash.c to independent program ### 📊 Changes **13 files changed** (+295 additions, -157 deletions) <details> <summary>View changed files</summary> 📝 `.clang-tidy` (+1 -1) 📝 `CMakeLists.txt` (+1 -0) 📝 `DIRECTORY.md` (+5 -3) ➕ `hash/CMakeLists.txt` (+20 -0) 📝 `hash/README.md` (+1 -2) ➖ `hash/hash.c` (+0 -82) ➖ `hash/hash.h` (+0 -49) ➕ `hash/hash_adler32.c` (+54 -0) ➕ `hash/hash_crc32.c` (+62 -0) ➕ `hash/hash_djb2.c` (+50 -0) ➕ `hash/hash_sdbm.c` (+50 -0) ➕ `hash/hash_xor8.c` (+51 -0) ➖ `hash/test_program.c` (+0 -20) </details> ### 📄 Description #### Description of Change  Programs in `hash` directory were not documented and present as one file. This PR splits the individual functions to individual files and enables CMAKE on the repo. #### References  #### Checklist  - [x] Added description of change - [x] 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:  <a href="https://gitpod.io/#https://github.com/TheAlgorithms/C/pull/580"><img src="https://gitpod.io/api/apps/github/pbs/github.com/kvedala/C.git/be1dbec9abcf65089fb387b972242142acc7ca25.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:50 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/C#1011

[PR #580] [MERGED] [enhancement] formatted and added Hash directory to cmake #1011