Use Status::dup() to wrap borrowed SVN status as owned

Cargo.toml

@@ -16,5 +16,5 @@ edition = "2021"
 [workspace.dependencies]
 pyo3 = { version = "0.27" }
 #subversion = { version = ">=0.0.5" }
-subversion = { version = "0.1.4" }
+subversion = { version = "0.1.5" }
 pyo3-filelike = { version = "0.5" }

wc/src/status.rs

@@ -5,35 +5,95 @@ use pyo3::prelude::*;
 /// Working copy status for a node.
 #[pyclass(name = "Status", unsendable)]
 pub struct Status {
-    #[pyo3(get)]
-    pub kind: i32,
-    #[pyo3(get)]
-    pub depth: i32,
-    #[pyo3(get)]
-    pub filesize: i64,
-    #[pyo3(get)]
-    pub versioned: bool,
-    #[pyo3(get)]
-    pub repos_uuid: Option<String>,
-    #[pyo3(get)]
-    pub repos_root_url: Option<String>,
-    #[pyo3(get)]
-    pub repos_relpath: Option<String>,
+    inner: subversion::wc::Status<'static>,
 }
 
+#[pymethods]
 impl Status {
-    /// Create a Status from a subversion::wc::Status reference.
-    ///
-    /// Copies all fields since the SVN Status borrows from an APR pool.
+    #[getter]
+    fn node_status(&self) -> u32 {
+        self.inner.node_status() as u32
+    }
+
+    #[getter]
+    fn text_status(&self) -> u32 {
+        self.inner.text_status() as u32
+    }
+
+    #[getter]
+    fn prop_status(&self) -> u32 {
+        self.inner.prop_status() as u32
+    }
+
+    #[getter]
+    fn copied(&self) -> bool {
+        self.inner.copied()
+    }
+
+    #[getter]
+    fn switched(&self) -> bool {
+        self.inner.switched()
+    }
+
+    #[getter]
+    fn locked(&self) -> bool {
+        self.inner.locked()
+    }
+
+    #[getter]
+    fn revision(&self) -> i64 {
+        self.inner.revision().as_i64()
+    }
+
+    #[getter]
+    fn changed_rev(&self) -> i64 {
+        self.inner.changed_rev().as_i64()
+    }
+
+    #[getter]
+    fn kind(&self) -> i32 {
+        self.inner.kind()
+    }
+
+    #[getter]
+    fn depth(&self) -> i32 {
+        self.inner.depth()
+    }
+
+    #[getter]
+    fn filesize(&self) -> i64 {
+        self.inner.filesize()
+    }
+
+    #[getter]
+    fn versioned(&self) -> bool {
+        self.inner.versioned()
+    }
+
+    #[getter]
+    fn repos_uuid(&self) -> Option<String> {
+        self.inner.repos_uuid()
+    }
+
+    #[getter]
+    fn repos_root_url(&self) -> Option<String> {
+        self.inner.repos_root_url()
+    }
+
+    #[getter]
+    fn repos_relpath(&self) -> Option<String> {
+        self.inner.repos_relpath()
+    }
+}
+
+impl Status {
+    /// Wrap an owned SVN Status.
+    pub fn from_owned(s: subversion::wc::Status<'static>) -> Self {
+        Self { inner: s }
+    }
+
+    /// Dup a borrowed SVN Status into an owned copy.
     pub fn from_svn_status(s: &subversion::wc::Status<'_>) -> Self {
-        Self {
-            kind: s.kind(),
-            depth: s.depth(),
-            filesize: s.filesize(),
-            versioned: s.versioned(),
-            repos_uuid: s.repos_uuid(),
-            repos_root_url: s.repos_root_url(),
-            repos_relpath: s.repos_relpath(),
-        }
+        Self { inner: s.dup() }
     }
 }