New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename pubtools #6
Conversation
@@ -151,9 +163,8 @@ class Dart2jsTools { | |||
} | |||
|
|||
String _execName(String name) { | |||
if (Platform.isWindows) { | |||
if (Platform.isWindows) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally, you do want to use braces with if statements, even if the if clause only has a single statement.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, sorry. Now I checked curly brace rule again. https://www.dartlang.org/articles/style-guide/#do-use-curly-braces-for-all-flow-control-structures
Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, as you probably saw, the exception is when it all fits on one line. I personally tend to avoid even that use. I have used it once or twice for things like early return at the start of methods.
Cool, thanks! |
This moves away from tuned split costs to unit-cost spans. Those compose better, which yields better results in nested expressions. Some of the existing tests get changed because the old output is better now. Fixes google#6. Fixes google#19. Fixes google#25. Fixes google#28. Fixes google#55. Fixes google#81. Fixes google#91. Fixes google#95. Fixes google#98. (!) R=pquitslund@google.com, sigmund@google.com Review URL: https://chromiumcodereview.appspot.com//824693005
pub install
topub get
and
pub update
topub upgrade
TEST=./grind init