Skip to content

Fix heapsort implementation#34

Merged
cbl merged 11 commits intomainfrom
jonas/feat/heapsort
Jul 26, 2021
Merged

Fix heapsort implementation#34
cbl merged 11 commits intomainfrom
jonas/feat/heapsort

Conversation

@jonasclauss
Copy link
Collaborator

I've written with Mr. Weickert because I've recogniced a mistake in his example in the course. You have at least one write acces per calling 'descendMaxHeap`. I've fixed the mistake and now comparison and writing should be correct.
Bildschirmfoto 2021-07-23 um 22 08 12
Heapsort

jonasclauss and others added 10 commits July 23, 2021 13:55
Co-authored-by: github-actions[bot] <action@github.com>
Co-authored-by: github-actions[bot] <action@github.com>
It was added unaware of the already existing method ArryWriter#set
and is an unwanted duplicate of the former.
Co-authored-by: github-actions[bot] <action@github.com>
@jonasclauss jonasclauss requested review from cbl and herrmanno July 23, 2021 20:08
@cbl cbl changed the title Jonas/feat/heapsort Fix heapsort implementation Jul 23, 2021
@cbl cbl merged commit 48567b8 into main Jul 26, 2021
@cbl cbl deleted the jonas/feat/heapsort branch July 27, 2021 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants