Bug Tracker

Opened 9 years ago

Closed 9 years ago

Last modified 6 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 9 years ago.
patch to remove unnecessary use of Math.max

Download all attachments as: .zip

Change History (3)

Changed 9 years ago by lrbabe

Attachment: patch.txt added

patch to remove unnecessary use of Math.max

comment:1 Changed 9 years ago by flesler

Cc: lrbabe added
Component: coredimensions
Owner: set to flesler
Status: newassigned

Will look into this for 1.3. Thanks!

comment:2 Changed 9 years ago by flesler

Resolution: fixed
Status: assignedclosed

Fixed at [5985].

Note: See TracTickets for help on using tickets.