gur111 / mtmex1 Goto Github PK

See CR comment for details

I need this function to verify data is overridden when trying to put something for an existing key

If after using the mapDestroy function we are using mapGetSize we will get an int and not a 0/NULL

We need this for the calculation of the votes.
Related to #20

Make a test which add many many values to the dictionary and then removes them all.
To test how the map handles lots of data.
Maybe 20k values or so.

I wanna fail everyone who limits the map size with a constant 😈

when giving a an empty map, the function mapDestroy tries to free a not existing data

there is some problem in testMapPutGet. specifically in the long string part.
Valgrind shows some kind of a probem in that part of the code
but i cant find the cause

intToStr in augmented_map.c is a void function

augMapPutStr should get a const char* and not char*

We need to add (for each area):

if(max_votes == 0){
    winning_tribe_id = augMapGetMinKey(election->tribes);
}

Originally posted by @gur111 in https://github.com/_render_node/MDExOlB1bGxSZXF1ZXN0NDA2NzIwNTQ1/timeline/more_items

Something like I did in the AugMap to convert from MapResult:

ElectionResult augMapResultToElectionResult(AugMapResult result);

Maybe add a test for the FOR_EACH?
Something like adding to map multiple values.
Then doing a FOR_EACH and for each value we'll add it to a second map. Also validate the iterations is equal to the map's size.
Then FOR_EACH on the second map and verify all the keys exist in the first map.

Now, it is invalid to pass keys below 0 to augmented map.
I need -1 to be used as an error in augMapGetFirst so I changed it.
We need to amend election.c to not validate itself that the id is valid and instead rely on AugMap.

election.c code to use AugMap instead of Map