Skip to content

libgetopts: doc fix. #16226

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from
Closed

libgetopts: doc fix. #16226

wants to merge 2 commits into from

Conversation

conradkleinespel
Copy link
Contributor

This confused me.

Seems like the Occur Multi enum allows the option to not appear at all, contrary to what the docs say.

@@ -128,7 +128,7 @@ pub enum Occur {
Req,
/// The option could or not occur.
Copy link
Contributor

Choose a reason for hiding this comment

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

While you're at it, you could update this one to be more consistent.

/// The option occurs zero or one times.

@conradkleinespel
Copy link
Contributor Author

Messed up the PR... doing a new one with a single commit.

@steveklabnik
Copy link
Member

You can just force push, rather than opening a new pull request.

@conradkleinespel
Copy link
Contributor Author

Oh. Didn't think of that :-( will do next time. Sorry about that.

@steveklabnik
Copy link
Member

It's all good! I say it purely for informational purposes :)

lnicola pushed a commit to lnicola/rust that referenced this pull request Jan 3, 2024
internal: Expose whether a channel has been dropped in lsp-server errors

Not the best way to expose this, but this should allow us to give somewhat better errors when the initialization request is malformed, as currently that just results in a channel disconnected error instead of the deserialization error. cc rust-lang/rust-analyzer#15859
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.

3 participants