Skip to content

Commit fc61262

Browse files
Rich-HarrisRich-Harris
authored
chore: delete some unused code (#15783)
* tidy up * more * more
1 parent 189bd47 commit fc61262

32 files changed

+27
-53
lines changed

‎packages/svelte/src/compiler/migrate/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/** @import { VariableDeclarator, Node, Identifier, AssignmentExpression, LabeledStatement, ExpressionStatement } from 'estree' */
22
/** @import { Visitors } from 'zimmerframe' */
33
/** @import { ComponentAnalysis } from '../phases/types.js' */
4-
/** @import { Scope, ScopeRoot } from '../phases/scope.js' */
4+
/** @import { Scope } from '../phases/scope.js' */
55
/** @import { AST, Binding, ValidatedCompileOptions } from '#compiler' */
66
import MagicString from 'magic-string';
77
import { walk } from 'zimmerframe';

‎packages/svelte/src/compiler/phases/2-analyze/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import { walk } from 'zimmerframe';
66
import * as e from '../../errors.js';
77
import * as w from '../../warnings.js';
8-
import { extract_identifiers, is_text_attribute } from '../../utils/ast.js';
8+
import { extract_identifiers } from '../../utils/ast.js';
99
import * as b from '../../utils/builders.js';
1010
import { Scope, ScopeRoot, create_scopes, get_rune, set_scope } from '../scope.js';
1111
import check_graph_for_cycles from './utils/check_graph_for_cycles.js';

‎packages/svelte/src/compiler/phases/2-analyze/visitors/CallExpression.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
/** @import { Context } from '../types' */
44
import { get_rune } from '../../scope.js';
55
import * as e from '../../../errors.js';
6-
import { get_parent, unwrap_optional } from '../../../utils/ast.js';
6+
import { get_parent } from '../../../utils/ast.js';
77
import { is_pure, is_safe_identifier } from './shared/utils.js';
88
import { dev, locate_node, source } from '../../../state.js';
99
import * as b from '../../../utils/builders.js';

‎packages/svelte/src/compiler/phases/2-analyze/visitors/ExportNamedDeclaration.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
/** @import { ExportNamedDeclaration, Identifier, Node } from 'estree' */
2-
/** @import { Binding } from '#compiler' */
1+
/** @import { ExportNamedDeclaration, Identifier } from 'estree' */
32
/** @import { Context } from '../types' */
4-
/** @import { Scope } from '../../scope' */
53
import * as e from '../../../errors.js';
64
import { extract_identifiers } from '../../../utils/ast.js';
75

‎packages/svelte/src/compiler/phases/2-analyze/visitors/Identifier.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
/** @import { Expression, Identifier } from 'estree' */
2-
/** @import { EachBlock } from '#compiler' */
32
/** @import { Context } from '../types' */
43
import is_reference from 'is-reference';
54
import { should_proxy } from '../../3-transform/client/utils.js';

‎packages/svelte/src/compiler/phases/2-analyze/visitors/MemberExpression.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
1-
/** @import { MemberExpression, Node } from 'estree' */
1+
/** @import { MemberExpression } from 'estree' */
22
/** @import { Context } from '../types' */
33
import * as e from '../../../errors.js';
4-
import * as w from '../../../warnings.js';
5-
import { object } from '../../../utils/ast.js';
64
import { is_pure, is_safe_identifier } from './shared/utils.js';
7-
import { mark_subtree_dynamic } from './shared/fragment.js';
85

96
/**
107
* @param {MemberExpression} node

‎packages/svelte/src/compiler/phases/2-analyze/visitors/shared/utils.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/** @import { AssignmentExpression, Expression, Identifier, Literal, Node, Pattern, PrivateIdentifier, Super, UpdateExpression, VariableDeclarator } from 'estree' */
1+
/** @import { AssignmentExpression, Expression, Literal, Node, Pattern, Super, UpdateExpression, VariableDeclarator } from 'estree' */
22
/** @import { AST, Binding } from '#compiler' */
33
/** @import { AnalysisState, Context } from '../../types' */
44
/** @import { Scope } from '../../../scope' */

‎packages/svelte/src/compiler/phases/3-transform/client/visitors/BlockStatement.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/** @import { ArrowFunctionExpression, BlockStatement, CallExpression, Expression, FunctionDeclaration, FunctionExpression, Statement } from 'estree' */
1+
/** @import { ArrowFunctionExpression, BlockStatement, Expression, FunctionDeclaration, FunctionExpression, Statement } from 'estree' */
22
/** @import { ComponentContext } from '../types' */
33
import { add_state_transformers } from './shared/declarations.js';
44
import * as b from '../../../../utils/builders.js';

‎packages/svelte/src/compiler/phases/3-transform/client/visitors/ClassBody.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
/** @import { ClassBody, Expression, Identifier, Literal, MethodDefinition, PrivateIdentifier, PropertyDefinition } from 'estree' */
2-
/** @import { } from '#compiler' */
32
/** @import { Context, StateField } from '../types' */
4-
import { dev, is_ignored } from '../../../../state.js';
53
import * as b from '../../../../utils/builders.js';
64
import { regex_invalid_identifier_chars } from '../../../patterns.js';
75
import { get_rune } from '../../../scope.js';

‎packages/svelte/src/compiler/phases/3-transform/client/visitors/IfBlock.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/** @import { BlockStatement, Expression, Identifier } from 'estree' */
1+
/** @import { BlockStatement, Expression } from 'estree' */
22
/** @import { AST } from '#compiler' */
33
/** @import { ComponentContext } from '../types' */
44
import * as b from '../../../../utils/builders.js';

‎packages/svelte/src/compiler/phases/3-transform/client/visitors/LabeledStatement.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
1-
/** @import { Location } from 'locate-character' */
21
/** @import { Expression, LabeledStatement, Statement } from 'estree' */
32
/** @import { ReactiveStatement } from '#compiler' */
43
/** @import { ComponentContext } from '../types' */
5-
import { dev, is_ignored, locator } from '../../../../state.js';
64
import * as b from '../../../../utils/builders.js';
75
import { build_getter } from '../utils.js';
86

‎packages/svelte/src/compiler/phases/3-transform/client/visitors/RegularElement.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/** @import { ArrayExpression, Expression, ExpressionStatement, Identifier, MemberExpression, ObjectExpression, Statement } from 'estree' */
1+
/** @import { ArrayExpression, Expression, ExpressionStatement, Identifier, MemberExpression, ObjectExpression } from 'estree' */
22
/** @import { AST } from '#compiler' */
33
/** @import { SourceLocation } from '#shared' */
44
/** @import { ComponentClientTransformState, ComponentContext } from '../types' */

‎packages/svelte/src/compiler/phases/3-transform/client/visitors/SvelteElement.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { is_text_attribute } from '../../../../utils/ast.js';
66
import * as b from '../../../../utils/builders.js';
77
import { determine_namespace_for_children } from '../../utils.js';
88
import { build_attribute_value, build_set_attributes, build_set_class } from './shared/element.js';
9-
import { build_render_statement, get_expression_id } from './shared/utils.js';
9+
import { build_render_statement } from './shared/utils.js';
1010

1111
/**
1212
* @param {AST.SvelteElement} node

‎packages/svelte/src/compiler/phases/3-transform/client/visitors/shared/element.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/** @import { ArrayExpression, Expression, Identifier, ObjectExpression } from 'estree' */
22
/** @import { AST, ExpressionMetadata } from '#compiler' */
3-
/** @import { ComponentClientTransformState, ComponentContext } from '../../types' */
3+
/** @import { ComponentContext } from '../../types' */
44
import { escape_html } from '../../../../../../escaping.js';
55
import { normalize_attribute } from '../../../../../../utils.js';
66
import { is_ignored } from '../../../../../state.js';

‎packages/svelte/src/compiler/phases/3-transform/server/visitors/EachBlock.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/** @import { BlockStatement, Expression, Pattern, Statement } from 'estree' */
1+
/** @import { BlockStatement, Expression, Statement } from 'estree' */
22
/** @import { AST } from '#compiler' */
33
/** @import { ComponentContext } from '../types.js' */
44
import { BLOCK_OPEN_ELSE } from '../../../../../internal/server/hydration.js';

‎packages/svelte/src/compiler/phases/3-transform/server/visitors/IfBlock.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/** @import { BlockStatement, Expression, IfStatement } from 'estree' */
1+
/** @import { BlockStatement, Expression } from 'estree' */
22
/** @import { AST } from '#compiler' */
33
/** @import { ComponentContext } from '../types.js' */
44
import { BLOCK_OPEN_ELSE } from '../../../../../internal/server/hydration.js';

‎packages/svelte/src/compiler/phases/3-transform/server/visitors/shared/element.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,7 @@
11
/** @import { ArrayExpression, Expression, Literal, ObjectExpression } from 'estree' */
2-
/** @import { AST, Namespace } from '#compiler' */
2+
/** @import { AST } from '#compiler' */
33
/** @import { ComponentContext, ComponentServerTransformState } from '../../types.js' */
4-
import {
5-
get_attribute_chunks,
6-
is_event_attribute,
7-
is_text_attribute
8-
} from '../../../../../utils/ast.js';
4+
import { is_event_attribute, is_text_attribute } from '../../../../../utils/ast.js';
95
import { binding_properties } from '../../../../bindings.js';
106
import {
117
create_attribute,

‎packages/svelte/src/compiler/phases/3-transform/server/visitors/shared/utils.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/** @import { AssignmentOperator, Expression, Identifier, Node, Statement, TemplateElement } from 'estree' */
1+
/** @import { AssignmentOperator, Expression, Identifier, Node, Statement } from 'estree' */
22
/** @import { AST } from '#compiler' */
33
/** @import { ComponentContext, ServerTransformState } from '../../types.js' */
44

‎packages/svelte/src/compiler/phases/scope.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,10 @@ import { is_reserved, is_rune } from '../../utils.js';
1616
import { determine_slot } from '../utils/slot.js';
1717
import { validate_identifier_name } from './2-analyze/visitors/shared/utils.js';
1818

19-
export const UNKNOWN = Symbol('unknown');
19+
const UNKNOWN = Symbol('unknown');
2020
/** Includes `BigInt` */
21-
export const NUMBER = Symbol('number');
22-
export const STRING = Symbol('string');
21+
const NUMBER = Symbol('number');
22+
const STRING = Symbol('string');
2323

2424
export class Binding {
2525
/** @type {Scope} */

‎packages/svelte/src/constants.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ export const HYDRATION_START = '[';
2222
/** used to indicate that an `{:else}...` block was rendered */
2323
export const HYDRATION_START_ELSE = '[!';
2424
export const HYDRATION_END = ']';
25-
export const HYDRATION_AWAIT_THEN = '!';
2625
export const HYDRATION_ERROR = {};
2726

2827
export const ELEMENT_IS_NAMESPACED = 1;

‎packages/svelte/src/internal/client/context.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,7 @@ import {
77
active_effect,
88
active_reaction,
99
set_active_effect,
10-
set_active_reaction,
11-
untrack
10+
set_active_reaction
1211
} from './runtime.js';
1312
import { effect, teardown } from './reactivity/effects.js';
1413
import { legacy_mode_flag } from '../flags/index.js';

‎packages/svelte/src/internal/client/dev/tracing.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/** @import { Derived, Reaction, Signal, Value } from '#client' */
1+
/** @import { Derived, Reaction, Value } from '#client' */
22
import { UNINITIALIZED } from '../../../constants.js';
33
import { snapshot } from '../../shared/clone.js';
44
import { define_property } from '../../shared/utils.js';

‎packages/svelte/src/internal/client/dom/blocks/css-props.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/** @import { TemplateNode } from '#client' */
22
import { render_effect, teardown } from '../../reactivity/effects.js';
3-
import { hydrate_node, hydrating, set_hydrate_node } from '../hydration.js';
3+
import { hydrating, set_hydrate_node } from '../hydration.js';
44
import { get_first_child } from '../operations.js';
55

66
/**

‎packages/svelte/src/internal/client/dom/blocks/each.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ import { source, mutable_source, internal_set } from '../../reactivity/sources.j
3535
import { array_from, is_array } from '../../../shared/utils.js';
3636
import { INERT } from '../../constants.js';
3737
import { queue_micro_task } from '../task.js';
38-
import { active_effect, active_reaction, get } from '../../runtime.js';
38+
import { active_effect, get } from '../../runtime.js';
3939
import { DEV } from 'esm-env';
4040
import { derived_safe_equal } from '../../reactivity/deriveds.js';
4141

‎packages/svelte/src/internal/client/dom/elements/bindings/media.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import { hydrating } from '../../hydration.js';
21
import { render_effect, effect, teardown } from '../../../reactivity/effects.js';
32
import { listen } from './shared.js';
43

‎packages/svelte/src/internal/client/dom/elements/custom-element.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { createClassComponent } from '../../../../legacy/legacy-client.js';
2-
import { destroy_effect, effect_root, render_effect } from '../../reactivity/effects.js';
2+
import { effect_root, render_effect } from '../../reactivity/effects.js';
33
import { append } from '../template.js';
44
import { define_property, get_descriptor, object_keys } from '../../../shared/utils.js';
55

‎packages/svelte/src/internal/client/dom/elements/events.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
/** @import { Location } from 'locate-character' */
21
import { teardown } from '../../reactivity/effects.js';
32
import { define_property, is_array } from '../../../shared/utils.js';
43
import { hydrating } from '../hydration.js';

‎packages/svelte/src/internal/client/dom/legacy/event-modifiers.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
/** @import { ActionReturn } from 'svelte/action' */
21
import { noop } from '../../../shared/utils.js';
32
import { user_pre_effect } from '../../reactivity/effects.js';
43
import { on } from '../elements/events.js';

‎packages/svelte/src/internal/client/reactivity/equality.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
/** @import { Equals } from '#client' */
2+
23
/** @type {Equals} */
34
export function equals(value) {
45
return value === this.v;

‎packages/svelte/src/internal/client/reactivity/props.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import {
77
PROPS_IS_RUNES,
88
PROPS_IS_UPDATED
99
} from '../../../constants.js';
10-
import { define_property, get_descriptor, is_function } from '../../shared/utils.js';
10+
import { get_descriptor, is_function } from '../../shared/utils.js';
1111
import { mutable_source, set, source, update } from './sources.js';
1212
import { derived, derived_safe_equal } from './deriveds.js';
1313
import { get, captured_signals, untrack } from '../runtime.js';

‎packages/svelte/src/internal/client/runtime.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ import {
2727
} from './constants.js';
2828
import { flush_tasks } from './dom/task.js';
2929
import { internal_set, old_values } from './reactivity/sources.js';
30-
import { destroy_derived_effects, execute_derived, update_derived } from './reactivity/deriveds.js';
30+
import { destroy_derived_effects, update_derived } from './reactivity/deriveds.js';
3131
import * as e from './errors.js';
3232
import { FILENAME } from '../../constants.js';
3333
import { tracing_mode_flag } from '../flags/index.js';

‎packages/svelte/src/internal/server/dev.js

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -26,14 +26,6 @@ let parent = null;
2626
/** @type {Set<string>} */
2727
let seen;
2828

29-
/**
30-
* @param {Element} element
31-
*/
32-
function stringify(element) {
33-
if (element.filename === null) return `\`<${element.tag}>\``;
34-
return `\`<${element.tag}>\` (${element.filename}:${element.line}:${element.column})`;
35-
}
36-
3729
/**
3830
* @param {Payload} payload
3931
* @param {string} message

0 commit comments

Comments
 (0)