Add serde::Serialize support for AisMessage #25

salsabiljb · 2024-10-14T16:23:22Z

Implemented serde::Serialize for the AisMessage enum and all its variants:
* This enables serialization of AIS messages into JSON format.
Implemented the fmt::Display trait for AisMessage:
*This provides a readable string representation for each AisMessage variant.

src/messages/mod.rs

src/messages/addressed_safety_related.rs

src/messages/aid_to_navigation_report.rs

src/messages/binary_acknowledge.rs

src/messages/binary_addressed.rs

src/messages/binary_broadcast_message.rs

src/messages/interrogation.rs

src/messages/safety_related_acknowledgment.rs

src/messages/safety_related_broadcast.rs

src/messages/static_and_voyage_related_data.rs

github-advanced-security

clippy found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

src/lib.rs

+use serde_json::Error as SerdeError;
+
+/// Serializes an `AisSentence` to JSON
+pub fn serialize_to_json(sentence: &AisSentence) -> std::result::Result<String, SerdeError> {


src/lib.rs

+use serde_json::Error as SerdeError;
+
+/// Serializes an `AisSentence` to JSON
+pub fn serialize_to_json(sentence: &AisSentence) -> std::result::Result<String, SerdeError> {


src/lib.rs

+}
+
+/// Deserializes an `AisSentence` from JSON
+pub fn deserialize_from_json(json_data: &str) -> std::result::Result<AisSentence, SerdeError> {


github-advanced-security bot found potential problems Oct 14, 2024

View reviewed changes

src/messages/mod.rs Fixed Show fixed Hide fixed

github-advanced-security bot found potential problems Oct 14, 2024

View reviewed changes

src/messages/mod.rs Fixed Show resolved Hide resolved

github-advanced-security bot found potential problems Oct 14, 2024

View reviewed changes

github-advanced-security bot found potential problems Nov 11, 2024

View reviewed changes

Add serde::Serialize support for AisMessage

2cc0521

salsabiljb force-pushed the serialize branch from 84c1d75 to 2cc0521 Compare November 11, 2024 00:29

fixed stable(alloc) check

3628919

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add serde::Serialize support for AisMessage #25

Add serde::Serialize support for AisMessage #25

salsabiljb commented Oct 14, 2024

github-advanced-security bot left a comment

Add serde::Serialize support for AisMessage #25

Are you sure you want to change the base?

Add serde::Serialize support for AisMessage #25

Conversation

salsabiljb commented Oct 14, 2024

github-advanced-security bot left a comment

Choose a reason for hiding this comment