Can you please post a full stack trace ?

I also found another defect in that widget

borderColor is a private class field


    @Override
    public void paintControl(PaintEvent e) {
    ...
        /*
        * Draws the border
        */
        if ((style & SWT.BORDER) != 0) {
            if (borderColor == null || borderColor.isDisposed())
                borderColor = getDisplay().getSystemColor(SWT.COLOR_BLACK);
    ...
    }

    @Override
    public void setBackground(Color color) {
        this.backgroundColor = color;
        if (this.borderColor != null && !this.borderColor.isDisposed())
            this.borderColor.dispose();
        this.borderColor = ColorUtils.scaleColorBy(color,
            BORDER_LIGHTNESS_SCALE);
        this.redraw();
    }


Disposing a system color is strictly forbidden.


---

** [bugs:#830] Switching Eclipse themes triggers exceptions when using Saros**

**Status:** open
**Group:** OTHER
**Labels:** Eclipse 4.4 
**Created:** Fri Jun 27, 2014 11:11 AM UTC by Franz Zieris
**Last Updated:** Fri Jun 27, 2014 11:11 AM UTC
**Owner:** nobody

Since Eclipse 4.4 (in particular since [this 
commit](http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/bundles/org.eclipse.e4.ui.css.swt/src/org/eclipse/e4/ui/css/swt/dom/ControlElement.java?h=R4_4_maintenance&id=946c6d54de1d199bb882124673f4ede122a00f89),
 which fixes [this bug](https://bugs.eclipse.org/bugs/show_bug.cgi?id=423813)), 
the SWT method Control.setBackground() is sometimes called with a null value, 
in particular when switching the theme of Eclipse (Preferences > General > 
Appearance), e.g. to the new "Dark" theme.
This null value is not expected in the RoundedComposite class and produces a 
NullPointerException.

But the fix is not as easy as just ignoring null values. For some reason, even 
if the "setBackground()" method does nothing on null values, there will be a 
ConcurrentModificationException in the Eclipse code.

Steps to reproduce:

* Use Eclipse 4.4
* Use Saros 13.12.6, or the current master (i.e. 
[fa1f320](https://github.com/saros-project/saros/commit/fa1f320))
* Start a Saros session, so the session chat appears (this should also work 
with a single user chat -- we only need instances of RoundedComposite).
* Change the theme of Eclipse (Prefences > General > Appearance), e.g. to "Dark"
* An error message will appear, and the theme change will not be performed 
completely.


---

Sent from sourceforge.net because [email protected] is subscribed 
to https://sourceforge.net/p/dpp/bugs/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/dpp/admin/bugs/options.  Or, if this is a mailing 
list, you can unsubscribe from the mailing list.
------------------------------------------------------------------------------
Open source business process management suite built on Java and Eclipse
Turn processes into business applications with Bonita BPM Community Edition
Quickly connect people, data, and systems into organized workflows
Winner of BOSSIE, CODIE, OW2 and Gartner awards
http://p.sf.net/sfu/Bonitasoft
_______________________________________________
Dpp-robot mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dpp-robot

Reply via email to