Fix issue with BuiltMap deserialization. Add test.

[davidmorgan]

This change is

[manuelF]

---

Review status: 0 of 12 files reviewed at latest revision, 1 unresolved discussion.

---

example/lib/collections.g.dart, line 116 at r1 (raw file):

class _$CollectionsBuilder extends CollectionsBuilder {
  _$CollectionsBuilder() : super._();
  void replace(Collections other) {

Maybe not for this PR, but could we have a better error message here if other is null? Otherwise it will fail with a exception on the copy of the first element, instead of actually saying that other is null.

---

Comments from Reviewable

[davidmorgan]

Review status: 0 of 12 files reviewed at latest revision, 1 unresolved discussion.

---

example/lib/collections.g.dart, line 116 at r1 (raw file):

Previously, manuelF (Manuel Ferreria) wrote…

Maybe not for this PR, but could we have a better error message here if other is null? Otherwise it will fail with a exception on the copy of the first element, instead of actually saying that other is null.

Thanks, will add that as a FR for built_value.

---

Comments from Reviewable