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

Unified Diff: pkg/kernel/lib/frontend/accessors.dart

Issue 2614663007: Revert "Non-format-changing kernel offset changes" (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « pkg/kernel/lib/clone.dart ('k') | pkg/kernel/lib/kernel.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/kernel/lib/frontend/accessors.dart
diff --git a/pkg/kernel/lib/frontend/accessors.dart b/pkg/kernel/lib/frontend/accessors.dart
index ec010c659b166990cbe8fc12b4419fcd2f54a655..17355b063d6deeead9524712ea2404aeee592bab 100644
--- a/pkg/kernel/lib/frontend/accessors.dart
+++ b/pkg/kernel/lib/frontend/accessors.dart
@@ -41,32 +41,31 @@ abstract class Accessor {
_makeWrite(value, voidContext), new VariableGet(tmp), type)));
}
- Expression buildCompoundAssignment(
- Name binaryOperator, Expression value, int offset,
+ Expression buildCompoundAssignment(Name binaryOperator, Expression value,
{bool voidContext: false, Procedure interfaceTarget}) {
return _finish(_makeWrite(
- builtBinary = makeBinary(
- _makeRead(), binaryOperator, interfaceTarget, value, offset),
+ builtBinary =
+ makeBinary(_makeRead(), binaryOperator, interfaceTarget, value),
voidContext));
}
- Expression buildPrefixIncrement(Name binaryOperator, int offset,
+ Expression buildPrefixIncrement(Name binaryOperator,
{bool voidContext: false, Procedure interfaceTarget}) {
- return buildCompoundAssignment(binaryOperator, new IntLiteral(1), offset,
+ return buildCompoundAssignment(binaryOperator, new IntLiteral(1),
voidContext: voidContext, interfaceTarget: interfaceTarget);
}
- Expression buildPostfixIncrement(Name binaryOperator, int offset,
+ Expression buildPostfixIncrement(Name binaryOperator,
{bool voidContext: false, Procedure interfaceTarget}) {
if (voidContext) {
- return buildPrefixIncrement(binaryOperator, offset,
+ return buildPrefixIncrement(binaryOperator,
voidContext: true, interfaceTarget: interfaceTarget);
}
var value = new VariableDeclaration.forValue(_makeRead());
valueAccess() => new VariableGet(value);
var dummy = new VariableDeclaration.forValue(_makeWrite(
- builtBinary = makeBinary(valueAccess(), binaryOperator, interfaceTarget,
- new IntLiteral(1), offset),
+ builtBinary = makeBinary(
+ valueAccess(), binaryOperator, interfaceTarget, new IntLiteral(1)),
true));
return _finish(makeLet(value, makeLet(dummy, valueAccess())));
}
@@ -402,16 +401,15 @@ Expression makeLet(VariableDeclaration variable, Expression body) {
}
Expression makeBinary(Expression left, Name operator, Procedure interfaceTarget,
- Expression right, int offset) {
+ Expression right) {
return new MethodInvocation(
- left, operator, new Arguments(<Expression>[right]), interfaceTarget)
- ..fileOffset = offset;
+ left, operator, new Arguments(<Expression>[right]), interfaceTarget);
}
final Name _equalOperator = new Name('==');
-Expression buildIsNull(Expression value, {int offset: TreeNode.noOffset}) {
- return makeBinary(value, _equalOperator, null, new NullLiteral(), offset);
+Expression buildIsNull(Expression value) {
+ return makeBinary(value, _equalOperator, null, new NullLiteral());
}
VariableDeclaration makeOrReuseVariable(Expression value) {
« no previous file with comments | « pkg/kernel/lib/clone.dart ('k') | pkg/kernel/lib/kernel.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698