[PR #1262] [MERGED] Update binary_to_decimal.c #1860

Closed
opened 2026-01-29 15:25:27 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/1262
Author: @Indrranil
Created: 5/24/2023
Status: Merged
Merged: 5/29/2023
Merged by: @Panquesito7

Base: masterHead: patch-1


📝 Commits (2)

  • 9c22f1d Update binary_to_decimal.c
  • 7c26b20 Update binary_to_decimal.c

📊 Changes

1 file changed (+26 additions, -13 deletions)

View changed files

📝 conversions/binary_to_decimal.c (+26 -13)

📄 Description

  1. Removed the unused variable remainder.

  2. Changed the variable name number to binary_number for clarity.

  3. Removed the initialisation of number and temp since they are assigned values later.

  4. Removed the newline character from the printf statement to improve readability.

5.Added a return statement at the end of main function.

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/1262 **Author:** [@Indrranil](https://github.com/Indrranil) **Created:** 5/24/2023 **Status:** ✅ Merged **Merged:** 5/29/2023 **Merged by:** [@Panquesito7](https://github.com/Panquesito7) **Base:** `master` ← **Head:** `patch-1` --- ### 📝 Commits (2) - [`9c22f1d`](https://github.com/TheAlgorithms/C/commit/9c22f1dbba24607842650439b2dae1bc892d8b74) Update binary_to_decimal.c - [`7c26b20`](https://github.com/TheAlgorithms/C/commit/7c26b2012ba4ee38a83e6c6bb6d16f751cd61604) Update binary_to_decimal.c ### 📊 Changes **1 file changed** (+26 additions, -13 deletions) <details> <summary>View changed files</summary> 📝 `conversions/binary_to_decimal.c` (+26 -13) </details> ### 📄 Description 1. Removed the unused variable remainder. 2. Changed the variable name number to binary_number for clarity. 3. Removed the initialisation of number and temp since they are assigned values later. 4. Removed the newline character from the printf statement to improve readability. 5.Added a return statement at the end of main function. #### 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 - [ ] Added file name matches [File name guidelines](https://github.com/TheAlgorithms/C/blob/master/CONTRIBUTING.md#File-Name-guidelines) - [ ] Added tests and example, test must pass - [ ] 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) - [ ] 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:25:27 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#1860