Remove more bounds checks

[yuyichao]

Mostly for nzrange but also added more @inbounds as I went through the files. All of the functions that has additional @inbounds either takes no user index input or have already performed explicit checks on the index (e.g. isstored).

[codecov]

Codecov Report

❌ Patch coverage is 98.07692% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 84.18%. Comparing base (8b03562) to head (7c39af1).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
src/linalg.jl	97.14%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #661      +/-   ##
==========================================
+ Coverage   84.15%   84.18%   +0.03%     
==========================================
  Files          12       12              
  Lines        9300     9301       +1     
==========================================
+ Hits         7826     7830       +4     
+ Misses       1474     1471       -3     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ViralBShah]

Does the @inbounds have measurable impact on performance? I guess there is no major challenge here since all the usage is safe.

[yuyichao]

The inbounds on nzrange have measurable performance difference for small matrices (less than 10x10 and for about 10%). Definitely not for larger matrices. I did not benchmark the other functions.

[dkarrasch]

This was merged after the v1.13 split. Should this be backported?

[ViralBShah]

Yes, we should backport this.