[Libreoffice] [PATCH] Add numbering type in persian word.

Stephan Bergmann stephan.bergmann.secondary at googlemail.com
Sun Aug 28 07:57:29 PDT 2011


On Aug 28, 2011, at 2:12 PM, Mohammad Elahi wrote:
> static sal_Unicode *table_PersianWord_decade1[]={
>    (sal_Unicode[]){0}, // 0
>    (sal_Unicode[]){0x06cc, 0x06a9, 0}, // 1
>> };
> 
> is not valid C++03 (and neither is Eike's version using the ArrUnicode typedef).  To be standards conform (and hence maximally portable) would require the more verbose
> OK. But it is too verbose ;)
> Any  other solution? what about the following:
> 
>  static sal_Unicode table_PersianWord_decade1[][MAX_LEN]={
>    {0}, // 0
>    {0x06cc, 0x06a9, 0}, // 1
> ...
> };
> 
> Although it uses more memory, but it is less vebose.
> What's your opinion?

Should be OK, too (as the difference in lengths of the individual entries is not too large)---I would make it an explicit "7" in such a case, though, instead of a #define MAX_LEN 7.

-Stephan


More information about the LibreOffice mailing list