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

Side by Side Diff: utils/archive/options.dart

Issue 10919146: Get rid of a lot of () for getters. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 3 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 | « utils/archive/input_stream.dart ('k') | utils/archive/utils.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #library('options'); 5 #library('options');
6 6
7 /** 7 /**
8 * An individual option. 8 * An individual option.
9 */ 9 */
10 class ArchiveOption { 10 class ArchiveOption {
(...skipping 12 matching lines...) Expand all
23 /** 23 /**
24 * A collection of options. 24 * A collection of options.
25 */ 25 */
26 class ArchiveOptions { 26 class ArchiveOptions {
27 /** The internal options map. */ 27 /** The internal options map. */
28 final Map<String, ArchiveOption> _options; 28 final Map<String, ArchiveOption> _options;
29 29
30 ArchiveOptions() : _options = <String, ArchiveOption>{}; 30 ArchiveOptions() : _options = <String, ArchiveOption>{};
31 31
32 /** Returns whether any options have been set. */ 32 /** Returns whether any options have been set. */
33 bool get isEmpty() => _options.isEmpty(); 33 bool get isEmpty => _options.isEmpty();
34 34
35 /** 35 /**
36 * Sets an option. [value] should either be a bool or something with a 36 * Sets an option. [value] should either be a bool or something with a
37 * reasonable `toString` method. 37 * reasonable `toString` method.
38 * 38 *
39 * To set the module for an option, use [operator[]]. 39 * To set the module for an option, use [operator[]].
40 */ 40 */
41 void operator[]=(String name, value) { 41 void operator[]=(String name, value) {
42 _options[name] = new ArchiveOption(name, value); 42 _options[name] = new ArchiveOption(name, value);
43 } 43 }
44 44
45 /** Gets the option with the given name. */ 45 /** Gets the option with the given name. */
46 ArchiveOption operator[](String name) { 46 ArchiveOption operator[](String name) {
47 return _options[name]; 47 return _options[name];
48 } 48 }
49 49
50 /** Gets all options. */ 50 /** Gets all options. */
51 List<ArchiveOption> get all() => _options.getValues(); 51 List<ArchiveOption> get all => _options.getValues();
52 } 52 }
OLDNEW
« no previous file with comments | « utils/archive/input_stream.dart ('k') | utils/archive/utils.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698