100-exercises-to-learn-rust icon indicating copy to clipboard operation
100-exercises-to-learn-rust copied to clipboard
verified publisher icon mainmatter

fix typo in comment

Open Espand-Masoon opened this issue 10 months ago • 2 comments

Espand-Masoon avatar Jan 28 '25 09:01 Espand-Masoon

Hi @MHMasoon Thanks for helping out! This here's not actually a typo: 255 can't fit in an i8. Do you have any suggestions for improving the wording so it becomes more clear?

hdoordt avatar Jan 28 '25 09:01 hdoordt

Hi @hdoordt, I just made this comment the same as it is in the "solutions" branch. 255u8 can't fit in an i8 but can be converted to it when #[allow(overflowing_literals)] is used; that's what the comment suggests. As for making it more clear, I don't have any suggestions, but I think the current version encourages students to explore and find the solution. A more explicit comment might spoon-feed them the answer.

Espand-Masoon avatar Jan 28 '25 10:01 Espand-Masoon