tmf: Dispose clipboard after Copy to Clipboard operation

Change-Id: Icc6ee4b8de1a936ef86c61f0e8b5b9e6afb0de71
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/50633
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

diff --git a/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/commands/CopyToClipboardOperation.java b/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/commands/CopyToClipboardOperation.java
index 2aca2e7efa319b648a7a1d785846c635f8a58a32..41da2e83e268e0eb3feed1b506ad202ec5f1a57f 100644 (file)
--- a/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/commands/CopyToClipboardOperation.java
+++ b/tmf/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/commands/CopyToClipboardOperation.java
@@ -94,14 +94,16 @@ public class CopyToClipboardOperation implements IRunnableWithProgress {
                 if (sb.length() == 0) {
                     return;
                 }
+                Clipboard clipboard = new Clipboard(Display.getDefault());
                 try {
-                    Clipboard clipboard = new Clipboard(Display.getDefault());
                     clipboard.setContents(new Object[] { sb.toString() },
                             new Transfer[] { TextTransfer.getInstance() });
                 } catch (OutOfMemoryError e) {
                     sb.setLength(0);
                     sb.trimToSize();
                     showErrorDialog();
+                } finally {
+                    clipboard.dispose();
                 }
             }
         });