Changes between Initial Version and Version 1 of Ticket #11838, comment 1


Ignore:
Timestamp:
2015-09-14T22:33:57+02:00 (10 years ago)
Author:
simon04

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #11838, comment 1

    initial v1  
    99* In `MapView#getLayerPos`, the indentation seems incorrect (did you diff with `--ignore-all-space`?)
    1010* `LayerManager#getLayersOfType` claims returns a modifiable list, contradicting the Javadoc.
    11 * I think, we should reduce the synchronization for many `get*` methods of the `LayerManager`/`LayerManagerWithActive`. The `layers` field could be implemented as `CopyOnWriteArrayList` (since changes are far less frequent than reads).
     11* I think, we should reduce the synchronization for many `get*` methods of the `LayerManager`/`LayerManagerWithActive`. The `layers` field could be implemented as `CopyOnWriteArrayList` (since changes are far less frequent than reads). Or use a `ReentrantReadWriteLock` as in `DataSet#lock` …