tree 3dc1bf0eee247c7ac911812d93e9f2821f346058
parent 50e01d247596d018ac6686568926c2a52de0fa88
author Nate Bosch <nbosch1@gmail.com> 1601055625 -0700
committer GitHub <noreply@github.com> 1601055625 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJfbiuJCRBK7hj4Ov3rIwAAdHIIABxs+L3p1N94fvkxHBxSy4CO
 dZlLBUPqB5RRrRRN45+TfcoRp/upaSK+1lpYS6ciweobTCaohcLC3YcRj+puSq3W
 Uxk9GRkQ5AC55KhZmT5vyTjC2kkT0qqNv+fjcOG/et1EkDhd6g614jw/NJIpzVuu
 1w9t75POB7jdasKV3ayIFh6m0lO7oO8vSts6bYkpxg77iFvXnqHfG98F6LO5PQvk
 +7Pok+CXg7RaGzMtnyJX+9CWSOTW6mM90PJ+Qit1X6PRI6dipKe6GDqezI9slELs
 3oGYolW6JdVWV/j2AV9Yp20vTWS1ZnGYxQV9byB6ut3We1CZI/TzVNNdRVWxPxs=
 =uDET
 -----END PGP SIGNATURE-----
 

Avoid a `late final` field without initializer (#139)

This had been set in the constructor because it needs to reference
`this`. A new capability with `late final` files is to allow an
initializer expression referencing `this`. This approach also avoid
introducing a setter to the public API. The behavior difference is that
the `CancelableOperation` wont be instantiated until it is read, but
this is fine because `CancelableOperation` does not have any side
effects during initialization and it's only field is the
`CancelableCompleter`. Technically it's statically breaking to remove
the setter, but since it could never be called at runtime this is safe
to do.

Also initialize the `_inner` field where it is defined rather than in an
initializer list since there are not multiple constructors that would
assign different values.