Skip to content

Commit

Permalink
Delete "record_function_information" flag and supporting infrastructure.
Browse files Browse the repository at this point in the history
This feature is no longer supported.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=229114925
  • Loading branch information
nreid260 authored and EatingW committed Jan 14, 2019
1 parent 483bd58 commit 3a13b14
Show file tree
Hide file tree
Showing 9 changed files with 0 additions and 825 deletions.
6 changes: 0 additions & 6 deletions src/com/google/javascript/jscomp/AbstractCompiler.java
Original file line number Diff line number Diff line change
Expand Up @@ -115,12 +115,6 @@ public abstract class AbstractCompiler implements SourceExcerptProvider {
/** Sets the string replacement map */
public abstract void setStringMap(VariableMap stringMap);

/** Sets the fully qualified function name and globally unique id mapping. */
public abstract void setFunctionNames(FunctionNames functionNames);

/** Gets the fully qualified function name and globally unique id mapping. */
public abstract FunctionNames getFunctionNames();

/** Sets the css names found during compilation. */
public abstract void setCssNames(Map<String, Integer> newCssNames);

Expand Down
240 changes: 0 additions & 240 deletions src/com/google/javascript/jscomp/CollectFunctionNames.java

This file was deleted.

30 changes: 0 additions & 30 deletions src/com/google/javascript/jscomp/Compiler.java
Original file line number Diff line number Diff line change
Expand Up @@ -852,10 +852,6 @@ public void performPostCompilationTasks() {
* Performs all the bookkeeping required at the end of a compilation.
*/
private void performPostCompilationTasksInternal() {
if (options.recordFunctionInformation) {
recordFunctionInformation();
}

if (options.devMode == DevMode.START_AND_END) {
runValidityCheck();
}
Expand Down Expand Up @@ -2413,16 +2409,6 @@ void prepareAst(Node root) {
pass.process(null, root);
}

void recordFunctionInformation() {
logger.fine("Recording function information");
startPass("recordFunctionInformation");
RecordFunctionInformation recordFunctionInfoPass =
new RecordFunctionInformation(this, this.functionNames);
process(recordFunctionInfoPass);
functionInformationMap = recordFunctionInfoPass.getMap();
endPass("recordFunctionInformation");
}

protected final RecentChange recentChange = new RecentChange();
private final List<CodeChangeHandler> codeChangeHandlers = new ArrayList<>();
private final Map<Class<?>, IndexProvider<?>> indexProvidersByType =
Expand Down Expand Up @@ -2930,9 +2916,6 @@ public SourceMap getSourceMap() {
/** The naming map for anonymous functions */
private VariableMap anonymousFunctionNameMap = null;

/** Fully qualified function names and globally unique ids */
private FunctionNames functionNames = null;

/** String replacement map */
private VariableMap stringMap = null;

Expand Down Expand Up @@ -2965,11 +2948,6 @@ public void setStringMap(VariableMap stringMap) {
this.stringMap = stringMap;
}

@Override
public void setFunctionNames(FunctionNames functionNames) {
this.functionNames = functionNames;
}

@Override
public void setCssNames(Map<String, Integer> cssNames) {
this.cssNames = cssNames;
Expand All @@ -2994,11 +2972,6 @@ public void setAnonymousFunctionNameMap(VariableMap functionMap) {
this.anonymousFunctionNameMap = functionMap;
}

@Override
public FunctionNames getFunctionNames() {
return functionNames;
}

VariableMap getStringMap() {
return this.stringMap;
}
Expand Down Expand Up @@ -3463,7 +3436,6 @@ private static class CompilerState implements Serializable {
private final VariableMap variableMap;
private final VariableMap propertyMap;
private final VariableMap anonymousFunctionaMap;
private final FunctionNames functionNames;
private final VariableMap stringMap;
private final String idGeneratorMap;
private final IdGenerator crossModuleIdGenerator;
Expand Down Expand Up @@ -3498,7 +3470,6 @@ private static class CompilerState implements Serializable {
this.variableMap = compiler.variableMap;
this.propertyMap = compiler.propertyMap;
this.anonymousFunctionaMap = compiler.anonymousFunctionNameMap;
this.functionNames = compiler.functionNames;
this.stringMap = compiler.stringMap;
this.idGeneratorMap = compiler.idGeneratorMap;
this.crossModuleIdGenerator = compiler.crossModuleIdGenerator;
Expand Down Expand Up @@ -3598,7 +3569,6 @@ public CompilerState call() throws Exception {
anonymousFunctionNameMap = compilerState.anonymousFunctionaMap;
idGeneratorMap = compilerState.idGeneratorMap;
crossModuleIdGenerator = compilerState.crossModuleIdGenerator;
functionNames = compilerState.functionNames;
defaultDefineValues = checkNotNull(compilerState.defaultDefineValues);
annotationMap = checkNotNull(compilerState.annotationMap);
inputSourceMaps = compilerState.inputSourceMaps;
Expand Down
10 changes: 0 additions & 10 deletions src/com/google/javascript/jscomp/CompilerOptions.java
Original file line number Diff line number Diff line change
Expand Up @@ -829,10 +829,6 @@ public void setReplaceMessagesWithChromeI18n(
/** Move top-level function declarations to the top */
public boolean moveFunctionDeclarations;


/** Record function information */
public boolean recordFunctionInformation;

boolean checksOnly;

/** What type of JS file should be output by this compilation */
Expand Down Expand Up @@ -1331,7 +1327,6 @@ public CompilerOptions() {
tweakProcessing = TweakProcessing.OFF;
tweakReplacements = new HashMap<>();
moveFunctionDeclarations = false;
recordFunctionInformation = false;
checksOnly = false;
outputJs = OutputJs.NORMAL;
generateExports = false;
Expand Down Expand Up @@ -2534,10 +2529,6 @@ public void setMoveFunctionDeclarations(boolean moveFunctionDeclarations) {
this.moveFunctionDeclarations = moveFunctionDeclarations;
}

public void setRecordFunctionInformation(boolean recordFunctionInformation) {
this.recordFunctionInformation = recordFunctionInformation;
}

public void setCssRenamingMap(CssRenamingMap cssRenamingMap) {
this.cssRenamingMap = cssRenamingMap;
}
Expand Down Expand Up @@ -3023,7 +3014,6 @@ public String toString() {
.add("propertyRenaming", propertyRenaming)
.add("protectHiddenSideEffects", protectHiddenSideEffects)
.add("quoteKeywordProperties", quoteKeywordProperties)
.add("recordFunctionInformation", recordFunctionInformation)
.add("removeAbstractMethods", removeAbstractMethods)
.add("removeClosureAsserts", removeClosureAsserts)
.add("removeJ2clAsserts", removeJ2clAsserts)
Expand Down

0 comments on commit 3a13b14

Please sign in to comment.