Bug Tracker

Ticket #3702 (closed enhancement: fixed)

Opened 6 years ago

Last modified 3 years ago

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
Blocking: Blocked by:

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

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

Change History

Changed 6 years ago by lrbabe

patch to remove unnecessary use of Math.max

comment:1 Changed 6 years ago by flesler

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

Will look into this for 1.3. Thanks!

comment:2 Changed 6 years ago by flesler

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

Fixed at [5985].

Note: See TracTickets for help on using tickets.