| Index: pkg/observe/lib/src/bindable.dart
|
| diff --git a/pkg/observe/lib/src/bindable.dart b/pkg/observe/lib/src/bindable.dart
|
| index 636ef3034364f352155df1fa35d15fe58402adb7..4b628a60b82e3931ec67ecba50986bc090ac5e55 100644
|
| --- a/pkg/observe/lib/src/bindable.dart
|
| +++ b/pkg/observe/lib/src/bindable.dart
|
| @@ -9,8 +9,10 @@ library observe.src.bindable;
|
| // TODO(jmesserly): Node.bind polyfill calls this "observable"
|
| abstract class Bindable {
|
| // TODO(jmesserly): since we have "value", should open be a void method?
|
| - // TODO(jmesserly): not sure how I feel about open taking a variable number
|
| - // of arguments, but it's the easiest way to make CompoundObserver work.
|
| + // Dart note: changed setValue to be "set value" and discardChanges() to
|
| + // be "get value". Also "set value" implies discard changes.
|
| + // TOOD(jmesserly): is this change too subtle? Is there any other way to
|
| + // make Bindable friendly in a world with getters/setters?
|
|
|
| /// Initiates observation and returns the initial value.
|
| /// The callback will be called with the updated [value].
|
|
|