Skip to content

Add more tests for grid#1144

Open
eivindjahren wants to merge 5 commits intomainfrom
add_more_tests_for_grid
Open

Add more tests for grid#1144
eivindjahren wants to merge 5 commits intomainfrom
add_more_tests_for_grid

Conversation

@eivindjahren
Copy link
Copy Markdown
Collaborator

No description provided.

@eivindjahren eivindjahren force-pushed the add_more_tests_for_grid branch 6 times, most recently from c8ea960 to cff9fdd Compare April 24, 2026 18:07
@eivindjahren eivindjahren force-pushed the add_more_tests_for_grid branch 2 times, most recently from 6434f16 to 804a4de Compare April 24, 2026 18:51
This is to make it easier to do cpp style memory handling. Also
updates all the lib/tests code to use the new definitions.
The hostnum map is one based as witnessed by the following:

 1. (line 1927): int host_index = hostnum[i] - 1;
 2. (line 824) — same concept in different file format: rd_kw_iset_int(coords_kw, 5, cell->host_cell + 1);
 3. (line 1508): cell->host_cell = coords[5] - 1;
 4. (line 1473): comment uses 0  as sentinel which implies 1-based
@eivindjahren eivindjahren force-pushed the add_more_tests_for_grid branch from 804a4de to 37bb585 Compare April 24, 2026 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant