buzz-lang / BittyBuzz

BittyBuzz is an implementation of Buzz for microcrontrollers.
MIT License
8 stars 7 forks source link

WIP: Removed internal string in neighbors #22

Closed xgroleau closed 3 years ago

xgroleau commented 3 years ago

As specified #12, an internal string is inserted in the neighbors table, but never seems to be used since bbztable_size function is used to get the number of neighbors. Also the added internal string element to the table made the count wrong. I'm not sure why the code was there, so maybe I've missed something and this is not the way to do it though.

xgroleau commented 3 years ago

This seems to fix the neighbors.foreach problem, but I still have issues with the neighbors.reduce function after using a neighbors.map

beltrame commented 3 years ago

This includes the removal of the internal string for counting neighbors. I am not sure why it was there in the first place, @Belkharym do you have an idea?

beltrame commented 3 years ago

Covered by #23