[PR #237] [MERGED] Fix: Predictable iteration order #369

Closed
opened 2026-06-08 11:27:35 +03:00 by zhus · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/bootandy/dust/pull/237
Author: @bootandy
Created: 8/18/2022
Status: Merged
Merged: 8/18/2022
Merged by: @bootandy

Base: masterHead: pred_order


📝 Commits (1)

  • fd4cc07 Fix: Predictable iteration order

📊 Changes

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

View changed files

📝 src/dir_walker.rs (+18 -2)

📄 Description

Sort the Node objects so that duplicate inodes appear in a predictable
order.

Include new sort_by_inode method because this is notably quicker than
sorting by size & name as defined in the nodes.rs file.

Should fix: https://github.com/bootandy/dust/issues/234


🔄 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/bootandy/dust/pull/237 **Author:** [@bootandy](https://github.com/bootandy) **Created:** 8/18/2022 **Status:** ✅ Merged **Merged:** 8/18/2022 **Merged by:** [@bootandy](https://github.com/bootandy) **Base:** `master` ← **Head:** `pred_order` --- ### 📝 Commits (1) - [`fd4cc07`](https://github.com/bootandy/dust/commit/fd4cc07fae7cf5470b6e95686e3ff13604e7648c) Fix: Predictable iteration order ### 📊 Changes **1 file changed** (+18 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/dir_walker.rs` (+18 -2) </details> ### 📄 Description Sort the Node objects so that duplicate inodes appear in a predictable order. Include new sort_by_inode method because this is notably quicker than sorting by size & name as defined in the nodes.rs file. Should fix: https://github.com/bootandy/dust/issues/234 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
zhus added the pull-request label 2026-06-08 11:27:35 +03:00
zhus closed this issue 2026-06-08 11:27:37 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bootandy/archived-dust#369