(#6099) - use Map/Set in viewCleanup() #6099

Merged
merged 1 commit into from Jan 4, 2017

Projects

None yet

2 participants

@nolanlawson
Member

One more place where we were unsafely mapping docIds using {} instead of Map/Set...

I should probably add a test to do map/reduce with a doc ID like "constructor" but haven't gotten around to it yet.

@nolanlawson nolanlawson (#6099) - use Map/Set in viewCleanup()
a72a60d
@nolanlawson
Member

OK, added a new test.

@daleharvey

Awesome, lets get this in for the release

@daleharvey daleharvey merged commit 77547f7 into master Jan 4, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 100.0%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment