diff options
author | Sarika Sinha | 2018-10-11 08:59:53 +0000 |
---|---|---|
committer | Sarika Sinha | 2018-10-11 09:16:54 +0000 |
commit | fb5b817008012540fb111acd48a7d128c65bd6b9 (patch) | |
tree | 0307f13452d4d9b04b0f4f4d15ea55c79df58b0f | |
parent | 8619dcb5b08b7ff36e0daeca31f16de73c006c05 (diff) | |
download | eclipse.platform.debug-fb5b817008012540fb111acd48a7d128c65bd6b9.tar.gz eclipse.platform.debug-fb5b817008012540fb111acd48a7d128c65bd6b9.tar.xz eclipse.platform.debug-fb5b817008012540fb111acd48a7d128c65bd6b9.zip |
Bug 307309 - [console] Some uses of IOConsoleInputStream.available()I20181017-1800
throw IOException for no good reason
Change-Id: Iaa7ad7e96bb602532ed7af4c02e35911d26a7e3f
6 files changed, 19 insertions, 13 deletions
diff --git a/org.eclipse.debug.tests/META-INF/MANIFEST.MF b/org.eclipse.debug.tests/META-INF/MANIFEST.MF index a62332e0d..0e97d9e66 100644 --- a/org.eclipse.debug.tests/META-INF/MANIFEST.MF +++ b/org.eclipse.debug.tests/META-INF/MANIFEST.MF @@ -2,7 +2,7 @@ Manifest-Version: 1.0 Bundle-ManifestVersion: 2 Bundle-Name: %pluginName Bundle-SymbolicName: org.eclipse.debug.tests;singleton:=true -Bundle-Version: 3.11.100.qualifier +Bundle-Version: 3.11.200.qualifier Bundle-Activator: org.eclipse.debug.tests.TestsPlugin Bundle-Localization: plugin Require-Bundle: org.eclipse.ui;bundle-version="[3.6.0,4.0.0)", diff --git a/org.eclipse.debug.tests/pom.xml b/org.eclipse.debug.tests/pom.xml index 8d54f45f4..1f098481e 100644 --- a/org.eclipse.debug.tests/pom.xml +++ b/org.eclipse.debug.tests/pom.xml @@ -1,6 +1,6 @@ <?xml version="1.0" encoding="UTF-8"?> <!-- - Copyright (c) 2012, 2017 Eclipse Foundation and others. + Copyright (c) 2012, 2018 Eclipse Foundation and others. All rights reserved. This program and the accompanying materials are made available under the terms of the Eclipse Distribution License v1.0 which accompanies this distribution, and is available at @@ -18,7 +18,7 @@ </parent> <groupId>org.eclipse.debug</groupId> <artifactId>org.eclipse.debug.tests</artifactId> - <version>3.11.100-SNAPSHOT</version> + <version>3.11.200-SNAPSHOT</version> <packaging>eclipse-test-plugin</packaging> <properties> <code.ignoredWarnings>${tests.ignoredWarnings}</code.ignoredWarnings> diff --git a/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ConsoleTests.java b/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ConsoleTests.java index 7ecda8cba..cd9d51eae 100644 --- a/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ConsoleTests.java +++ b/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ConsoleTests.java @@ -15,6 +15,7 @@ package org.eclipse.debug.tests.console; import java.io.IOException; +import java.io.InputStream; import java.nio.charset.StandardCharsets; import org.eclipse.core.commands.Command; @@ -187,4 +188,16 @@ public class ConsoleTests extends AbstractDebugTest { activePage.hideView(consoleView); } } + + public void testIOConsoleAvailable() throws Exception { + + final IOConsole console = new IOConsole("", null); + + try (InputStream consoleInput = console.getInputStream()) { + consoleInput.available(); + consoleInput.available(); + } catch (IOException ioe) { + TestCase.assertEquals("Input Stream is closed", ioe.getMessage()); //$NON-NLS-1$ + } + } } diff --git a/org.eclipse.ui.console/META-INF/MANIFEST.MF b/org.eclipse.ui.console/META-INF/MANIFEST.MF index 42a938ad2..7e0087375 100644 --- a/org.eclipse.ui.console/META-INF/MANIFEST.MF +++ b/org.eclipse.ui.console/META-INF/MANIFEST.MF @@ -2,7 +2,7 @@ Manifest-Version: 1.0 Bundle-ManifestVersion: 2 Bundle-Name: %pluginName Bundle-SymbolicName: org.eclipse.ui.console; singleton:=true -Bundle-Version: 3.8.200.qualifier +Bundle-Version: 3.8.300.qualifier Bundle-Activator: org.eclipse.ui.console.ConsolePlugin Bundle-Vendor: %providerName Bundle-Localization: plugin diff --git a/org.eclipse.ui.console/pom.xml b/org.eclipse.ui.console/pom.xml index 588dcfd94..b77083974 100644 --- a/org.eclipse.ui.console/pom.xml +++ b/org.eclipse.ui.console/pom.xml @@ -18,6 +18,6 @@ </parent> <groupId>org.eclipse.ui</groupId> <artifactId>org.eclipse.ui.console</artifactId> - <version>3.8.200-SNAPSHOT</version> + <version>3.8.300-SNAPSHOT</version> <packaging>eclipse-plugin</packaging> </project> diff --git a/org.eclipse.ui.console/src/org/eclipse/ui/console/IOConsoleInputStream.java b/org.eclipse.ui.console/src/org/eclipse/ui/console/IOConsoleInputStream.java index 44e0d7046..0791aa560 100644 --- a/org.eclipse.ui.console/src/org/eclipse/ui/console/IOConsoleInputStream.java +++ b/org.eclipse.ui.console/src/org/eclipse/ui/console/IOConsoleInputStream.java @@ -1,5 +1,5 @@ /******************************************************************************* - * Copyright (c) 2000, 2017 IBM Corporation and others. + * Copyright (c) 2000, 2018 IBM Corporation and others. * * This program and the accompanying materials * are made available under the terms of the Eclipse Public License 2.0 @@ -269,14 +269,7 @@ public class IOConsoleInputStream extends InputStream { public int available() throws IOException { if (closed && eofSent) { throw new IOException("Input Stream Closed"); //$NON-NLS-1$ - } else if (size == 0) { - if (!eofSent) { - eofSent = true; - return -1; - } - throw new IOException("Input Stream Closed"); //$NON-NLS-1$ } - return size; } |