Control: tags -1 +patch
On Sun, Apr 24, 2022 at 10:18 PM Adrian Bunk wrote:
> Source: qtwebkit-opensource-src
> Version: 5.212.0~alpha4-14
> Severity: serious
> Tags: ftbfs
>
> In file included from
> /<>/Source/WebCore/platform/text/TextAllInOne.cpp:31:
> /<>/Source/WebCore/platform/text/TextCodecICU.cpp: In member
> function ‘void WebCore::TextCodecICU::createICUConverter() const’:
> /<>/Source/WebCore/platform/text/TextCodecICU.cpp:311:42: error:
> ‘TRUE’ was not declared in this scope
> 311 | ucnv_setFallback(m_converterICU, TRUE);
> | ^~~~
> ...
My bad, submitting the required patch only now.
Regards,
Laszalo/GCS
Description: replace old ICU TRUE / FALSE constants
For some time ICU (since 68.1+ for sure) no longer specify nonstandard
TRUE / FALSE constants. Use UBool(1) and UBool(0) instead.
Author: Laszlo Boszormenyi (GCS)
Bug-Debian: https://bugs.debian.org/1010118
Forwarded: no
Last-Update: 2022-04-24
---
--- qtwebkit-opensource-src-5.212.0~alpha4.orig/Source/WebCore/platform/text/TextCodecICU.cpp
+++ qtwebkit-opensource-src-5.212.0~alpha4/Source/WebCore/platform/text/TextCodecICU.cpp
@@ -308,7 +308,7 @@ void TextCodecICU::createICUConverter()
m_converterICU = ucnv_open(m_canonicalConverterName, &err);
ASSERT(U_SUCCESS(err));
if (m_converterICU)
-ucnv_setFallback(m_converterICU, TRUE);
+ucnv_setFallback(m_converterICU, UBool(1));
}
int TextCodecICU::decodeToBuffer(UChar* target, UChar* targetLimit, const char*& source, const char* sourceLimit, int32_t* offsets, bool flush, UErrorCode& err)
--- qtwebkit-opensource-src-5.212.0~alpha4.orig/Source/WebCore/platform/text/icu/UTextProvider.h
+++ qtwebkit-opensource-src-5.212.0~alpha4/Source/WebCore/platform/text/icu/UTextProvider.h
@@ -80,12 +80,12 @@ inline bool uTextAccessInChunkOrOutOfRan
// Ensure chunk offset is well formed if computed offset exceeds int32_t range.
ASSERT(offset < std::numeric_limits::max());
text->chunkOffset = offset < std::numeric_limits::max() ? static_cast(offset) : 0;
-isAccessible = TRUE;
+isAccessible = UBool(1);
return true;
}
if (nativeIndex >= nativeLength && text->chunkNativeLimit == nativeLength) {
text->chunkOffset = text->chunkLength;
-isAccessible = FALSE;
+isAccessible = UBool(0);
return true;
}
} else {
@@ -94,12 +94,12 @@ inline bool uTextAccessInChunkOrOutOfRan
// Ensure chunk offset is well formed if computed offset exceeds int32_t range.
ASSERT(offset < std::numeric_limits::max());
text->chunkOffset = offset < std::numeric_limits::max() ? static_cast(offset) : 0;
-isAccessible = TRUE;
+isAccessible = UBool(1);
return true;
}
if (nativeIndex <= 0 && !text->chunkNativeStart) {
text->chunkOffset = 0;
-isAccessible = FALSE;
+isAccessible = UBool(0);
return true;
}
}
--- qtwebkit-opensource-src-5.212.0~alpha4.orig/Source/WebCore/platform/text/icu/UTextProviderLatin1.cpp
+++ qtwebkit-opensource-src-5.212.0~alpha4/Source/WebCore/platform/text/icu/UTextProviderLatin1.cpp
@@ -100,23 +100,23 @@ static UBool uTextLatin1Access(UText* uT
if (index < uText->chunkNativeLimit && index >= uText->chunkNativeStart) {
// Already inside the buffer. Set the new offset.
uText->chunkOffset = static_cast(index - uText->chunkNativeStart);
-return TRUE;
+return UBool(1);
}
if (index >= length && uText->chunkNativeLimit == length) {
// Off the end of the buffer, but we can't get it.
uText->chunkOffset = static_cast(index - uText->chunkNativeStart);
-return FALSE;
+return UBool(0);
}
} else {
if (index <= uText->chunkNativeLimit && index > uText->chunkNativeStart) {
// Already inside the buffer. Set the new offset.
uText->chunkOffset = static_cast(index - uText->chunkNativeStart);
-return TRUE;
+return UBool(1);
}
if (!index && !uText->chunkNativeStart) {
// Already at the beginning; can't go any farther.
uText->chunkOffset = 0;
-return FALSE;
+return UBool(0);
}
}
@@ -144,7 +144,7 @@ static UBool uTextLatin1Access(UText* uT
uText->nativeIndexingLimit = uText->chunkLength;
-return TRUE;
+return UBool(1);
}
static int32_t uTextLatin1Extract(UText* uText, int64_t start, int64_t limit, UChar*
