Memory leak on 1.0.1 fixed on master, please T&R 1.0.2 with fix?

[martin-langhoff]

v1.0.1 has a nasty memory leak, which is fixed on master. Unfortunately...

• the commit which brings the fix is very untidy, commit msg unhelpful, changes a bunch of random things, including whitespace and ... a memory leak. See 9b02f2a46cd55d0570a3bcaa0631924874b53191
• there is no tagged release with the fix :-( !

I can confirm that v1.0.1 leaks calendars because it does not handle scope destroy, I can confirm that current master is fixed, and the code execution goes right through the scope.$on('destroy') handler installed in the patch above.

Please kindly tag and release a 1.0.2 with a fix?

[martin-langhoff]

It turns out current master has other unaddressed bugs. I have backported and tidied up the memleak fix at https://github.com/martin-langhoff/ui-calendar/releases/tag/1.0.2ml .

You can use this in bower with git://github.com/martin-langhoff/ui-calendar.git#1.0.2ml

[martin-langhoff]

If anyone considers opening a 1.0.x stable branch for this project, I would be happy to submit the fix as a PR, see https://github.com/martin-langhoff/ui-calendar/tree/1.0.x

[martin-langhoff]

The branch mentioned above passes the builtin tests.

[martin-langhoff]

This is addressed by 2702f1c on maint-1.0 branch.