Christoph Läubrich
Christoph Läubrich
@mehmet-karaman can you please more explain the rationale behind the changes? I think we can assume code is there for a reason so if we change fundamental things we should...
It still would be good to more explain the individual changes and why they are required / needed. Also I think some changes are better a separate PR (as they...
Each of those can be an own PR - Removed dead code in AnnotationModel.cleanup because mapLock can't be null. - Added new javadoc to IAnnotationMap.getLockObject() - added assert.isLegal check to...
> I believe these could be seperated from the main PR, but should go in one PR, because they all require each other. Sure at laest I think these are...
It feels from the description that one needs a complete rework of the data model instead... in any case I think the idea with the lock object is/was that one...
Some test are failing here because Equinox is not parsing query strings correctly, I tried to address this here: - https://github.com/eclipse-equinox/equinox/pull/1217
Hm... so maybe like this?
So it seems we maybe need some kind of asyncExec here...
I assume this is simply two different functions here, so first question would be: Does it harm? The hide Icon feature was meant to have more space to show the...
> Do you plan to provide a fix for this, [@BeckerWdf](https://github.com/BeckerWdf) / [@praveen-skp](https://github.com/praveen-skp) ? > > I'm **not** in a hurry, I just want to know if it will be...