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

The smart_pointers section is too big #1701

Open
kenden opened this issue Oct 3, 2023 · 1 comment
Open

The smart_pointers section is too big #1701

kenden opened this issue Oct 3, 2023 · 1 comment
Labels
A-exercises Area: Exercises

Comments

@kenden
Copy link

kenden commented Oct 3, 2023

When following the exercises, it is nice to check the suggested reading.
For the smart_pointers exercises, it includes a lot of reading:

  • Smart Pointers
    -> the full chapter about Smart Pointers in The Book. It is a big chapter.
  • Using Box to Point to Data on the Heap
    -> Repetition. This is included in the Smart Pointers chapter
  • Rc<T>, the Reference Counted Smart Pointer
    -> Repetition. This is included in the Smart Pointers chapter
  • Shared-State Concurrency
    -> This is included in the chapter Fearless Concurrency in The Book.
    To understand it, it is better to read the full chapter, and it is a Big chapter.
  • Cow Documentation
    -> Advanced topic about Copy-On-Write (actually, Clone-On-Write in Rust), a performance optimization technique, which requires to check other sources (example) to understand well. (This is not actually covered in The Book).

The exercises and reading to do before the Smart Pointers section are quite fast to get through. However, reading the references for 2 full chapters in The Book + researching about the Cow type is more time consuming. So, less engaging.

I would suggest to move the arc1 exercise under the threads topics to not have to read about concurrency yet.

Rustlings v5.6.1

@asasine
Copy link

asasine commented Jan 23, 2024

I would suggest to move the arc1 exercise under the threads topics to not have to read about concurrency yet.

I had the same thought and support this. I also think cow1 isn’t conceptually relevant until during/after threads so it could be moved too.

@shadows-withal shadows-withal added the A-exercises Area: Exercises label Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-exercises Area: Exercises
Projects
None yet
Development

No branches or pull requests

3 participants