From 122cae7902db9cafa138d115b7546f72cdf6b3ae Mon Sep 17 00:00:00 2001 From: Martin Nordholts Date: Wed, 22 Sep 2021 09:22:43 +0200 Subject: [PATCH] build_assets.rs: Sort first to make dependencies.dedup() actually useful --- src/assets/build_assets.rs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/assets/build_assets.rs b/src/assets/build_assets.rs index 357fc2e7..2df80594 100644 --- a/src/assets/build_assets.rs +++ b/src/assets/build_assets.rs @@ -20,8 +20,8 @@ type SyntaxToDependencies = HashMap>; type SyntaxToDependents<'a> = HashMap>; /// Represents some other `*.sublime-syntax` file, i.e. another [SyntaxDefinition]. -#[derive(Debug, Eq, PartialEq, Clone, Hash)] -enum OtherSyntax { +#[derive(Debug, Eq, PartialEq, PartialOrd, Ord, Clone, Hash)] +pub(crate) enum OtherSyntax { /// By name. Example YAML: `include: C.sublime-syntax` (name is `"C"`) ByName(String), @@ -304,6 +304,7 @@ fn dependencies_for_syntax(syntax: &SyntaxDefinition) -> Vec { .collect(); // No need to track a dependency more than once + dependencies.sort(); dependencies.dedup(); dependencies