Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(872)

Unified Diff: pkg/front_end/lib/src/fasta/kernel/kernel_formal_parameter_builder.dart

Issue 2746013003: [fasta] Add and correct even more offsets (Closed)
Patch Set: Removed formalParametersToken parameter Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pkg/front_end/lib/src/fasta/kernel/kernel_formal_parameter_builder.dart
diff --git a/pkg/front_end/lib/src/fasta/kernel/kernel_formal_parameter_builder.dart b/pkg/front_end/lib/src/fasta/kernel/kernel_formal_parameter_builder.dart
index 09bd87580f71a848ee4885b7c8461d516e248d8d..f7cb56094f117a27bef4fb26ae297db8e68ba14b 100644
--- a/pkg/front_end/lib/src/fasta/kernel/kernel_formal_parameter_builder.dart
+++ b/pkg/front_end/lib/src/fasta/kernel/kernel_formal_parameter_builder.dart
@@ -17,6 +17,7 @@ import 'kernel_builder.dart'
class KernelFormalParameterBuilder
extends FormalParameterBuilder<KernelTypeBuilder> {
VariableDeclaration declaration;
+ final int charOffset;
KernelFormalParameterBuilder(
List<MetadataBuilder> metadata,
@@ -25,7 +26,7 @@ class KernelFormalParameterBuilder
String name,
bool hasThis,
KernelLibraryBuilder compilationUnit,
- int charOffset)
+ this.charOffset)
: super(metadata, modifiers, type, name, hasThis, compilationUnit,
charOffset);
@@ -33,7 +34,7 @@ class KernelFormalParameterBuilder
return declaration ??= new VariableDeclaration(name,
type: type?.build(library) ?? const DynamicType(),
isFinal: isFinal,
- isConst: isConst);
+ isConst: isConst)..fileOffset = charOffset;
}
VariableDeclaration get target => declaration;

Powered by Google App Engine
This is Rietveld 408576698