Ticket #3073 (closed feature: fixed)

Opened 2 years ago

Last modified 2 years ago

add a reusable empty function

Reported by: tschaub Owned by: ahocevar
Priority: minor Milestone: 2.11 Release
Component: BaseTypes Version: 2.10
Keywords: Cc:
State: Complete

Description

For places where we create an anonymous empty function, we should be able to reuse one defined elsewhere.

Attachments

3703.patch Download (1.0 KB) - added by tschaub 2 years ago.
void

Change History

Changed 2 years ago by tschaub

void

Changed 2 years ago by tschaub

  • owner set to ahocevar
  • state set to Review

Tests pass in IE6 and FF3 (date parsing needs fixed in Chrome - unrelated).

Changed 2 years ago by ahocevar

  • state changed from Review to Commit

Looks good. Please commit. The only comment I have is that you could change the line in the test to follow the t.eq(got, expected, message) signature:

t.eq(OpenLayers.Function.Void(), undefined, "returns undefined");

Changed 2 years ago by fredj

  • status changed from new to closed
  • state changed from Commit to Complete
  • resolution set to fixed
Note: See TracTickets for help on using tickets.