Code review comment for lp:~gz/juju-core/add_state_address
Revision history for this message
| John A Meinel (jameinel) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
How is this linked with the other data structures?
Is it intended that machine will grow an array of Address objects?
I want us to be aware of what happened with adding the container field
(be ready to handle fields that don't exist, as well as ones that have
empty content).
Otherwise LGTM.
I like the Address.Value change proposed by William.
https:/