Superfluous variable in DeserializeObject

facebook-csharp-sdk / simple-json

JSON library for .NET 2.0+/SL4+/WP7/WindowsStore with optional support for dynamic and DataContract

MIT License

380 stars 143 forks source link

Superfluous variable in DeserializeObject #59

Closed ghost closed 9 years ago

ghost commented 10 years ago

The jsonObject variable declared on Line 1398 of SimpleJson.cs is superfluous because it is only being used as a reference to the objects variable.

To fix this, the objects variable could be renamed to valueAsIDict and the declaration of jsonObject on line 1398 could be removed.

(Same thing for the valueAsList / jsonObject variables in the next section down starting at Line 1437.)

ghost commented 10 years ago

Sorry for not making a pull request yet, I may do so at some point in the future.

ghost commented 9 years ago

Closing issues on projects that don't seem to have a maintainer.