[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: speed up u8_strstr
From: |
Pádraig Brady |
Subject: |
Re: speed up u8_strstr |
Date: |
Fri, 21 Jan 2011 13:06:30 +0000 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.8) Gecko/20100227 Thunderbird/3.0.3 |
On 21/01/11 12:41, Bruno Haible wrote:
> Hi Pádraig,
> 5) The unit test tests/unistr/test-u-strstr.h: It is good to share code among
> tests.
> rather than write down the same code 3 times. But please follow GNU coding
> style:
> indentation by 2 spaces, not 4, and put spaces around arithmetic operators.
> More importantly, though, the test code that you put there is part of
> tests/test-strstr.c and tests/test-mbsstr1.c. Why only a part? u16_strstr and
> u32_strstr should have all the properties that strstr and mbsstr have. So the
> right thing to do is to adapt all of tests/test-strstr.c into
> tests/unistr/test-u-strstr.h. I've done this, and now I get test failures:
>
> /bin/sh: line 5: 24308 Alarm clock EXEEXT='' srcdir='.'
> MAKE='make' ${dir}$tst
> FAIL: test-u16-strstr
> /bin/sh: line 5: 24325 Alarm clock EXEEXT='' srcdir='.'
> MAKE='make' ${dir}$tst
> FAIL: test-u32-strstr
Yes I was being a bit lazy here.
Eek, can you attached your updated test-u-strstr.h
> So, the first part that can go in is this:
>
>
> 2011-01-21 Pádraig Brady <address@hidden>
> Bruno Haible <address@hidden>
>
> Prepare for faster uN_strstr functions.
OK, I'll adjust on top of that.
Thanks for the review!
Pádraig.