search

BriefFiniteElementNet / BriefFiniteElement.Net

BriefFiniteElementDotNET (BFE.NET) is a library for linear-static Finite Element Method (FEM) analysis of solids and structures in .NET
GNU Lesser General Public License v3.0
154 stars 57 forks source link

Project cleanup: usage of params keyword. #80

Closed wo80 closed 3 years ago

wo80 commented 4 years ago

There are a lot of methods that use params double[] args as function parameter. Since this hides the array allocation, I suggest to remove the params keyword.

epsi1on commented 4 years ago

The param keyworks are at most size of 3, mostly for getting iso-parametric coordinates. Not sure but maybe array with three members (24 byte or less) do not cause serious problem

wo80 commented 4 years ago

There are a couple of methods in the Matrix class that use params, those will be removed with #72 . Doing a quick grep I found 227 occurances, but most of them seem to be related to iso coords, as you said, and I think that's a valid use case. Memory profiling will show if it's a problem.