[PATCH] fix fdo#59892 completely and fdo#33302 partially.

navin patidar (via Code Review) gerrit at gerrit.libreoffice.org
Fri Feb 1 22:07:39 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1961

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/61/1961/1

fix fdo#59892 completely and fdo#33302 partially.

fix fdo#33302 for rtl scripts in cal,impress and draw.

Change-Id: I83b1e077faf0f2578308e03b7598557f60e7cfd8
---
M editeng/source/editeng/impedit3.cxx
1 file changed, 4 insertions(+), 3 deletions(-)



diff --git a/editeng/source/editeng/impedit3.cxx b/editeng/source/editeng/impedit3.cxx
index 964579c..9aba093 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -4288,7 +4288,7 @@
 void ImpEditEngine::ImplInitLayoutMode( OutputDevice* pOutDev, sal_uInt16 nPara, sal_uInt16 nIndex )
 {
     sal_Bool bCTL = sal_False;
-    sal_uInt8 bR2L = sal_False;
+    sal_Bool bR2L = sal_False;
     if ( nIndex == 0xFFFF )
     {
         bCTL = HasScriptType( nPara, i18n::ScriptType::COMPLEX );
@@ -4299,8 +4299,9 @@
         ContentNode* pNode = GetEditDoc().GetObject( nPara );
         short nScriptType = GetScriptType( EditPaM( pNode, nIndex+1 ) );
         bCTL = nScriptType == i18n::ScriptType::COMPLEX;
-        bR2L = GetRightToLeft( nPara, nIndex + 1);  // this change was discussed in issue 37190
-                                                    // it also works for issue 55927
+        // this change was discussed in issue 37190
+        bR2L = GetRightToLeft( nPara, nIndex + 1) == 1 /*RTL*/ ? true : false;
+        // it also works for issue 55927
     }
 
     sal_uLong nLayoutMode = pOutDev->GetLayoutMode();

-- 
To view, visit https://gerrit.libreoffice.org/1961
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I83b1e077faf0f2578308e03b7598557f60e7cfd8
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: navin patidar <patidar at kacst.edu.sa>


More information about the LibreOffice mailing list