mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-23 14:42:11 +02:00
Code Issues Wiki Activity

Clarify null handling in javadoc of ProcessClosure constructor

Browse source 
The javadoc of ProcessClosure specifies that outputStream/errorStream
"Can be null, if not interested in reading the output" but that's not
correct: passing null triggers NPEs. This patch updates the javadoc to
specify that null is not handled.

Change-Id: Iea65486cc938d1f3bf7f0beb479b2329c9f0ecdc
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
This commit is contained in:
Marc-Andre Laperle 2015-11-05 22:30:21 -05:00 • committed by Gerrit Code Review @ Eclipse.org
parent 79f4bf7d1a
commit b1104da01e
1 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
core/org.eclipse.cdt.core/src/org/eclipse/cdt/internal/core/ProcessClosure.java
View file

@ -117,11 +117,9 @@ public class ProcessClosure {
* console document. * console document.
* *
* @param outputStream * @param outputStream
* prcess stdout is written to this stream. Can be * prcess stdout is written to this stream. Cannot be <code>null</code>.
* <code>null</code>, if not interested in reading the output
* @param errorStream * @param errorStream
* prcess stderr is written to this stream. Can be * prcess stderr is written to this stream. Cannot be <code>null</code>.
* <code>null</code>, if not interested in reading the output
*/ */
public ProcessClosure(Process process, OutputStream outputStream, OutputStream errorStream) { public ProcessClosure(Process process, OutputStream outputStream, OutputStream errorStream) {
fProcess = process; fProcess = process;