Skip to content

Comment cleanup#149

Open
thejpster wants to merge 1 commit intorust-embedded:mainfrom
thejpster:comment-cleanup
Open

Comment cleanup#149
thejpster wants to merge 1 commit intorust-embedded:mainfrom
thejpster:comment-cleanup

Conversation

@thejpster
Copy link
Copy Markdown
Contributor

@thejpster thejpster commented Mar 28, 2026

Comment updates.

  • Turned on deny(missing_docs), and added missing docs
  • Turned on deny(clippy::missing_safety_doc) and added missing safety docs
  • Turned on some more useful lints
  • Clippy fixes
  • Rationalised/renamed the various CachePolicy types

@wt
Copy link
Copy Markdown

wt commented Apr 1, 2026

I'm curious. Did you write all of these or is this generated from some other source?

@thejpster
Copy link
Copy Markdown
Contributor Author

Mostly copy-pasted from the arm Arm ARM.

@thejpster thejpster changed the title Comment cleanup Comment cleanup (includes #131) Apr 3, 2026
@robamu
Copy link
Copy Markdown
Contributor

robamu commented Apr 16, 2026

I am a bit confused. This was rebased, but I thought this only includes docs updates, but I still see various new test cases etc. in the changes?

@robamu
Copy link
Copy Markdown
Contributor

robamu commented Apr 16, 2026

Ah, I get it. #131 was not merged yet :)

@jonathanpallant
Copy link
Copy Markdown
Contributor

#131 is in so I'll rebase this

@thejpster thejpster force-pushed the comment-cleanup branch 3 times, most recently from d351452 to bf3f8f3 Compare April 18, 2026 17:10
@jonathanpallant
Copy link
Copy Markdown
Contributor

#158 updates the tests to use the non-deprecated function, so this PR now needs re-working to not un-deprecate the function.

* Turned on deny(missing_docs), and added missing docs
* Turned on deny(clippy::missing_safety_doc) and added missing safety docs
* Turned on some more useful lints
* Clippy fixes
* Rationalised/renamed the various CachePolicy types
@jonathanpallant
Copy link
Copy Markdown
Contributor

Re-worked to leave the process function deprecated.

@jonathanpallant jonathanpallant changed the title Comment cleanup (includes #131) Comment cleanup Apr 22, 2026
@jonathanpallant
Copy link
Copy Markdown
Contributor

I'm moving the arm-targets stuff to another PR because it's orthogonal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants