|From:||Artur Zakirov <a(dot)zakirov(at)postgrespro(dot)ru>|
|To:||Teodor Sigaev <teodor(at)sigaev(dot)ru>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>|
|Subject:||Re: [PROPOSAL] Improvements of Hunspell dictionaries support|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
I attached a new version of the patch.
On 10.02.2016 19:46, Teodor Sigaev wrote:
>> I duplicate the patch here.
> it's very good thing to update disctionaries to support modern versions.
> And thank you for improving documentation. Also I've impressed by long
> description in spell.c header.
> Som notices about code:
> struct SPELL. Why do you remove union p? You leave comment
> about using d struct instead of flag field and as can see
> it's right comment. It increases size of SPELL structure.
> 2 struct AFFIX. I'm agree with Alvaro taht sum of sizes of bit fields
> should be less or equal to size of integer. In opposite case, suppose,
> we can get undefined behavior. Please, split bitfields to two integers.
> 3 unsigned char flagval;
> Is it forbidden to use 65555 number? In any case, decodeFlag() doesn't
> restrict return value. I suggest to enlarge array to 1<<16 and add limit
> to return value of decodeFlag().
flagval array was enlarged. Added verification of return value of
DecodeFlag() for for various FLAG parameter (FM_LONG, FM_NUM and FM_CHAR).
> I'd like to see a short comment describing at least new functions
Added some comments which describe new functions and old functions for
loading dictionaries into PostgreSQL. This patch adds new functions and
modifies functions which is used for loading dictionaries.
At the moment, comments does not describe functions which used for word
normalization. But I can add more comments.
> Pls, add tests for new code.
Added tests. Old sample dictionaries files was moved to the folder
"dicts". New sample dictionaries files was added:
Postgres Professional: http://www.postgrespro.com
Russian Postgres Company
|Next Message||Dmitry Ivanov||2016-02-16 15:19:23||Re: [WIP] ALTER ... OWNER TO ... CASCADE|
|Previous Message||Tom Lane||2016-02-16 15:10:11||Re: Remove or weaken hints about "effective resolution of sleep delays is 10 ms"?|