1772: Remove lifetime parameter from `NativeFunc` r=MarkMcCaskey a=MarkMcCaskey
Ran into this as an annoyance on #1739 ; will need to account for this when we fix the memory leak in `InstanceHandle`.
Note: the lifetime wasn't doing anything useful for us, this change doesn't make anything new possible (other than not having to deal with the lifetime parameter)
# Review
- [x] Add a short description of the the change to the CHANGELOG.md file
Co-authored-by: Mark McCaskey <mark@wasmer.io>
Also fix a syntax error, use assert_eq! instead of assert! to compare two values for equality.
Also in passing, convert a comment about something not being done into a TODO.
`NativeFunc::call` is about 8% faster in the case we're benchmarking
by avoiding allocating a vector for the params / returns, instead we
do logic to determine which is larger and use that, conditionally
copying it back to the rets array if needed.
