Modify ↓

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#12924 closed enhancement (fixed)

[Patch] Do not use deprecated EditLayerChangedListener any more.

Reported by: michael2402 Owned by: team
Priority: normal Milestone: 16.06
Component: Core Version:
Keywords: gsoc-core Cc: Don-vip, bastiK, stoecker

Description

I replaced the EditLayerChangedListener by ActiveLayerChangedListener in the JOSM core codebase.

The new add/remove classes throw exceptions if the listener was added twice or if it was removed without being added. This might cause some bug reports but should help us trace down lifecycle issues.

I also changed addActiveLayerChangeListener(..., boolean initalFire) to use two methods instead of the boolean parameter.

Attachments (1)

patch-layer-manager-migrate-edit-layer-changed.patch​ (39.5 KB ) - added by michael2402 8 years ago.

Download all attachments as: .zip

Change History (3)

by michael2402, 8 years ago

Attachment: patch-layer-manager-migrate-edit-layer-changed.patch​ added

comment:1 by stoecker, 8 years ago

Resolution: → fixed
Status: new → closed

In 10332/josm:

Do not use deprecated EditLayerChangedListener - patch my Michael Zangl - gsoc-core - fix #12924

comment:2 by stoecker, 8 years ago

See #12932 for first results :-)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.