Ticket #2603 (closed feature: fixed)

Opened 3 years ago

Last modified 3 years ago

Layer.Grid.setTileSize no need to clone size

Reported by: fredj Owned by: elemoine
Priority: trivial Milestone: 2.10 Release
Component: Layer.Grid Version: SVN
Keywords: Cc:
State: Complete

Description

The size object returned by Map.getSize is already a clone, no need to clone the clone.

Attachments

2603.0.patch Download (505 bytes) - added by fredj 3 years ago.

Change History

Changed 3 years ago by fredj

Changed 3 years ago by fredj

  • state set to Review

Tests still pass, please review.

Changed 3 years ago by elemoine

  • state changed from Review to Commit

Looks good fredj, thanks.

Changed 3 years ago by fredj

  • status changed from new to closed
  • state changed from Commit to Complete
  • resolution set to fixed

(In [10377]) don't clone the size from map.getSize(). r=elemoine (closes #2603)

Note: See TracTickets for help on using tickets.