Update comment, types

[adamchalmers]

We can use the NonZeroU32 to ensure the Rust code checks if the users passes 0, before passing any u32 to the C++ codebase. You can convert it into u32 using .try_into() or u32::try_from().

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 26.84%. Comparing base (c6829d0) to head (de626bd).
Report is 1 commits behind head on mike/lofting-enums.

Additional details and impacted files

@@                  Coverage Diff                   @@
##           mike/lofting-enums     #470      +/-   ##
======================================================
+ Coverage               26.42%   26.84%   +0.41%     
======================================================
  Files                      29       29              
  Lines                    1680     1654      -26     
======================================================
  Hits                      444      444              
+ Misses                   1236     1210      -26     

Flag	Coverage Δ
unittests	26.84% <ø> (+0.41%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jtran]

I generally prefer T::from() and T::try_from() over their into counterparts because it's explicit about the type you're converting to (imagine when a code change causes the expected return type to change) and you can go to definition. OTOH, the definition of into() is always the same generic function that calls from() so you can't easily jump to the real conversion code.