VBA-tools / VBA-Dictionary

Drop-in replacement for Scripting.Dictionary on Mac
MIT License
348 stars 89 forks source link

Fix dict_RemoveKeyValue #14

Closed yaaax closed 8 years ago

yaaax commented 8 years ago

No need to remove 1 to dict_Index

timhall commented 8 years ago

Thanks for the PR @yaaacov! I'll review it shortly

yaaax commented 8 years ago

Hi Tim, Any news for that fix? It fixed an issue on my particular macro.. but it definitely needs a quick review. Thanks for tools anyway!

timhall commented 8 years ago

What issue did you run into? (I'd like to add a test case for it)

timhall commented 8 years ago

Hmm, there seem to be some other indexing issues with Keys and Items, I'm continuing to look into it