[PR #928] [MERGED] Update dijkstra.c #1457

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

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/928
Author: @IdanBanani
Created: 1/8/2022
Status: Merged
Merged: 1/11/2022
Merged by: @mishraabhinn

Base: masterHead: master


📝 Commits (1)

📊 Changes

1 file changed (+2 additions, -1 deletions)

View changed files

📝 data_structures/graphs/dijkstra.c (+2 -1)

📄 Description

in this way valgrind checks will pass and result will stay the same (functions as a static variable), previously it assumed that the stack memory structure will stay the same on the next calls

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/928 **Author:** [@IdanBanani](https://github.com/IdanBanani) **Created:** 1/8/2022 **Status:** ✅ Merged **Merged:** 1/11/2022 **Merged by:** [@mishraabhinn](https://github.com/mishraabhinn) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`32a9874`](https://github.com/TheAlgorithms/C/commit/32a98749f8afa6fff00f36109d22ede4a46bda7b) Update dijkstra.c ### 📊 Changes **1 file changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `data_structures/graphs/dijkstra.c` (+2 -1) </details> ### 📄 Description in this way valgrind checks will pass and result will stay the same (functions as a static variable), previously it assumed that the stack memory structure will stay the same on the next calls #### 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 - [ ] 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 --> <a href="https://gitpod.io/#https://github.com/TheAlgorithms/C/pull/928"><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:20:56 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#1457