Skip to content

Commit

Permalink
Transition typedAstOutputFile from a stream to a Path.
Browse files Browse the repository at this point in the history
Streams are not good fields in options.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=258491821
  • Loading branch information
nreid260 authored and lauraharker committed Jul 17, 2019
1 parent 3cc33fe commit be60e0c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions src/com/google/javascript/jscomp/CompilerOptions.java
Expand Up @@ -144,15 +144,15 @@ public boolean getInstrumentForCoverageOnly() {
return instrumentForCoverageOnly;
}

@Nullable private OutputStream typedAstOutputFile = null;
@Nullable private Path typedAstOutputFile = null;

/** Sets file to output in-progress TypedAST format to. DO NOT USE! */
void setTypedAstOutputFile(@Nullable OutputStream file) {
void setTypedAstOutputFile(@Nullable Path file) {
this.typedAstOutputFile = file;
}

@Nullable
OutputStream getTypedAstOutputFile() {
Path getTypedAstOutputFile() {
return this.typedAstOutputFile;
}

Expand Down
Expand Up @@ -19,12 +19,12 @@
import com.google.javascript.jscomp.AbstractCompiler;
import com.google.javascript.jscomp.CompilerPass;
import com.google.javascript.rhino.Node;
import java.io.OutputStream;
import java.nio.file.Path;

/** Fail-fast replacement */
public final class SerializeTypedAstPass implements CompilerPass {

public SerializeTypedAstPass(AbstractCompiler compiler, OutputStream out) {
public SerializeTypedAstPass(AbstractCompiler compiler, Path out) {
throw new RuntimeException("Serialization not yet supported in JS version of compiler");
}

Expand Down

0 comments on commit be60e0c

Please sign in to comment.