[BUG] Sentinel search contains length comparison #104

New Issue

claunia · 2026-01-29T15:03:32Z

claunia commented

2026-01-29 15:03:32 +00:00

Originally created by @appgurueu on GitHub (Oct 15, 2021).

Description

See 3cfdbb040d (r58105657) - the point of sentinel search is literally to remove that comparison

Expected behavior

No comparison

Actual behavior

One comparison

Possible fix

Remove the if statement and the contained break

Steps to reproduce

None

Context

None

Additional information

No response

Originally created by @appgurueu on GitHub (Oct 15, 2021). ### Description See https://github.com/TheAlgorithms/C/commit/3cfdbb040db7e905b7074340196ddd23ee64fae6#r58105657 - the point of sentinel search is literally to remove that comparison ### Expected behavior No comparison ### Actual behavior One comparison ### Possible fix Remove the `if` statement and the contained `break` ### Steps to reproduce None ### Context None ### Additional information _No response_

claunia added the bug label 2026-01-29 15:03:32 +00:00

claunia closed this issue

2026-01-29 15:03:32 +00:00

claunia commented

2026-01-29 15:03:33 +00:00

@Panquesito7 commented on GitHub (Oct 15, 2021):

Would you like to work on this, @appgurueu? 🙂

@Panquesito7 commented on GitHub (Oct 15, 2021): Would you like to work on this, @appgurueu? 🙂

claunia referenced this issue

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

[PR #104] [MERGED] #102 pull request heap sort is modified at input process to receive an integer directly. #330

Sign in to join this conversation.