llrb.js – Remove Gets Values Mixed Up

[bherschb]

Thanks for this project. Good stuff. I noticed one small defect, which has a quick fix:

There is a minor typo in llrb.js, in the remove(h, key, compare) method, line 111 currently, where it uses “val”, instead of “value”. This results in the value of the removed node being retained, and the value of another node being lost. Easy fix: just change the two instances of “val” on that line to “value”. Seems to work great after that.