diff options
-rw-r--r-- | src/corelib/tools/qchar.cpp | 13 | ||||
-rw-r--r-- | src/corelib/tools/qchar.h | 6 | ||||
-rw-r--r-- | tests/auto/corelib/tools/qchar/tst_qchar.cpp | 21 |
3 files changed, 35 insertions, 5 deletions
diff --git a/src/corelib/tools/qchar.cpp b/src/corelib/tools/qchar.cpp index 56773bd527..a60fac4a7b 100644 --- a/src/corelib/tools/qchar.cpp +++ b/src/corelib/tools/qchar.cpp @@ -511,18 +511,23 @@ bool QChar::isPrint() const } /*! + \fn bool QChar::isSpace() const + Returns true if the character is a separator character (Separator_* categories or certain code points from Other_Control category); otherwise returns false. */ -bool QChar::isSpace() const + +/*! + \internal + \overload +*/ +bool QChar::isSpace(ushort ucs2) { - if(ucs >= 9 && ucs <=13) - return true; const int test = FLAG(Separator_Space) | FLAG(Separator_Line) | FLAG(Separator_Paragraph); - return FLAG(qGetProp(ucs)->category) & test; + return FLAG(qGetProp(ucs2)->category) & test; } /*! diff --git a/src/corelib/tools/qchar.h b/src/corelib/tools/qchar.h index 36e185ebce..b82addb4c5 100644 --- a/src/corelib/tools/qchar.h +++ b/src/corelib/tools/qchar.h @@ -232,7 +232,10 @@ public: inline bool isNull() const { return ucs == 0; } bool isPrint() const; bool isPunct() const; - bool isSpace() const; + inline bool isSpace() const { + return ucs == 0x20 || (ucs <= 0x0D && ucs >= 0x09) + || (ucs > 127 && isSpace(ucs)); + } bool isMark() const; inline bool isLetter() const { return (ucs >= 'a' && ucs <= 'z') @@ -329,6 +332,7 @@ private: static bool QT_FASTCALL isDigit(ushort ucs2); static bool QT_FASTCALL isLetter(ushort ucs2); static bool QT_FASTCALL isLetterOrNumber(ushort ucs2); + static bool QT_FASTCALL isSpace(ushort ucs2); #ifdef QT_NO_CAST_FROM_ASCII QChar(char c); diff --git a/tests/auto/corelib/tools/qchar/tst_qchar.cpp b/tests/auto/corelib/tools/qchar/tst_qchar.cpp index f16d32a4f9..195abcd578 100644 --- a/tests/auto/corelib/tools/qchar/tst_qchar.cpp +++ b/tests/auto/corelib/tools/qchar/tst_qchar.cpp @@ -82,6 +82,8 @@ private slots: void isPrint(); void isUpper(); void isLower(); + void isSpace_data(); + void isSpace(); void isTitle(); void category(); void direction(); @@ -335,6 +337,25 @@ void tst_QChar::isLower() } } +void tst_QChar::isSpace_data() +{ + QTest::addColumn<ushort>("ucs"); + QTest::addColumn<bool>("expected"); + + for (ushort ucs = 0; ucs < 256; ++ucs) { + bool isSpace = (ucs <= 0x0D && ucs >= 0x09) || ucs == 0x20 || ucs == 0xA0; + QString tag = QString::fromLatin1("0x%0").arg(QString::number(ucs, 16)); + QTest::newRow(tag.toLatin1()) << ucs << isSpace; + } +} + +void tst_QChar::isSpace() +{ + QFETCH(ushort, ucs); + QFETCH(bool, expected); + QCOMPARE(QChar(ucs).isSpace(), expected); +} + void tst_QChar::isTitle() { for (uint codepoint = 0; codepoint <= UNICODE_LAST_CODEPOINT; ++codepoint) { |