[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: LMDB and text encoding

Timur Kristóf wrote:

I forgot to add an ENOMEM check. I added it now. I think this patch is
ready for Howard and Hallvard to review. :)

It looks OK to me. No one raises any concerns I'll commit it in a few hours.


On Thu, Jan 29, 2015 at 2:42 PM, Timur Kristóf <timur.kristof@gmail.com> wrote:
Here is a fixed version of the patch.

On Thu, Jan 29, 2015 at 10:29 AM, Timur Kristóf <timur.kristof@gmail.com> wrote:
mdb_dbi_open treats its name parameter as a C string. This means UTF-8 on
unixes and ANSI on Windows, which is problematic for cross-platform
applications. [...]

Here is a patch that addresses this concern.
If you like it, I'll move on to the other issue.

  -- Howard Chu
  CTO, Symas Corp.           http://www.symas.com
  Director, Highland Sun     http://highlandsun.com/hyc/
  Chief Architect, OpenLDAP  http://www.openldap.org/project/