removed implemented TODO comments

samualtnorman · samualtnorman · commit 26dab582ddf5 · 2022-02-23T17:33:41.000Z
diff --git a/src/processScript/minify.ts b/src/processScript/minify.ts
@@ -18,7 +18,6 @@ type MinifyOptions = {
 }
 
 // TODO move autocomplete code outside this function
-// TODO allow not mangling class and function names
 // TODO replace references to `arguments`
 
 /**
@@ -172,7 +171,6 @@ export async function minify(file: File, autocomplete?: string, {
 		}
 	})
 
-	// BUG the source char count is off for this version of the script
 	const scriptBeforeJSONValueReplacement = (await terser.minify(generate(fileBeforeJSONValueReplacement!).code, {
 		ecma: 2015,
 		compress: {
diff --git a/src/processScript/transform.ts b/src/processScript/transform.ts
@@ -208,7 +208,6 @@ export function transform(file: File, sourceCode: string, {
 		}
 	}
 
-	// TODO turn not calling into a arrow function wrapper
 	if (program.scope.hasGlobal(`$D`)) {
 		for (const referencePath of getReferencePathsToGlobal(`$D`, program)) {
 			if (referencePath.parentPath.type == `CallExpression`)

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,6 @@ type MinifyOptions = {`
`18`	`18`	`}`
`19`	`19`
`20`	`20`	`// TODO move autocomplete code outside this function`
`21`		`-// TODO allow not mangling class and function names`
`22`	`21`	// TODO replace references to `arguments`
`23`	`22`
`24`	`23`	`/**`
`@@ -172,7 +171,6 @@ export async function minify(file: File, autocomplete?: string, {`
`172`	`171`	`}`
`173`	`172`	`})`
`174`	`173`
`175`		`- // BUG the source char count is off for this version of the script`
`176`	`174`	`const scriptBeforeJSONValueReplacement = (await terser.minify(generate(fileBeforeJSONValueReplacement!).code, {`
`177`	`175`	`ecma: 2015,`
`178`	`176`	`compress: {`
Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,6 @@ export function transform(file: File, sourceCode: string, {`
`208`	`208`	`}`
`209`	`209`	`}`
`210`	`210`
`211`		`- // TODO turn not calling into a arrow function wrapper`
`212`	`211`	if (program.scope.hasGlobal(`$D`)) {
`213`	`212`	for (const referencePath of getReferencePathsToGlobal(`$D`, program)) {
`214`	`213`	if (referencePath.parentPath.type == `CallExpression`)