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

Unified Diff: pkg/analyzer/lib/src/summary/format.dart

Issue 1648783002: Repalce shiftOr with pushLongInt, tweak makeSymbol. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 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 | « no previous file | pkg/analyzer/lib/src/summary/summarize_const_expr.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analyzer/lib/src/summary/format.dart
diff --git a/pkg/analyzer/lib/src/summary/format.dart b/pkg/analyzer/lib/src/summary/format.dart
index 4a0424b95421c18b30288ea1013ac5f483eb4f5d..4522e2443ec1659932028a88376aaa57d08f370d 100644
--- a/pkg/analyzer/lib/src/summary/format.dart
+++ b/pkg/analyzer/lib/src/summary/format.dart
@@ -96,17 +96,17 @@ enum UnlinkedConstOperation {
* onto the stack.
*
* Note that Dart supports integers larger than 32 bits; these are
- * represented by composing 32 bit values using the [shiftOr] operation.
+ * represented by composing 32-bit values using the [pushLongInt] operation.
*/
pushInt,
/**
- * Pop the top value off the stack, which should be an integer. Multiply it
- * by 2^32, "or" in the next value from [UnlinkedConst.ints] (which is
- * interpreted as a 32-bit unsigned integer), and push the result back onto
- * the stack.
+ * Get the number of components from [UnlinkedConst.ints], then do this number
+ * of times the following operations: multiple the current value by 2^32, "or"
+ * it with the next value in [UnlinkedConst.ints]. The initial value is zero.
+ * Push the result into the stack.
*/
- shiftOr,
+ pushLongInt,
/**
* Push the next value from [UnlinkedConst.doubles] (a double precision
@@ -140,8 +140,8 @@ enum UnlinkedConstOperation {
concatenate,
/**
- * Pop the top value from the stack which should be string, convert it to
- * a symbol, and push it back onto the stack.
+ * Get the next value from [UnlinkedConst.strings], convert it to a symbol,
+ * and push it onto the stack.
*/
makeSymbol,
« no previous file with comments | « no previous file | pkg/analyzer/lib/src/summary/summarize_const_expr.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698