avoid calling f twice in map

[bkamins]

I think calling map(f, ks) twice is not needed, but maybe I am missing something.

@nalimilan - you were the last one to have touched this line so maybe you know the reason for the duplicate call of map?

[codecov]

Codecov Report

Merging #70 (ff0a8cd) into main (35ecfd1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #70   +/-   ##
=======================================
  Coverage   88.36%   88.36%           
=======================================
  Files           1        1           
  Lines         275      275           
=======================================
  Hits          243      243           
  Misses         32       32           

Impacted Files	Coverage Δ
src/PooledArrays.jl	88.36% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35ecfd1...ff0a8cd. Read the comment docs.

[quinnj]

lgtm

[bkamins]

Thank you for approving. Let us wait for @nalimilan to comment as there is no rush in merging this.

[nalimilan]

Good catch. This code was introduced by @shashi at 02dbbdd, and I don't see any reason there not to reuse ks1. Probably just an oversight.

[shashi]

Sounds correct!

[bkamins]

Thank you!