On Fri, Jan 27, 2017 at 10:47:38AM +0100, Fabian Groffen wrote:
> On 27-01-2017 09:58:52 +0100, Fabian Groffen wrote:
> > > Two minor issues:
> > >  - You may want to set LMDB_DB_SIZE to a larger value, see
> > >    https://github.com/neomutt/neomutt/issues/267
> > 
> > Ok, I'll add that.
> > 
> > >  - There's a missing free(&ctx) in hcache_lmdb_open() when
> > >    mdb_env_create() fails.
> > 
> > I'll add it, thanks.
> 
> I looked at this but can't see a problem in the code.  I think you have
> the NeoMutt version in mind because you reference ctx?
> 
> Revised patch attached.

I've pushed this up, with some additional fixes in a separate patch.
Please let me know if I bungled anything.

Thanks again for posting this to the list.

-- 
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA

Attachment: signature.asc
Description: PGP signature

Reply via email to