OpenLayers OpenLayers

Ticket #625 (closed task: fixed)

Opened 2 years ago

Last modified 1 year ago

no reason for separate _observeAndCache() function

Reported by: euzuro Assigned to:
Priority: minor Milestone: 2.4 Release
Component: Events Version:
Keywords: Cc:
State:

Description

the _observeAndCache() function is only used once, right above, called from observe. there is just no reason to have this be a separate function. merge it with its caller. makes code eaiser to read.

Attachments

observe.patch (1.0 kB) - added by euzuro on 04/02/07 21:49:46.
merge functions

Change History

04/02/07 21:49:46 changed by euzuro

  • attachment observe.patch added.

merge functions

04/02/07 21:53:08 changed by euzuro

  • keywords set to review.

04/03/07 11:40:09 changed by sderle

  • keywords changed from review to commit.

I agree that this makes sense. please commit.

04/03/07 11:54:51 changed by euzuro

  • status changed from new to closed.
  • resolution set to fixed.

r2991

07/05/07 16:54:39 changed by euzuro

  • keywords deleted.