style: use member initializer in the move constructor of `Any` - Githubissues

julelang / jule

Effective programming language to build efficient, fast, reliable and safe software while maintaining simplicity

https://jule.dev

BSD 3-Clause "New" or "Revised" License

128 stars 13 forks source link

style: use member initializer in the move constructor of `Any` #77

Closed vil02 closed 7 months ago

vil02 commented 7 months ago

Description

Use member initializer.

By the way: having the following move constructor:

Any(const jule::Any &&src)

does not make much sense. The standard signature should be:

Any(jule::Any &&src)

I will change it in another PR.

Checklist

[x] A description of the changes in this PR is mentioned above.
[x] All the new and existing tests pass.
[x] The code follows the code style and conventions of the project.
[x] No plagiarized, duplicated, or repetitive code that has been directly copied from another source.
[x] I have read the whole Contributing guidelines of the project and its resources/related pages.

Screenshots (if any)

Note to reviewers

vil02 commented 7 months ago

LGTM. I will merge your PR after your draft done. Thanks for your contribution.

I just want to make sure that all of the checks pass.