Modify ↓

Opened 5 years ago

Closed 5 years ago

#1588 closed defect (fixed)

Various cleaning up

Reported by: bruce89 Owned by: framm
Priority: minor Component: Core
Version: latest Keywords:
Cc:

Description

This patch removes as many warnings as I can make sense of. It contains removal of unused imports as well as other changes.

Attachments (1)

cleanup.patch​ (32.2 KB) - added by bruce89 5 years ago.
Cleanup

Download all attachments as: .zip

Change History (3)

Changed 5 years ago by bruce89

Cleanup

comment:1 Changed 5 years ago by anonymous

about the change in ExtrudeAction: The leftover line:

77: try { Integer.parseInt(Main.pref.get("edit.initial-move-threshold","5")); } catch (NumberFormatException x) {}

is useless. I'd suggest to drop it completely.

---

About the change from "main.ds." to "DataSet.": Um, at first glance this seems to be a bad idea. Someone knowing the internals of JOSM better should check this.

comment:2 Changed 5 years ago by anonymous

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

Applied in r999.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.