Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#304 Sort category can remove layer

open
nobody
7
2013-04-03
2013-04-03
michael michaud
No

Today, I had to sort a category containing about 40 layers. I used
Layer > Sort Selected Categories > By Name > Ascending
I get a ConcurrentModificationException and lost all the layers in the Category but one.
Too bad...

Discussion

  • ede
    ede
    2013-04-03

    try to reproduce it and save the error stack.. important for fixing to pinpoint where the concurrent mod happens.

    ..ede

     
  • Cannot reproduce it at home.
    It will be difficult as I did not notice any special operation I did during the sort operation.
    I'll try again tomorrow.

     
  • I can see two ways to make SortACategory more safe :
    - synchronizing removeLayers / addLayers in SortCategoryAbstractPlugIn
    - setting setFiringEvents to false during these operations
    Not sure which is the good approach, would like to reproduce the problem first.