tooltip.dynamic.js: variable leaks into global scope

[tbasse]

https://github.com/jquerytools/jquerytools/blob/master/src/tooltip/tooltip.dynamic.js#L147

ret is not defined in current scope.

[alibby251]

I've checked through the code - the ret variable is not defined in the scope, so this needs to be checked. Marked as High priority, as it is bad coding not to declare the variable; I've checked through the rest of the code for tooltip, and cannot see it being declared anywhere else?

[duclet]

This is an easy enough fix. I'll take care of it though I'm pushing this to 1.2.8 since we are already past the deadline for 1.2.7.

[alibby251]

Hi Duc,

Thanks - this is fine.

Alex.

On 17/03/2012 22:09, Duc Tri Le wrote:

This is an easy enough fix. I'll take care of it though I'm pushing this to 1.2.8 since we are already past the deadline for 1.2.7.

---

Reply to this email directly or view it on GitHub:
#563 (comment)

__________ Information from ESET NOD32 Antivirus, version of virus signature database 6975 (20120317) __________

The message was checked by ESET NOD32 Antivirus.

http://www.eset.com

[duclet]

Merged into dev for 1.2.8.