Side navigation
#3702 closed enhancement (fixed)
Opened December 09, 2008 01:23AM UTC
Closed December 25, 2008 05:10PM UTC
Last modified March 14, 2012 10:47PM UTC
Math.max should be able to handle more than two args
Reported by: | lrbabe | Owned by: | flesler |
---|---|---|---|
Priority: | trivial | Milestone: | 1.3 |
Component: | dimensions | Version: | 1.2.6 |
Keywords: | Cc: | lrbabe | |
Blocked by: | Blocking: |
Description
We can spare some bytes and be more efficient by removing unnecessary use of Math.max
The syntax:
Math.max( Math.max(1,2), Math.max(3,4) )
can be replaced by this one:
Math.max( 1, 2, 3, 4 )
Will look into this for 1.3.
Thanks!