Skip to content
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

Switch from RAII to OBRM terminology #1624

Open
9SMTM6 opened this issue Oct 11, 2022 · 2 comments
Open

Switch from RAII to OBRM terminology #1624

9SMTM6 opened this issue Oct 11, 2022 · 2 comments

Comments

@9SMTM6
Copy link

9SMTM6 commented Oct 11, 2022

Same as in rust-unofficial/patterns#322, as I found out in a discussion, a lot of the documentation - even this official one - still primarily or even exclusively uses RAII terminology for that feature.

AFAIK OBRM is the official term, and based on my own experience and others feedback it's a lot more intuitive.

As referenced in the linked PR I don't want to remove RAII references, just switch to primarily using OBRM terminology, and as with that issue I'll hopefully get to making a PR on this over the weekend, but others can feel free to to it themselves before that. And of course here too I'd first like to know that such a PR would actually be appreciated.

@9SMTM6
Copy link
Author

9SMTM6 commented Oct 11, 2022

@9SMTM6
Copy link
Author

9SMTM6 commented Oct 16, 2022

Sadly I was not able to get to both this as well as the pattern book over this weekend, as in the Pattern book there appears to be need for diskussion I priorized that.

If there would be an aknowledgement that this issue and the proposed solution is acceptable that would also be great, I dont want to waste my time.

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

No branches or pull requests

1 participant