[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [bug-libunistring] [PATCH v4] unistr: New modules for backward itera
From: |
Daiki Ueno |
Subject: |
Re: [bug-libunistring] [PATCH v4] unistr: New modules for backward iteration in string. |
Date: |
Fri, 19 Sep 2014 16:55:24 +0900 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.4.50 (gnu/linux) |
Ben Pfaff <address@hidden> writes:
> New module 'unistr/u8-mb-prev-uc'.
> New module 'unistr/u16-mb-prev-uc'.
> New module 'unistr/u32-mb-prev-uc'.
Thanks, looks good to me. Some nit-picking below.
* _GL_UNUSED_PARAMETER of u32_mb_prev_uc seems to be a leftover, as the
argument is actually used in the implementation.
* The largest value of possible leading octet tested in
test-u8-mb-prev-uc.c:exhaustive_test is 0xf5, while there are checks
against 0xf8 in u8-mb-prev-uc.c. Also code units above the surrogate
code-point are not checked in u16 and u32 tests.
* The license of u{8,16,32}-mbtouc changed to LGPLv2+ some
time ago, maybe good to follow the change?
> v3->v4: Changed the code to always be "safe". It looks to me like the
> "unsafe" version that I had written originally reflected a misunderstanding
> of how the gnulib option for that was supposed to work.
Are you going to add "unsafe" version later, or is it not useful at all?
Thanks,
--
Daiki Ueno