search

ViveSoftware / ViveInputUtility-Unity

A toolkit that helps developing/prototyping VR apps.
http://u3d.as/uF7
Other
352 stars 82 forks source link

Fixed bug in indexer setter #72

Closed baratgabor closed 5 years ago

baratgabor commented 5 years ago

Fixed a bug in the indexer setter that prevented the Dictionary from updating correctly (while the List did update correctly). The dictionary was re-assigning the old value again, instead of assigning the new one.

So this would fail:

    indexedSetInstance[0] = 666;
    Assert(indexedSetInstance.Contains(666));

Fortunately I didn't run into it during runtime. Just noticed it while I was looking at the code. :) Good job btw; the code so far feels much cleaner than what I saw in the SteamVR plugin. 👍

lawwong commented 5 years ago

Thanks for the bug fix!