PulseAdapter has a confusing setup

[stefanhoth]

PulseAdapter needs to be able to sort entries by value.

This could be easily handled through a TreeMap with PulseEntry implementing Comparable (or a static Comparator) but instead the adapter stores values in an ArrayList<Map.Entry<String, PulseEntry>> which I would call confusing and inefficient.

[friedger]

@stefanhoth Could you elaborate how to get this less confusing?

The current constraints are:

• Pulse is a HashMap, that is why Map.Entry is used in the adapter
• The adapter needs to return an item for a given position
• The sorting depends on the mode the adapter is used in.

[friedger]

@stefanhoth Should we close this?