Modify ↓

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#13785 closed enhancement (fixed)

[patch] refactoring bits to use streams + unit tests associated

Reported by: alno Owned by: team
Priority: trivial Milestone: 16.10
Component: Core Version: latest
Keywords: datasets, streams Cc:

Description (last modified by alno)

There are 2 patches:

  • one adds unit tests about some methods were streams can be used
  • the other actually changes these method loops into streams

Attachments (2)

use-of-streams-unit-tests.patch​ (11.8 KB ) - added by alno 8 years ago.
use-of-streams.patch​ (5.8 KB ) - added by alno 8 years ago.

Download all attachments as: .zip

Change History (5)

by alno, 8 years ago

Attachment: use-of-streams-unit-tests.patch​ added

by alno, 8 years ago

Attachment: use-of-streams.patch​ added

comment:1 by alno, 8 years ago

Description: modified (diff)

comment:2 by simon04, 8 years ago

Resolution: → fixed
Status: new → closed

In 11115/josm:

fix #13785 - Use streams, add unit tests (patch by alno, modified)

comment:3 by simon04, 8 years ago

Milestone: → 16.10

Thank you. Modifications before committing:

  • Unit tests go to test/unit
  • .stream().forEach → .forEach
  • we already had a ChangesetCacheTest.java

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.