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

Side by Side Diff: pkg/analyzer/test/services/data/cu_tests.data

Issue 153623006: formatter nits (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 10 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | pkg/analyzer/test/services/data/stmt_tests.data » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 >>> 1 >>>
2 class 2 class
3 A 3 A
4 { 4 {
5 } 5 }
6 <<< 6 <<<
7 class A { 7 class A {
8 } 8 }
9 >>> dartbug.com/15876 9 >>> dartbug.com/15876
10 // fisk 10 // fisk
(...skipping 182 matching lines...) Expand 10 before | Expand all | Expand 10 after
193 List<double> tangentImpulses = new List<double>(Settings.MAX_MANIFOLD_POINTS); 193 List<double> tangentImpulses = new List<double>(Settings.MAX_MANIFOLD_POINTS);
194 194
195 ContactImpulse(); 195 ContactImpulse();
196 } 196 }
197 <<< 197 <<<
198 class ContactImpulse { 198 class ContactImpulse {
199 List<double> normalImpulses = new List<double>(Settings.MAX_MANIFOLD_POINTS); 199 List<double> normalImpulses = new List<double>(Settings.MAX_MANIFOLD_POINTS);
200 List<double> tangentImpulses = new List<double>(Settings.MAX_MANIFOLD_POINTS); 200 List<double> tangentImpulses = new List<double>(Settings.MAX_MANIFOLD_POINTS);
201 201
202 ContactImpulse(); 202 ContactImpulse();
203 } 203 }
204 >>> one line call to super is okay
pquitslund 2014/02/07 00:19:13 Fixed!
205 class Foo extends Bar {
206 Foo():super();
207 }
208 <<<
209 class Foo extends Bar {
210 Foo() : super();
211 }
212 >>> keep empty methods on a single line
pquitslund 2014/02/07 00:19:13 Fixed!
213 void main(){}
214 <<<
215 void main() { }
216 >>> avoid splitting types and names in argument list
217 void aVeryLongMethodWithParamsAndOther(List<String> a, List<String> b, List<Stri ng> c, List<String> d) {
218 // TODO impl
219 }
220 <<<
221 void aVeryLongMethodWithParamsAndOther(List<String> a, List<String> b,
222 List<String> c, List<String> d) {
223 // TODO impl
224 }
OLDNEW
« no previous file with comments | « no previous file | pkg/analyzer/test/services/data/stmt_tests.data » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698