Modify ↓

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#19918 closed enhancement (fixed)

Fix fixme request [patch]

Reported by: gaben Owned by: team
Priority: trivial Milestone: 20.11
Component: Core Version:
Keywords: Cc:

Description

r3686 introduced a fixme comment which asks help to sort files by modify time.

Is it needed? 10 years passed and no modifications made since. If yes, add these lines:

import java.util.Comparator;
//...
Arrays.sort(files, Comparator.comparingLong(File::lastModified));

Attachments (1)

fix_fixme.patch​ (925 bytes ) - added by gaben 5 years ago.

Download all attachments as: .zip

Change History (5)

by gaben, 5 years ago

Attachment: fix_fixme.patch​ added

comment:1 by Klumbumbus, 5 years ago

Summary: Fix fixme request → Fix fixme request [patch]

comment:2 by simon04, 5 years ago

Resolution: → fixed
Status: new → closed

In 17181/josm:

fix #19918 - AutosaveTask: sort files by modify time (patch by gaben, modified)

comment:3 by simon04, 5 years ago

Milestone: → 20.10

Thank you!

The sorting is relevant inside #moveToDeletedLayersFolder which removes layers until autosave.deletedLayersBackupCount=5 is no longer exceeded.

comment:4 by Don-vip, 5 years ago

Milestone: 20.10 → 20.11

Milestone renamed

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.