comprehensive-rust icon indicating copy to clipboard operation
comprehensive-rust copied to clipboard

Change String representation to actual

Open errge opened this issue 1 year ago • 2 comments

Tested with rust 2015, 2018, 2021 and 2024, on amd64 musl, amd64 glibc and aarch64 musl, all of them represent Strings with (ptr, capacity, len).

This is an internal implementation detail, that shouldn't be exposed anyway, so it's no big deal, but in the speaker notes, we provide a debugging tool for demonstration purposes, so at least that should have a correct output.

errge avatar Feb 14 '24 23:02 errge

Tested resulting website with mdbook serve, modified diagram renders correctly.

errge avatar Feb 14 '24 23:02 errge

Note: src/memory-management/move.md and src/smart-pointers/trait-objects.md also has this order in diagrams, I plan to fix them, when I fully reviewed those chapters of the slides/course.

I prefer to put those in separate pull requests, as here, the important fix is the debugging Rust code, because that was just incorrect.

errge avatar Feb 14 '24 23:02 errge