Attachment 90356 Details for Bug 73979 – patch 2

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] patch 2

patch-73979-b.txt (text/plain), 1.35 KB, created by Martin Aeschlimann

on 2008-02-21 10:06:38 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Aeschlimann

Created: 2008-02-21 10:06:38 EST

Size: 1.35 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.ui.workbench
>Index: Eclipse UI/org/eclipse/ui/internal/decorators/DecoratorManager.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/decorators/DecoratorManager.java,v
>retrieving revision 1.59
>diff -u -r1.59 DecoratorManager.java
>--- Eclipse UI/org/eclipse/ui/internal/decorators/DecoratorManager.java	8 May 2006 20:55:51 -0000	1.59
>+++ Eclipse UI/org/eclipse/ui/internal/decorators/DecoratorManager.java	21 Feb 2008 14:52:14 -0000
>@@ -875,11 +875,10 @@
>         scheduler.queueForDecoration(element, getResourceAdapter(element),
>                 force, originalText, context); 
> 
>-        //If all that is there is deferred ones then defer decoration.
>-        //For the sake of efficiency we do not test for enablement at this
>-        //point and just abandon deferment if there are any to run right
>-        //away
>-        return getFullDefinitions().length > 0;
>+        // if we forced the decorator to update, then the caller can wait for the notification.
>+        // when notified he will update and get the light and full label decorations together
>+        // if the decoration was not forced, he needs to decorate the label right away.
>+        return !force;
>     }
> 
>     /* (non-Javadoc)

Actions: View | Diff

Attachments on bug 73979: 90320 | 90356