Deprecate .prefixes(), .resolve(), .resolve_attribute(), and .resolve_element() of NsReader

Use `NamespaceResolver` methods instead of `NsReader` methods

Author: Mingun

Changelog.md

@@ -21,8 +21,11 @@
 ### Misc Changes
 
 - [#908]: Increase minimal supported `serde` version from 1.0.139 to 1.0.180.
+- [#913]: Deprecate `.prefixes()`, `.resolve()`, `.resolve_attribute()`, and `.resolve_element()`
+  of `NsReader`. Use `.resolver().bindings()` and `.resolver().resolve()` methods instead.
 
 [#908]: https://github.com/tafia/quick-xml/pull/908
+[#913]: https://github.com/tafia/quick-xml/pull/913
 
 
 ## 0.38.4 -- 2025-11-??

src/name.rs

@@ -365,13 +365,11 @@ impl<'a> AsRef<[u8]> for Namespace<'a> {
 
 ////////////////////////////////////////////////////////////////////////////////////////////////////
 
-/// Result of [prefix] resolution which creates by [`NamespaceResolver::resolve`], [`NsReader::resolve_attribute`],
-/// [`NsReader::resolve_element`], [`NsReader::read_resolved_event`] and
+/// Result of [prefix] resolution which creates by [`NamespaceResolver::resolve`],
+/// [`NsReader::read_resolved_event`] and
 /// [`NsReader::read_resolved_event_into`] methods.
 ///
 /// [prefix]: Prefix
-/// [`NsReader::resolve_attribute`]: crate::reader::NsReader::resolve_attribute
-/// [`NsReader::resolve_element`]: crate::reader::NsReader::resolve_element
 /// [`NsReader::read_resolved_event`]: crate::reader::NsReader::read_resolved_event
 /// [`NsReader::read_resolved_event_into`]: crate::reader::NsReader::read_resolved_event_into
 #[derive(Clone, PartialEq, Eq, Hash)]
@@ -1084,6 +1082,7 @@ impl<'a> Iterator for NamespaceBindingsIter<'a> {
 impl<'a> FusedIterator for NamespaceBindingsIter<'a> {}
 
 /// The previous name for [`NamespaceBindingsIter`].
+#[deprecated = "Use NamespaceBindingsIter instead"]
 pub type PrefixIter<'a> = NamespaceBindingsIter<'a>;
 
 /// Iterator on the declared namespace bindings on specified level. Returns pairs of the _(prefix, namespace)_.

src/reader/ns_reader.rs

@@ -130,6 +130,7 @@ impl<R> NsReader<R> {
     /// # quick_xml::Result::Ok(())
     /// ```
     #[inline]
+    #[deprecated = "Use `.resolver().bindings()` instead"]
     pub const fn prefixes(&self) -> NamespaceBindingsIter<'_> {
         self.ns_resolver.bindings()
     }
@@ -241,6 +242,7 @@ impl<R> NsReader<R> {
     /// [`resolve_attribute()`]: Self::resolve_attribute()
     /// [`resolve_element()`]: Self::resolve_element()
     #[inline]
+    #[deprecated = "Use `.resolver().resolve()` instead. Note, that boolean argument should be inverted!"]
     pub fn resolve<'n>(
         &self,
         name: QName<'n>,
@@ -301,6 +303,7 @@ impl<R> NsReader<R> {
     /// [`Unknown`]: ResolveResult::Unknown
     /// [`read_resolved_event()`]: Self::read_resolved_event
     #[inline]
+    #[deprecated = "Use `.resolver().resolve_element(name)` instead"]
     pub fn resolve_element<'n>(&self, name: QName<'n>) -> (ResolveResult<'_>, LocalName<'n>) {
         self.ns_resolver.resolve_element(name)
     }
@@ -371,6 +374,7 @@ impl<R> NsReader<R> {
     /// [`Unbound`]: ResolveResult::Unbound
     /// [`Unknown`]: ResolveResult::Unknown
     #[inline]
+    #[deprecated = "Use `.resolver().resolve_attribute(name)` instead"]
     pub fn resolve_attribute<'n>(&self, name: QName<'n>) -> (ResolveResult<'_>, LocalName<'n>) {
         self.ns_resolver.resolve_attribute(name)
     }