facebook-csharp-sdk/simple-json

Superfluous variable in DeserializeObject

waynebloss opened this issue · 2 comments

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.)

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

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