Community
Participate
Working Groups
Created attachment 286563 [details] example screenshot of diff.png Since bug 180756 the attributes are saved ordered by name (by using treemap). If i change a launch configuration which was saved before (like for example org.eclipse.team.tests.cvs.core/launchConfigurations/benchmark/Benchmark Test.launch) or manually add a attribute to the launch configuration then this results in a big diff. I suggest to instead keep the order in which it was saved/loaded. This could probably easy archived by using a LinkedHashMap instead. I'd like you to give feedback if you agree about this before developing a patch. Another "problem" is that the launchconfiguration automatically adds new attributes like <stringAttribute key="checked" value="[NONE]"/> or clearwslog which i did not touch. I would like the launchconfigration to only save values that have been either already been loaded from file or deviate from the default settings to keep the git history clean.
*** Bug 574143 has been marked as a duplicate of this bug. ***
Jörg, you refer to bug 180756, but set the "version" to 4.21 - by mistake? Or do you see some regression in 4.21 (which is not released yet)? Also I'm not sure how do you get a big diff if everything is sorted? You only should have few lines sorted at right place? Or is the original version unsorted - but how do you get that one?
(In reply to Andrey Loskutov from comment #2) > Jörg, you refer to bug 180756, but set the "version" to 4.21 - by mistake? no regression. i just do not know the policy which version number should be set here so i just took the newest. The behaviour is very old. > Also I'm not sure how do you get a big diff if everything is sorted? Someone else checked it in unsorted. Either because he edited it manually or because it is from a time before the sorting was added.