Change object member functions to use regular functions

[limzykenneth]

Resolves #3875

Changes:

As per the issue, prototype methods are all reverted to using regular function regardless of whether they use this in the function body or not. One additional thing I think is necessary but I'm not 100% sure about (90% sure) is static methods of objects. From what I found, in particular this stackoverflow question, it seems it is good practice to use regular functions for static members but I found only reference to class syntax static members only.

In any case, none of the converted functions use this in their function body so the functionalities will be exactly the same as before.

PR Checklist

• npm run lint passes
• Inline documentation is included / updated
• Unit tests are included / updated
• Benchmarks are included / updated

[lmccart]

thanks @limzykenneth this all looks good. I think you're correct on using regular functions for all static members.