JsEvaluator mResultCallback is growing and do not realease previous callbacks

[clydzik]

i realized that this list is holding results callbacks (probably for matching request/response interface) but as self is not maintained after finish (success or failure)
com.evgenii.jsevaluator.JsEvaluator#mResultCallbacks

the workaround is to use
jsEvaluator.getResultCallbacks().remove();

but this is not documented and a bit awkward

[evgenyneu]

That's a very good point! I will look at this. Pull request is welcome as well. 👍

[clydzik]

Yep most cases listeners are small so this make not much harm. But with inner classes can produce bigger leaks. Any way good to hear. I m taking some vacations now... Will follow from September on this. Will contribute than if still valid. Thanks.

[evgenyneu]

That's good to know, thanks.

[clydzik]

i provided some proposal for change. please letm me know what you think.

[evgenyneu]

Thank you! I will have a look.

[evgenyneu]

Thanks @clydzik for fixing the issue.

[clydzik]

np. @evgenyneu thanks for the library!