Use a semantic result type for tag.repeat

alpine-alpaca · Mar 10, 2024 · ba9f5ba · ba9f5ba
1 parent c83b4e3
commit ba9f5ba
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,11 @@
 # Changelog
 
+## Unreleased
+
+- Parse `repeat` property of a tag. [#23]
+
+[#23]: https://github.com/alpine-alpaca/asefile/pull/23
+
 ## 0.3.7 - 2023-03-03
 
 - Fix detection of pixel format for other editors (thanks to [@martinezjavier], [#22])

diff --git a/src/tags.rs b/src/tags.rs
@@ -1,3 +1,5 @@
+use std::num::NonZeroU32;
+
 use crate::{reader::AseReader, user_data::UserData, AsepriteParseError, Result};
 
 /// A tag is a grouping of one or more frames.
@@ -36,8 +38,10 @@ impl Tag {
     }
 
     /// Repeat count included in the tag.
-    pub fn repeat(&self) -> u32 {
-        self.repeat as u32
+    ///
+    /// `None` if unspecified/not enabled (UI shows infinity symbol).
+    pub fn repeat(&self) -> Option<NonZeroU32> {
+        NonZeroU32::new(self.repeat as u32)
     }
 
     /// Returns the user data for the tag, if any exists.