search

chrisosaurus / dodo

scriptable in place file editor
MIT License
4 stars 1 forks source link

Fix double free, implement a handful more tests #23

Closed phillid closed 8 years ago

phillid commented 8 years ago

I had an old branch lying around from October(?) last year with tests I used to check that the behaviour from a few bugs we found and fixed around that time were indeed gone. Some of the other tests (such as the whitespace test) are simply included for good measure.

I have also fixed a double free which I accidentally introduced way back in eb445d8 during my memory leak hunt.

Two misc documentation changes are included.

chrisosaurus commented 8 years ago

Sorry for the delay on getting back on this, been flat out.

Thanks again for the amazing work :)

Also super awesome to see more tests coming in, I haven't had the chance to go through all of them yet, but they all pass ;)

phillid commented 8 years ago

No problem about the delay at all. Good to see it merged