Remove parsing module (#2389)

The parsing module only contained the `FromStringOptional` trait implemented for a few primitives and implemented by our `create_enum` macros. But it's not used, and I could find no use of it even in the `legacy` branch. Maybe it once was, but it really is just `FromStr` now with `FromStr::Err` set to `typespec::error::Error`. No need to keep it, and it just creates problems for re-exporting macros into `azure_core` without necessitating a dependency on `typespec_client_core`.

Author: heaths

sdk/core/azure_core/CHANGELOG.md

@@ -45,6 +45,7 @@
 - Removed `EMPTY_BODY` constant from root.
 - Removed `future!()` macro.
 - Removed `Header` re-export from `http` module. It is still defined in the `http::headers` module.
+- Removed `parsing` module.
 - Removed `query_param` module.
 - Removed `RequestId` type alias from root.
 - Removed `SessionToken` type alias from root.

sdk/core/azure_core/src/lib.rs

@@ -24,7 +24,7 @@ pub use constants::*;
 pub use typespec_client_core::{
     base64, create_enum, create_extensible_enum, date,
     error::{self, Error, Result},
-    fmt, json, parsing, sleep, stream, Bytes, Uuid,
+    fmt, json, sleep, stream, Bytes, Uuid,
 };
 
 #[cfg(feature = "xml")]

sdk/typespec/typespec_client_core/CHANGELOG.md

@@ -9,6 +9,7 @@
 - Consolidated all the `tokio` features into a single feature named `tokio`. Traits remain separate but `tokio` support is enabled with a single feature.
 - Removed `Header` re-export from `http` module. It is still defined in the `http::headers` module.
 - Removed `Pager`.
+- Removed `parsing` module.
 
 ### Bugs Fixed
 

sdk/typespec/typespec_client_core/src/lib.rs

@@ -15,7 +15,6 @@ pub mod fs;
 pub mod http;
 #[cfg(feature = "json")]
 pub mod json;
-pub mod parsing;
 pub mod sleep;
 pub mod stream;
 #[cfg(feature = "xml")]

sdk/typespec/typespec_client_core/src/macros.rs

@@ -101,12 +101,6 @@ macro_rules! create_enum {
                 return s.serialize_str(&self.to_string())
             }
         }
-
-        impl $crate::parsing::FromStringOptional<$name> for $name {
-            fn from_str_optional(s : &str) -> $crate::error::Result<$name> {
-                s.parse::<$name>().map_err(::core::convert::Into::into)
-            }
-        }
     );
 }