Merge pull request #119 from facelessuser/bugfix/step-loop

Fix for long running step loop
facelessuser · Feb 19, 2022 · ccdd424 · ccdd424
2 parents 59ffef1 + 8dad0dd
commit ccdd424
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/coloraide/__meta__.py b/coloraide/__meta__.py
@@ -192,5 +192,5 @@ def parse_version(ver: str) -> Version:
     return Version(major, minor, micro, release, pre, post, dev)
 
 
-__version_info__ = Version(0, 9, 0, "final")
+__version_info__ = Version(0, 10, 0, "final")
 __version__ = __version_info__._get_canonical()
diff --git a/coloraide/interpolate.py b/coloraide/interpolate.py
@@ -390,8 +390,7 @@ def color_steps(
 
         # If we currently have delta over our limit inject more stops.
         # If inserting between every color would push us over the max_steps, halt.
-        count = len(ret)
-        while m_delta > max_delta_e and (count * 2 - 1 <= max_steps):
+        while m_delta > max_delta_e and (len(ret) * 2 - 1 <= max_steps):
             # Inject stops while measuring again to see if it was sufficient
             m_delta = 0.0
             i = 1

diff --git a/docs/src/markdown/about/changelog.md b/docs/src/markdown/about/changelog.md
@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.10.0
+
+- **FIX**: Fix an issue where when attempting to generate steps some ∆E distance apart, the maximum step range was not
+  respected and could result in large hangs.
+
 ## 0.9.0
 
 !!! warning "Breaking Change"