Bug Tracker

Opened 8 years ago

Closed 8 years ago

Last modified 5 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:


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 )

Change History (2)

comment:1 Changed 8 years ago by flesler

  • Cc lrbabe added
  • Component changed from core to dimensions
  • Owner set to flesler
  • Status changed from new to assigned

Will look into this for 1.3. Thanks!

comment:2 Changed 8 years ago by flesler

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed at [5985].

Note: See TracTickets for help on using tickets.