-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
review: needs-information
Needs info because of [1]
[1]
> -type SetNetworkConfiguration struct { - XMLName > xml.Name `xml:"NetworkConfiguration"` +type > NetworkConfiguration struct { XMLNS string > `xml:"xmlns,attr"` - XMLNS_XSI string > `xml:"xmlns:xsi,attr"`
Why remove this? It is required for requests. (It's also in the response from GetNetworkConfiguration but is ignorable there I guess.)
You also changed it in the test template (which is copied directly from the msdn web page):
> - <NetworkConfiguration xmlns="http://schemas.microsoft.com/ServiceHosting/2011/07/NetworkConfiguration" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"> > + <NetworkConfiguration xmlns="http://schemas.microsoft.com/ServiceHosting/2011/07/NetworkConfiguration">
[2]
> -func (s *SetNetworkConfiguration) Serialize() (string, error) { > +func (s *NetworkConfiguration) Serialize() (string, error) { > return toxml(s) } > > +func (s *NetworkConfiguration) Deserialize(data []byte) error { + > return xml.Unmarshal(data, s) +} +
A small point, but "s" is no longer appropriate.
[3]
> +func (suite *xmlSuite) TestNetworkConfigurationDeserialize(c *C) > { + // Template from + // > http://msdn.microsoft.com/en-us/library/windowsazure/jj157181.aspx
You quoted the URL for "set config". I think this should be the "get config" at: http://msdn.microsoft.com/en-us/library/windowsazure/jj157196.aspx
Everything else looks good, thanks! -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iEYEARECAAYFAlHcqY0ACgkQWhGlTF8G/Hcm7gCeIJUSZ1jQYX9dnpvErUAEechu rZEAnjhnxnaeqz0seeYphG1osAwEvn9e =sfAh -----END PGP SIGNATURE-----
« Back to merge proposal
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
review: needs-information
Needs info because of [1]
[1]
> -type SetNetworkConfi guration struct { - XMLName nfiguration" ` +type ation struct { XMLNS string xsi,attr" `
> xml.Name `xml:"NetworkCo
> NetworkConfigur
> `xml:"xmlns,attr"` - XMLNS_XSI string
> `xml:"xmlns:
Why remove this? It is required for requests. (It's also in the guration but is ignorable there I guess.)
response from GetNetworkConfi
You also changed it in the test template (which is copied directly
from the msdn web page):
> - <NetworkConfigu ration schemas. microsoft. com/ServiceHost ing/2011/ 07/NetworkConfi guration" www.w3. org/2001/ XMLSchema- instance"> ration schemas. microsoft. com/ServiceHost ing/2011/ 07/NetworkConfi guration">
xmlns="http://
xmlns:xsi="http://
> + <NetworkConfigu
xmlns="http://
[2]
> -func (s *SetNetworkConf iguration) Serialize() (string, error) { ration) Serialize() (string, error) { ration) Deserialize(data []byte) error { +
> +func (s *NetworkConfigu
> return toxml(s) }
>
> +func (s *NetworkConfigu
> return xml.Unmarshal(data, s) +} +
A small point, but "s" is no longer appropriate.
[3]
> +func (suite *xmlSuite) TestNetworkConf igurationDeseri alize(c *C) msdn.microsoft. com/en- us/library/ windowsazure/ jj157181. aspx
> { + // Template from + //
> http://
You quoted the URL for "set config". I think this should be the "get msdn.microsoft. com/en- us/library/ windowsazure/ jj157196. aspx
config" at:
http://
Everything else looks good, thanks! www.enigmail. net/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://
iEYEARECAAYFAlH cqY0ACgkQWhGlTF 8G/Hcm7gCeIJUSZ 1jQYX9dnpvErUAE echu seeYphG1osAwEvn 9e
rZEAnjhnxnaeqz0
=sfAh
-----END PGP SIGNATURE-----