[PR #840] [feat/fix/docs]: Improved on conversion code , length variable , asserts added #1351

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

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/840
Author: @lazy-dude
Created: 7/8/2021
Status: 🔄 Open

Base: masterHead: master


📝 Commits (10+)

  • 67eeefe Added asserts , number length check
  • 0008c67 Remove extra empty lines
  • 7185941 Auto indent
  • 6e5e561 fix: corrected shown hex number.
  • 905b80c Changed to recommended typical structure
  • e771021 docs: Added some doxygen doc.
  • 157f6d1 Update conversions/binary_to_decimal.c
  • bbb6024 Update conversions/binary_to_decimal.c
  • 82e1f41 Update conversions/binary_to_decimal.c
  • 1e4c396 Update conversions/binary_to_decimal.c

📊 Changes

2 files changed (+101 additions, -20 deletions)

View changed files

📝 conversions/binary_to_decimal.c (+96 -16)
📝 conversions/decimal_to_hexa.c (+5 -4)

📄 Description

Description of Change

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/840 **Author:** [@lazy-dude](https://github.com/lazy-dude) **Created:** 7/8/2021 **Status:** 🔄 Open **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (10+) - [`67eeefe`](https://github.com/TheAlgorithms/C/commit/67eeefe058fc247244c2d21efb287c4a90834836) Added asserts , number length check - [`0008c67`](https://github.com/TheAlgorithms/C/commit/0008c6745def650ec30f9e4de4f94cf0e69c1315) Remove extra empty lines - [`7185941`](https://github.com/TheAlgorithms/C/commit/7185941c68857d0261244367c3a141bb34cbef30) Auto indent - [`6e5e561`](https://github.com/TheAlgorithms/C/commit/6e5e5614919e38b9e42df52d91359942cb86b708) fix: corrected shown hex number. - [`905b80c`](https://github.com/TheAlgorithms/C/commit/905b80c41ee262ab3c07bc1c1978c3a4232468ff) Changed to recommended typical structure - [`e771021`](https://github.com/TheAlgorithms/C/commit/e771021978d265a9fa2efac9b3afd9f592f40a5f) docs: Added some doxygen doc. - [`157f6d1`](https://github.com/TheAlgorithms/C/commit/157f6d11b2497ea4b9bdda060c4e17f7039ab5b5) Update conversions/binary_to_decimal.c - [`bbb6024`](https://github.com/TheAlgorithms/C/commit/bbb602416b89868c913508283b6190f8862f7c4f) Update conversions/binary_to_decimal.c - [`82e1f41`](https://github.com/TheAlgorithms/C/commit/82e1f416896deadc680b4874732236b572973311) Update conversions/binary_to_decimal.c - [`1e4c396`](https://github.com/TheAlgorithms/C/commit/1e4c3962bdd16cd227a298e8f6201d62ccbcb0ae) Update conversions/binary_to_decimal.c ### 📊 Changes **2 files changed** (+101 additions, -20 deletions) <details> <summary>View changed files</summary> 📝 `conversions/binary_to_decimal.c` (+96 -16) 📝 `conversions/decimal_to_hexa.c` (+5 -4) </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/blob/master/CONTRIBUTING.md --> #### 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 - [x] Added file name matches [File name guidelines](https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md#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/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/840"><img src="https://gitpod.io/button/open-in-gitpod.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:19:37 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#1351