[PR #914] fix: fixed index of parent in sorting/heap_sort_2.c #1443

New Issue

claunia · 2026-01-29T15:20:43Z

claunia commented

2026-01-29 15:20:43 +00:00

Original Pull Request: https://github.com/TheAlgorithms/C/pull/914

State: closed
Merged: Yes

Fixed index of Parent in heapifyUp process!

parentIndex = [childIndex / 2] for 1-based indexing , but here we are doing the process in 0-based indexing, so the parentIndex should be = [(childIndex - 1) / 2]

References

Checklist

Added description of change
Relevant documentation/comments is changed or added
PR title follows semantic commit guidelines
I acknowledge that all my contributions will be made under the project's license.

Notes:

**Original Pull Request:** https://github.com/TheAlgorithms/C/pull/914 **State:** closed **Merged:** Yes --- #### Fixed index of Parent in heapifyUp process! > **_parentIndex = [childIndex / 2]_** for **_1-based indexing_** , but here we are doing the process in **_0-based indexing_**, so the _**parentIndex**_ should be = _**[(childIndex - 1) / 2]**_  #### References  #### Checklist  - [x] Added description of change - [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] 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/914"><img src="https://gitpod.io/button/open-in-gitpod.svg"/></a>

claunia added the pull-request label 2026-01-29 15:20:43 +00:00

claunia closed this issue

2026-01-29 15:20:44 +00:00

claunia referenced this issue

2026-01-29 15:27:55 +00:00

[PR #1443] add base conversion #2040

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/C#1443