organicmaps/icu
13
Fork 0
mirror of https://github.com/unicode-org/icu.git synced 2025-04-08 06:53:45 +00:00
Code Issues 1 Projects Releases Wiki Activity Actions 2

ICU-66 work[] needs to be initialized before using strlen, and don't use strlen() too often in a loop

Browse source 
X-SVN-Rev: 3662
This commit is contained in:
George Rhoten 2001-02-16 23:15:30 +00:00
parent 7d75aab3e8
commit 5b28ea2db1
1 changed files with 13 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
icu4c/source/test/cintltst/cregrtst.c
View file

@ -1048,6 +1048,7 @@ void TestBackwardLineIndexSelection()
free(testLineText);
/*free(lineSelectionData);*/
}
void TestLineInvariants()
{
int errorCount,l;
@ -1059,8 +1060,9 @@ void TestLineInvariants()
UBool saw2;
UChar work[5];
UChar *s, *ustr;
int32_t sLen;
AllocateTextBoundary();
AllocateTextBoundary();
s=(UChar*)malloc(sizeof(UChar) * (u_strlen(cannedTestChars) + 20));
u_strcpy(s, cannedTestChars);
ustr = CharsToUChars(".,;:\\u3001\\u3002\\u3041\\u3042\\u3043\\u3044\\u3045\\u30a3\\u4e00\\u4e01\\u4e02");
@ -1073,6 +1075,7 @@ AllocateTextBoundary();
u_uastrcpy(work, "aaa");
/* in addition to the other invariants, a line-break iterator should make sure that:
it doesn't break around the non-breaking characters */
e = ubrk_open(UBRK_LINE, "en_US", work, u_strlen(work), &status);
@ -1081,15 +1084,15 @@ AllocateTextBoundary();
ustr = CharsToUChars("\\u00a0\\u2007\\u2011\\ufeff");
u_strcpy(noBreak, ustr);
free(ustr);
u_uastrcpy(work, "aaa");
for (i = 0; i < u_strlen(s); i++) {
c = s[i];
sLen = u_strlen(s);
for (i = 0; i < sLen; i++) {
c = s[i];
if (c == '\r' || c == '\n' || c == 0x2029 || c == 0x2028 || c == 0x0003)
continue;
work[0] = c;
for (j = 0; j < u_strlen(noBreak); j++) {
work[1] = noBreak[j];
for (k = 0; k < u_strlen(s); k++) {
for (k = 0; k < sLen; k++) {
work[2] = s[k];
ubrk_setText(e, work, u_strlen(work), &status);
if(U_FAILURE(status)){
@ -1115,12 +1118,12 @@ AllocateTextBoundary();
u_strcpy(dashes, ustr);
free(ustr);
for (i = 0; i < u_strlen(s); i++) {
for (i = 0; i < sLen; i++) {
work[0] = s[i];
for (j = 0; j < u_strlen(dashes); j++) {
work[1] = dashes[j];
for (k = 0; k < u_strlen(s); k++) {
c = s[k];
for (k = 0; k < sLen; k++) {
c = s[k];
if (u_charType(c) == U_DECIMAL_DIGIT_NUMBER ||
u_charType(c) == U_OTHER_NUMBER ||
u_charType(c) == U_NON_SPACING_MARK ||
@ -1137,8 +1140,8 @@ AllocateTextBoundary();
work[2] = c;
ubrk_setText(e, work, u_strlen(work), &status);
if(U_FAILURE(status)){
log_err("FAIL: Error in setting text on the word break Iterator in testLineInvaiants:\n %s \n", myErrorName(status));
return;
log_err("FAIL: Error in setting text on the word break Iterator in testLineInvaiants:\n %s \n", myErrorName(status));
return;
}
saw2 = FALSE;
for (l = ubrk_first(e); l != UBRK_DONE; l = ubrk_next(e))