search

heapwolf / ldb

A C++ REPL / CLI for LevelDB
MIT License
201 stars 23 forks source link

use constant references for some parameters #29

Closed ralphtheninja closed 10 years ago

ralphtheninja commented 10 years ago

Next up: Attacking the main() function to make it super clean. You're gonna love this one ;)

heapwolf commented 10 years ago

W00t! I'm back on ldbcmp for a bit, needs more tests -- generating very large data sets to diff. I'll open source it if you can give my first commit a code review. +@dominictarr

On Saturday, May 17, 2014, ralphtheninja notifications@github.com wrote:

Next up: Attacking the main() function to make it super clean. You're gonna love this one ;)

— Reply to this email directly or view it on GitHubhttps://github.com/hij1nx/ldb/pull/29#issuecomment-43424372 .

Paolo Fragomeni Founder, Here is How http://hereishow.to

github.com/hij1nx twitter.com/hij1nx

ralphtheninja commented 10 years ago

I'd be happy to.

dominictarr commented 10 years ago

sure!

I do have a phobia of C++ though, but maybe @ralphtheninja can hold my hand.

ralphtheninja commented 10 years ago

Always! :)