This project is archived and is in readonly mode.

#174 ✓invalid
ronin-24756 (at lighthouseapp)

many "else" useless

Reported by ronin-24756 (at lighthouseapp) | June 28th, 2008 @ 05:07 PM | in 2.0 (closed)

Example:

if (this.events[type].contains(fn))

return false;

else

this.events[type].push(fn);

Should be:

if (this.events[type].contains(fn))

return false;

this.events[type].push(fn);

I was looking at the code and there are a lot of these.

Remove those and the library will execute a litle faster.

Comments and changes to this ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Shared Ticket Bins

People watching this ticket

Pages