Bug Tracker

Opened 14 years ago

Closed 14 years ago

Last modified 11 years ago

#3702 closed enhancement (fixed)

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 )

Attachments (1)

patch.txt​ (809 bytes) - added by lrbabe 14 years ago.
patch to remove unnecessary use of Math.max

Download all attachments as: .zip

Change History (3)

Changed 14 years ago by lrbabe

Attachment: patch.txt​ added

patch to remove unnecessary use of Math.max

comment:1 Changed 14 years ago by flesler

Cc: lrbabe added
Component: core → dimensions
Owner: set to flesler
Status: new → assigned

Will look into this for 1.3. Thanks!

comment:2 Changed 14 years ago by flesler

Resolution: → fixed
Status: assigned → closed

Fixed at [5985].

Note: See TracTickets for help on using tickets.