[incubator-hop] branch master updated: HOP-13 Display.getDefault() cannot be used in non-UI thread with RAP/RWT (#365)

[hansva]

This is an automated email from the ASF dual-hosted git repository.

hansva pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-hop.git


The following commit(s) were added to refs/heads/master by this push:
     new 119420c  HOP-13 Display.getDefault() cannot be used in non-UI thread 
with RAP/RWT (#365)
119420c is described below

commit 119420c424a6b98fabc4793e094340238fce9fc1
Author: Hiromu Hota <hiromu.h...@gmail.com>
AuthorDate: Tue Nov 10 01:26:18 2020 -0800

    HOP-13 Display.getDefault() cannot be used in non-UI thread with RAP/RWT 
(#365)
    
    See https://www.eclipse.org/forums/index.php/t/805303/
---
 .../transforms/fieldschangesequence/FieldsChangeSequenceDialog.java     | 2 +-
 .../pipeline/transforms/stringoperations/StringOperationsDialog.java    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/plugins/transforms/fieldschangesequence/src/main/java/org/apache/hop/pipeline/transforms/fieldschangesequence/FieldsChangeSequenceDialog.java
 
b/plugins/transforms/fieldschangesequence/src/main/java/org/apache/hop/pipeline/transforms/fieldschangesequence/FieldsChangeSequenceDialog.java
index d48d882..55cb35b 100644
--- 
a/plugins/transforms/fieldschangesequence/src/main/java/org/apache/hop/pipeline/transforms/fieldschangesequence/FieldsChangeSequenceDialog.java
+++ 
b/plugins/transforms/fieldschangesequence/src/main/java/org/apache/hop/pipeline/transforms/fieldschangesequence/FieldsChangeSequenceDialog.java
@@ -252,7 +252,7 @@ public class FieldsChangeSequenceDialog extends 
BaseTransformDialog implements I
           }
 
           // Dislay in red missing field names
-          Display.getDefault().asyncExec( () -> {
+          display.asyncExec( () -> {
             if ( !wFields.isDisposed() ) {
               for ( int i = 0; i < wFields.table.getItemCount(); i++ ) {
                 TableItem it = wFields.table.getItem( i );
diff --git 
a/plugins/transforms/stringoperations/src/main/java/org/apache/hop/pipeline/transforms/stringoperations/StringOperationsDialog.java
 
b/plugins/transforms/stringoperations/src/main/java/org/apache/hop/pipeline/transforms/stringoperations/StringOperationsDialog.java
index d7a7ff5..d987159 100644
--- 
a/plugins/transforms/stringoperations/src/main/java/org/apache/hop/pipeline/transforms/stringoperations/StringOperationsDialog.java
+++ 
b/plugins/transforms/stringoperations/src/main/java/org/apache/hop/pipeline/transforms/stringoperations/StringOperationsDialog.java
@@ -238,7 +238,7 @@ public class StringOperationsDialog extends 
BaseTransformDialog implements ITran
           }
 
           // Dislay in red missing field names
-          Display.getDefault().asyncExec( () -> {
+          display.asyncExec( () -> {
             if ( !wFields.isDisposed() ) {
               for ( int i = 0; i < wFields.table.getItemCount(); i++ ) {
                 TableItem it = wFields.table.getItem( i );