computablee / DotMP

A collection of powerful abstractions for parallel programming in .NET with an OpenMP-like API.

https://computablee.github.io/DotMP/

GNU Lesser General Public License v2.1

29 stars 7 forks source link

Add error checking to ForCollapse #120

Closed computablee closed 11 months ago

computablee commented 11 months ago

Which issue are you addressing?

Makes progress towards #115

How have you addressed the issue?

Overflow checking is now performed within ForCollapse loops, throwing a TooManyIterationsException if there are more than int.MaxValue total iterations to execute. Additionally, proper argument checking is now performed on ForCollapse loops.

How have you tested your patch?

A new test has been added to check for this situation, which passes.

codecov[bot] commented 11 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b5b6921) 99.13% compared to head (7ed086a) 99.22%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #120 +/- ## ========================================== + Coverage 99.13% 99.22% +0.09% ========================================== Files 12 12 Lines 1270 1291 +21 Branches 134 137 +3 ========================================== + Hits 1259 1281 +22 Misses 5 5 + Partials 6 5 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.