HashMap temp_init_from_map skips over some elements

[tomaskallup]

Example code:

import std::io;
import std::collections::map;

def Vector2 = int[<2>];

fn uint Vector2.hash(self) => self.x * 31 + self.y;

HashMap(<Vector2, Vector2>) visited_positions;

fn void main() {
  visited_positions.set({1,10}, {0,-1});
  visited_positions.set({1,5}, {0,-1});
  visited_positions.set({1,6}, {0,-1});
  visited_positions.set({1,7}, {0,-1});
  visited_positions.set({3,5}, {0,1});

  visited_positions.@each(;Vector2 pos, Vector2 dir) {
    io::printfn("%s: %s", pos, dir);
  };

  io::printn("Copy:");

  HashMap(<Vector2, Vector2>) visited_positions_copy;
  visited_positions_copy.temp_init_from_map(&visited_positions);
  visited_positions_copy.@each(;Vector2 pos, Vector2 dir) {
    io::printfn("%s: %s", pos, dir);
  };
}

Running this with c3c compile-run test.c3 will result in following output:

Program linked to executable 'test'.
Launching ./test
[<3, 5>]: [<0, 1>]
[<1, 7>]: [<0, -1>]
[<1, 5>]: [<0, -1>]
[<1, 6>]: [<0, -1>]
[<1, 10>]: [<0, -1>]
Copy:
[<3, 5>]: [<0, 1>]
[<1, 5>]: [<0, -1>]
[<1, 6>]: [<0, -1>]
[<1, 10>]: [<0, -1>]

As you can see, [<1, 7>]: [<0, -1>] is missing in the copied HashMap.

[tomaskallup]

Managed to get even simpler repro:

import std::io;
import std::collections::map;

def Map = HashMap(<String, usz>);

fn void main() {
  Map map;
  map.temp_init();

  map.set("aa", 1);
  map.set("b", 2);
  map.set("bb", 1);

  Map map_copy;
  map_copy.temp_init_from_map(&map);
  map_copy.@each(;String key, usz val) {
    io::printfn("%s: %s", key, val);
  };
}