davidedmundson added a comment.

  > I will try to cook up a more complete solution over the weekend.
  
  If you can explain what you would prefer instead, I can help find time.

INLINE COMMENTS

> bruns wrote in app.cpp:119
> This is a little bit confusing, as the lmdb code uses `rc == LMDB_OK == 0`, 
> but here `0 == ERROR`.

Sure, but engine is an abstraction layer over lmdb so you don't want to leak 
those details, and for a boolean it makes sense that true == good

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D25583

To: davidedmundson, bruns, ngraham
Cc: meven, kde-frameworks-devel, #baloo, hurikhan77, lots0logs, LeGast00n, 
fbampaloukas, GB_2, domson, ashaposhnikov, michaelh, astippich, spoorun, 
ngraham, bruns, abrahams

Reply via email to