XmlAttributeDeserializer crashes if value == ""

[thdxr]

Put in a check for value == string.Empty instead of just checking if value == null.

[johnsheehan]

Switch it to string.IsNullOrEmpty() and we'll pull it in.

[petejohanson]

Hrm... we can't easily just use String.IsNullOrEmpty since value is only of type System.Object.

As such, this patch as is works, but feels a bit "dirty" to me checking an object instance against String.Empty using ==. Thoughts on using:

if (value == null || String.IsNullOrEmpty (value.ToString()))

?

[ayoung]

if (value == null || String.IsNullOrEmpty (value as string))

[ayoung]

Scratch that. That would cause it to almost always be true.

[petejohanson]

Also, @swearword do you have a simple XML document that demonstrates the problem?

[ayoung]

Finally got this in. :) Thanks.