https://issues.apache.org/bugzilla/show_bug.cgi?id=56705
Chris Boyle <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW --- Comment #2 from Chris Boyle <[email protected]> --- Having just looked at this again: the map is a LinkedHashMap and most of the required operations are possible using only the map with no worse efficiency than before (walk the map's values() which is no worse than walking the previous List). Unfortunately it all falls down at setSheetOrder(sheetname, pos). With LinkedHashMap it would have to remove and reinsert every sheet between its desired position and the end. You would need a kind of LinkedHashMap with the ability to reorder its internal list. I don't know of one and I'm not sure adding one and maintaining it would be an improvement over the patch as it stands. -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
