Merge pull request #58 from baloo/baloo/unused-mutability

`Extension::parse` doesn't require mutability

Author: baloo

examples/key_storage.rs

@@ -200,10 +200,7 @@ impl Session for KeyStorage {
         Ok(())
     }
 
-    async fn extension(
-        &mut self,
-        mut extension: Extension,
-    ) -> Result<Option<Extension>, AgentError> {
+    async fn extension(&mut self, extension: Extension) -> Result<Option<Extension>, AgentError> {
         info!("Extension: {extension:?}");
 
         match extension.name.as_str() {

src/proto/message.rs

@@ -557,7 +557,7 @@ impl Extension {
     /// If there is a mismatch between the extension name
     /// and the [`MessageExtension::NAME`], this method
     /// will return [`None`]
-    pub fn parse_message<T>(&mut self) -> std::result::Result<Option<T>, <T as Decode>::Error>
+    pub fn parse_message<T>(&self) -> std::result::Result<Option<T>, <T as Decode>::Error>
     where
         T: MessageExtension + Decode,
     {
@@ -638,7 +638,7 @@ pub struct Unparsed(pub Vec<u8>);
 
 impl Unparsed {
     /// Decode unparsed bytes as SSH structures.
-    pub fn parse<T>(&mut self) -> std::result::Result<T, <T as Decode>::Error>
+    pub fn parse<T>(&self) -> std::result::Result<T, <T as Decode>::Error>
     where
         T: Decode,
     {