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

Side by Side 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 unified diff | Download patch
OLDNEW
1 Sky Style Guide 1 Sky Style Guide
2 =============== 2 ===============
3 3
4 In general, follow our [Design Principles](design.md) for all code. 4 In general, follow our [Design Principles](design.md) for all code.
5 5
6 The primary goal of this style guide is to improve code readability so 6 The primary goal of this style guide is to improve code readability so
7 that everyone, whether reading the code for the first time or 7 that everyone, whether reading the code for the first time or
8 maintaining it for years, can quickly determine what the code does. A 8 maintaining it for years, can quickly determine what the code does. A
9 secondary goal is avoiding arguments when there are disagreements. 9 secondary goal is avoiding arguments when there are disagreements.
10 10
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 > For example, 114 > For example,
115 > ```dart 115 > ```dart
116 > new EdgeDims.symmetric(horizontal: 8.0); 116 > new EdgeDims.symmetric(horizontal: 8.0);
117 > ``` 117 > ```
118 > ...rather than: 118 > ...rather than:
119 > ```dart 119 > ```dart
120 > new EdgeDims(0.0, 8.0, 0.0, 8.0); 120 > new EdgeDims(0.0, 8.0, 0.0, 8.0);
121 > ``` 121 > ```
122 122
123 123
124 Use for-in loops rather than forEach() where possible, since that
125 saves a stack frame per iteration.
126
127
124 C++ 128 C++
125 --- 129 ---
126 130
131 Put spaces around operators in expressions.
132
127 133
128 Java 134 Java
129 ---- 135 ----
130 136
OLDNEW
« 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