You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
182 lines
6.2 KiB
182 lines
6.2 KiB
From 49f2a69fa7d8eaa23d77519ac29748f4dc3e4c5a Mon Sep 17 00:00:00 2001
|
|
From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= <caolanm@redhat.com>
|
|
Date: Wed, 28 Nov 2012 11:15:56 +0000
|
|
Subject: [PATCH] bigendian utext mixup triggering regression test failure
|
|
|
|
text takes a pointer to a sal_uInt8 buffer but
|
|
utext takes a pointer to a sal_Unicode buffer
|
|
|
|
passing a sal_uInt8 sequence of e.g. "\x0D\x00" to utext only happens
|
|
to work on little endian machines to represent 0x000D, its 0x0D00 on
|
|
bigendian.
|
|
|
|
for more excitement text and utext do not share the same logic!
|
|
Various special chars are treated different in text vs utext so
|
|
we can't simply blindly change utext() calls to text() calls and
|
|
get the same behaviour without reworking those.
|
|
|
|
So keep the text()/utext() calls as they are, but change what's
|
|
passed to be the right thing.
|
|
|
|
Change-Id: I66696530c4a9482690c461146bdcf0a507b39b68
|
|
---
|
|
.../source/ooxml/OOXMLFastContextHandler.cxx | 51 +++++++++++-----------
|
|
1 file changed, 25 insertions(+), 26 deletions(-)
|
|
|
|
diff --git a/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx b/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx
|
|
index 920bf31..c6f03bb 100644
|
|
--- a/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx
|
|
+++ b/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx
|
|
@@ -39,19 +39,18 @@
|
|
#include "Handler.hxx"
|
|
#include "ooxmlLoggers.hxx"
|
|
|
|
-static const sal_uInt8 s0x7[] = { 0x7, 0x0 };
|
|
-static const sal_uInt8 s0xd[] = { 0xd, 0x0 };
|
|
-static const sal_uInt8 sCR[] = { 0xd, 0x0 };
|
|
-static const sal_uInt8 sFtnEdnRef[] = { 0x2, 0x0 };
|
|
-static const sal_uInt8 sFtnEdnSep[] = { 0x3, 0x0 };
|
|
-static const sal_uInt8 sFtnEdnCont[] = { 0x4, 0x0 };
|
|
-static const sal_uInt8 sTab[] = { 0x9, 0x0 };
|
|
-static const sal_uInt8 sPgNum[] = { 0x0, 0x0 };
|
|
-static const sal_uInt8 sFieldStart[] = { 0x13 };
|
|
-static const sal_uInt8 sFieldSep[] = { 0x14 };
|
|
-static const sal_uInt8 sFieldEnd[] = { 0x15 };
|
|
-static const sal_uInt8 sNoBreakHyphen[] = { 0x1e, 0x0 };
|
|
-static const sal_uInt8 sSoftHyphen[] = { 0x1f, 0x0 };
|
|
+static const sal_Unicode uCR = 0xd;
|
|
+static const sal_Unicode uFtnEdnRef = 0x2;
|
|
+static const sal_Unicode uFtnEdnSep = 0x3;
|
|
+static const sal_Unicode uTab = 0x9;
|
|
+static const sal_Unicode uPgNum = 0x0;
|
|
+static const sal_Unicode uNoBreakHyphen = 0x1e;
|
|
+static const sal_Unicode uSoftHyphen = 0x1f;
|
|
+
|
|
+static const sal_uInt8 cFtnEdnCont = 0x4;
|
|
+static const sal_uInt8 cFieldStart = 0x13;
|
|
+static const sal_uInt8 cFieldSep = 0x14;
|
|
+static const sal_uInt8 cFieldEnd = 0x15;
|
|
|
|
namespace writerfilter {
|
|
namespace ooxml
|
|
@@ -710,7 +709,7 @@ void OOXMLFastContextHandler::startField()
|
|
#endif
|
|
startCharacterGroup();
|
|
if (isForwardEvents())
|
|
- mpStream->text(sFieldStart, 1);
|
|
+ mpStream->text(&cFieldStart, 1);
|
|
endCharacterGroup();
|
|
}
|
|
|
|
@@ -721,7 +720,7 @@ void OOXMLFastContextHandler::fieldSeparator()
|
|
#endif
|
|
startCharacterGroup();
|
|
if (isForwardEvents())
|
|
- mpStream->text(sFieldSep, 1);
|
|
+ mpStream->text(&cFieldSep, 1);
|
|
endCharacterGroup();
|
|
}
|
|
|
|
@@ -732,7 +731,7 @@ void OOXMLFastContextHandler::endField()
|
|
#endif
|
|
startCharacterGroup();
|
|
if (isForwardEvents())
|
|
- mpStream->text(sFieldEnd, 1);
|
|
+ mpStream->text(&cFieldEnd, 1);
|
|
endCharacterGroup();
|
|
}
|
|
|
|
@@ -742,7 +741,7 @@ void OOXMLFastContextHandler::ftnednref()
|
|
debug_logger->element("contexthandler.ftnednref");
|
|
#endif
|
|
if (isForwardEvents())
|
|
- mpStream->utext(sFtnEdnRef, 1);
|
|
+ mpStream->utext((const sal_uInt8*)&uFtnEdnRef, 1);
|
|
}
|
|
|
|
void OOXMLFastContextHandler::ftnednsep()
|
|
@@ -751,7 +750,7 @@ void OOXMLFastContextHandler::ftnednsep()
|
|
debug_logger->element("contexthandler.ftnednsep");
|
|
#endif
|
|
if (isForwardEvents())
|
|
- mpStream->utext(sFtnEdnSep, 1);
|
|
+ mpStream->utext((const sal_uInt8*)&uFtnEdnSep, 1);
|
|
}
|
|
|
|
void OOXMLFastContextHandler::ftnedncont()
|
|
@@ -760,7 +759,7 @@ void OOXMLFastContextHandler::ftnedncont()
|
|
debug_logger->element("contexthandler.ftnedncont");
|
|
#endif
|
|
if (isForwardEvents())
|
|
- mpStream->text(sFtnEdnCont, 1);
|
|
+ mpStream->text(&cFtnEdnCont, 1);
|
|
}
|
|
|
|
void OOXMLFastContextHandler::pgNum()
|
|
@@ -769,7 +768,7 @@ void OOXMLFastContextHandler::pgNum()
|
|
debug_logger->element("contexthandler.pgNum");
|
|
#endif
|
|
if (isForwardEvents())
|
|
- mpStream->utext((const sal_uInt8*)sPgNum, 1);
|
|
+ mpStream->utext((const sal_uInt8*)&uPgNum, 1);
|
|
}
|
|
|
|
void OOXMLFastContextHandler::tab()
|
|
@@ -778,7 +777,7 @@ void OOXMLFastContextHandler::tab()
|
|
debug_logger->element("contexthandler.tab");
|
|
#endif
|
|
if (isForwardEvents())
|
|
- mpStream->utext((const sal_uInt8*)sTab, 1);
|
|
+ mpStream->utext((const sal_uInt8*)&uTab, 1);
|
|
}
|
|
|
|
void OOXMLFastContextHandler::cr()
|
|
@@ -787,7 +786,7 @@ void OOXMLFastContextHandler::cr()
|
|
debug_logger->element("contexthandler.cr");
|
|
#endif
|
|
if (isForwardEvents())
|
|
- mpStream->utext((const sal_uInt8*)sCR, 1);
|
|
+ mpStream->utext((const sal_uInt8*)&uCR, 1);
|
|
}
|
|
|
|
void OOXMLFastContextHandler::noBreakHyphen()
|
|
@@ -796,7 +795,7 @@ void OOXMLFastContextHandler::noBreakHyphen()
|
|
debug_logger->element("contexthandler.noBreakHyphen");
|
|
#endif
|
|
if (isForwardEvents())
|
|
- mpStream->utext((const sal_uInt8*)sNoBreakHyphen, 1);
|
|
+ mpStream->utext((const sal_uInt8*)&uNoBreakHyphen, 1);
|
|
}
|
|
|
|
void OOXMLFastContextHandler::softHyphen()
|
|
@@ -805,7 +804,7 @@ void OOXMLFastContextHandler::softHyphen()
|
|
debug_logger->element("contexthandler.softHyphen");
|
|
#endif
|
|
if (isForwardEvents())
|
|
- mpStream->utext((const sal_uInt8*)sSoftHyphen, 1);
|
|
+ mpStream->utext((const sal_uInt8*)&uSoftHyphen, 1);
|
|
}
|
|
|
|
void OOXMLFastContextHandler::handleLastParagraphInSection()
|
|
@@ -829,7 +828,7 @@ void OOXMLFastContextHandler::endOfParagraph()
|
|
if (! mpParserState->isInCharacterGroup())
|
|
startCharacterGroup();
|
|
if (isForwardEvents())
|
|
- mpStream->utext((const sal_uInt8*)sCR, 1);
|
|
+ mpStream->utext((const sal_uInt8*)&uCR, 1);
|
|
}
|
|
|
|
void OOXMLFastContextHandler::startTxbxContent()
|
|
@@ -1876,7 +1875,7 @@ void OOXMLFastContextHandlerTextTableRow::endRow()
|
|
startCharacterGroup();
|
|
|
|
if (isForwardEvents())
|
|
- mpStream->utext(s0xd, 1);
|
|
+ mpStream->utext((const sal_uInt8*)&uCR, 1);
|
|
|
|
endCharacterGroup();
|
|
endParagraphGroup();
|
|
--
|
|
1.7.11.7
|
|
|