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

Unified Diff: sky/specs/style-guide.md

Issue 1211573003: Fill out some more documentation about building RenderBox subclasses. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 6 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
« sky/sdk/lib/rendering/README.md ('K') | « sky/sdk/lib/rendering/stack.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/specs/style-guide.md
diff --git a/sky/specs/style-guide.md b/sky/specs/style-guide.md
index e71ad84061b45cd04ecf1063bc5a20870abe350e..123acd453eb91acbb0b43a1dbd59332dd9ccc1d1 100644
--- a/sky/specs/style-guide.md
+++ b/sky/specs/style-guide.md
@@ -121,9 +121,15 @@ options.
> ```
+Use for-in loops rather than forEach() where possible, since that
+saves a stack frame per iteration.
+
+
C++
---
+Put spaces around operators in expressions.
+
Java
----
« sky/sdk/lib/rendering/README.md ('K') | « sky/sdk/lib/rendering/stack.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698