<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div>PR <a href="https://github.com/mono/mono/pull/1257">#1257</a> seems to be causing a new regression for me since upgrading from 3.8.0 to 3.12.1 (this PR was merged before 3.10.0).  Xml serialization from SOAP responses where {V}Specified values would need to be set are no longer reliably set.  The only way to get them to work reliably seems to be if I add ShouldSerialize{V} methods.  It seems to go back to this commit.</div><div><br></div><div><a href="https://github.com/mono/mono/commit/52cdea5c0ad5e4ee8ee218b0f90522645ec3ad8a">https://github.com/mono/mono/commit/52cdea5c0ad5e4ee8ee218b0f90522645ec3ad8a</a></div><div><br></div><div>It seems that in some cases, the {V}Specified boolean isnít getting set.  Itís sporadic for me when this issue actually occurs, so I donít yet have an isolated test case, but looking through the code, this logic doesnít seem right to me:</div><div><br></div><div>public bool IsValueSpecifiedSettable () { </div><div>Ö</div><div>    return ((PropertyInfo) _specifiedMember).CanWrite;</div><div>Ö</div><div>    return ((FieldInfo) _specifiedMember).IsInitOnly;</div><div>Ö</div><div>}</div><div><br></div><div><a href="https://github.com/mono/mono/commit/52cdea5c0ad5e4ee8ee218b0f90522645ec3ad8a#diff-3c49e354661f4bb712bf89209bbb85b4R224">https://github.com/mono/mono/commit/52cdea5c0ad5e4ee8ee218b0f90522645ec3ad8a#diff-3c49e354661f4bb712bf89209bbb85b4R224</a></div><div><br></div><div>If itís a writable property we return true, but if it's a read-only field, we also return true?  I think weíd want to return false if it were a read-only field and since mine (and probably everyoneís) {V}Specified fields are generated public boolean fields, then this results in a false so specified doesnít get set.  Iím guessing there is some other race going on because it is set sometimes, not others.  Anyone else having issues with {V}Specified fields since this fix?</div></body></html>