Skip to content

Conversation

@titaniumtraveler
Copy link
Contributor

While reading the documentation/the source code (for the purpose of creating my own Deserializer for a different format) I found that in serde_json::de::Read &[u8] was rendered incorrectly.

So I did a quick grep and fixed all I could find by adding backticks.

I also switching the &[u8] in the README.md/src/lib.rs from escapes to backticks for consistency sake.
(I assume the README.md was generated from the src/lib.rs with some tool, but if both are changed the same way, it should still be consistent with the output.)

@titaniumtraveler titaniumtraveler changed the title Fix missing ` in doc comments letting markdown think &[u8] is a link to u8 Fix missing backtick in doc comments letting markdown think &[u8] is a link to u8 Mar 25, 2024
This was leading to rustdocs markdown parser to interpret slices like &[u8] as & and a link to u8
Copy link
Member

@dtolnay dtolnay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Good catch.

@dtolnay dtolnay merged commit c3dc153 into serde-rs:master Mar 26, 2024
@titaniumtraveler titaniumtraveler deleted the pr branch March 26, 2024 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants