Refactor DenseMatrix._forEach

[dvd101x]

Hi,

This is a refactor of DenseMatrix._forEach method without affecting performance.

This is in preparation for a non indexed version of that method.

[dvd101x]

BTW this is only a suggestion and not necessary to make the non indexed version.

[josdejong]

That is a neat simplification of the code! Nice.

The chart that shows benchmarks with the map function, but I think that is unrelated to the changes in _forEach. Can you check whether the performance of forEach remains the same?

[dvd101x]

Hi. Yes of course. Currently DenseMatrix._forEach is used in both DenseMatrix.map and DenseMatrix.forEach. I will update the chart with both.

[dvd101x]

Here is the chart with both benchmarks.

[josdejong]

Hi. Yes of course. Currently DenseMatrix._forEach is used in both DenseMatrix.map and DenseMatrix.forEach. I will update the chart with both.

Ah, clear. Thanks!