[Libreoffice-commits] .: editeng/source

2013-02-01 Thread navin patidar
 editeng/source/editeng/impedit3.cxx |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 73de30ca9ca0d3d4edd00e0e8f22451238699989
Author: navin patidar pati...@kacst.edu.sa
Date:   Sat Feb 2 08:40:30 2013 +0300

fix fdo#59892 completely and fdo#33302 partially.

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

Change-Id: I83b1e077faf0f2578308e03b7598557f60e7cfd8
Reviewed-on: https://gerrit.libreoffice.org/1961
Reviewed-by: Ahmad Harthi aalhar...@kacst.edu.sa
Tested-by: Ahmad Harthi aalhar...@kacst.edu.sa

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 74f6f9e..4458937 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -4275,7 +4275,7 @@ void ImpEditEngine::ImplInitDigitMode(OutputDevice* 
pOutDev, LanguageType eCurLa
 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 == 0x )
 {
 bCTL = HasScriptType( nPara, i18n::ScriptType::COMPLEX );
@@ -4286,8 +4286,9 @@ void ImpEditEngine::ImplInitLayoutMode( OutputDevice* 
pOutDev, sal_uInt16 nPara,
 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();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2013-01-08 Thread Libreoffice Gerrit user
 editeng/source/editeng/editobj2.hxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 63c19c881247bbb7730bdb767697018f61340697
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Jan 8 22:28:07 2013 -0500

Remove unnecessary virtual keywords.

Change-Id: I1227a4e04bc962da96b77947a706293f9a2a3850

diff --git a/editeng/source/editeng/editobj2.hxx 
b/editeng/source/editeng/editobj2.hxx
index 04b6ba1..6c0f586 100644
--- a/editeng/source/editeng/editobj2.hxx
+++ b/editeng/source/editeng/editobj2.hxx
@@ -213,7 +213,7 @@ public:
 String GetText(size_t nParagraph) const;
 void Insert(const EditTextObject rObj, size_t nPara);
 EditTextObject* CreateTextObject(size_t nPara, size_t nParas = 1) const;
-virtual void RemoveParagraph(size_t nPara);
+void RemoveParagraph(size_t nPara);
 
 bool HasPortionInfo() const;
 void ClearPortionInfo();
@@ -240,7 +240,7 @@ public:
 void SetStyleSheet(size_t nPara, const String rName, const 
SfxStyleFamily eFamily);
 bool ChangeStyleSheets(
 const XubString rOldName, SfxStyleFamily eOldFamily, const String 
rNewName, SfxStyleFamily eNewFamily );
-virtual voidChangeStyleSheetName( SfxStyleFamily eFamily, 
const XubString rOldName, const XubString rNewName );
+void ChangeStyleSheetName( SfxStyleFamily eFamily, const XubString 
rOldName, const XubString rNewName );
 
 editeng::FieldUpdater GetFieldUpdater();
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2013-01-08 Thread Libreoffice Gerrit user
 editeng/source/editeng/editobj.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit a796fdf07ba6196eaa1ed41b2846c99a9eda421a
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Jan 8 22:32:56 2013 -0500

Explicitly initialize SfxItemPoolUser in copy constructor.

To keep MacOSX-Intel tinderbox happy.

Change-Id: I8dbc8330fd688fc1c1ed005d1603ff52f8b7a16d

diff --git a/editeng/source/editeng/editobj.cxx 
b/editeng/source/editeng/editobj.cxx
index 06a2d34..41698af 100644
--- a/editeng/source/editeng/editobj.cxx
+++ b/editeng/source/editeng/editobj.cxx
@@ -185,6 +185,7 @@ EditTextObject::EditTextObject( SfxItemPool* pPool ) :
 }
 
 EditTextObject::EditTextObject( const EditTextObject r ) :
+SfxItemPoolUser(),
 mpImpl(new EditTextObjectImpl(this, *r.mpImpl))
 {
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source sc/source sw/source

2013-01-06 Thread Libreoffice Gerrit user
 editeng/source/editeng/editeng.src |2 +-
 sc/source/ui/src/globstr.src   |2 +-
 sw/source/core/undo/undo.src   |2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 93a9e52dccf15b53fc1935145df24306e3a4fc3e
Author: Korrawit Pruegsanusak detective.conan.1...@gmail.com
Date:   Sun Dec 23 12:40:09 2012 +0700

fdo#58671: Remove accelerators from Undo action texts

These three texts appear in Undo dropdown list, so they don't need 
accelerators

Change-Id: Icec8e199c7cc3990b6316937e49aacb7eb1015fb
Reviewed-on: https://gerrit.libreoffice.org/1473
Reviewed-by: Ivan Timofeev timofeev@gmail.com
Tested-by: Ivan Timofeev timofeev@gmail.com

diff --git a/editeng/source/editeng/editeng.src 
b/editeng/source/editeng/editeng.src
index 655861b..ad881e3 100644
--- a/editeng/source/editeng/editeng.src
+++ b/editeng/source/editeng/editeng.src
@@ -61,7 +61,7 @@ String RID_EDITUNDO_SETSTYLE
 
 String RID_EDITUNDO_TRANSLITERATE
 {
-Text [ en-US ] = ~Change Case;
+Text [ en-US ] = Change Case;
 };
 
 
diff --git a/sc/source/ui/src/globstr.src b/sc/source/ui/src/globstr.src
index 79ad06e..a5ac8b2 100644
--- a/sc/source/ui/src/globstr.src
+++ b/sc/source/ui/src/globstr.src
@@ -354,7 +354,7 @@ Resource RID_GLOBSTR
 };
 String STR_UNDO_TRANSLITERATE
 {
-Text [ en-US ] = ~Change Case;
+Text [ en-US ] = Change Case;
 };
 String STR_DBNAME_IMPORT
 {
diff --git a/sw/source/core/undo/undo.src b/sw/source/core/undo/undo.src
index aebd3b8..d5c1ac4 100644
--- a/sw/source/core/undo/undo.src
+++ b/sw/source/core/undo/undo.src
@@ -267,7 +267,7 @@ String STR_MERGE_TABLE
 
 String STR_TRANSLITERATE
 {
-Text [ en-US ] = ~Change Case;
+Text [ en-US ] = Change Case;
 };
 
 String STR_DELNUM
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2013-01-04 Thread Libreoffice Gerrit user
 editeng/source/editeng/editeng.src |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit d39f1fb508128c23c39c24e04b20a2f56c03cb5e
Author: Samuel Mehrbrodt s.mehrbr...@gmail.com
Date:   Fri Jan 4 00:55:23 2013 +0100

Spellcheck menu: Rename 'Add' to 'Add to Dictionary'

Change-Id: I601bf1fc34f3e4cccef02d1d4a5ecd1959007cf7
Reviewed-on: https://gerrit.libreoffice.org/1539
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/editeng/source/editeng/editeng.src 
b/editeng/source/editeng/editeng.src
index 2bea4aa..655861b 100644
--- a/editeng/source/editeng/editeng.src
+++ b/editeng/source/editeng/editeng.src
@@ -82,13 +82,13 @@ Menu RID_MENU_SPELL
 SubMenu = Menu
 {
 };
-Text [ en-US ] = ~Add ;
+Text [ en-US ] = ~Add to Dictionary ;
 };
 MenuItem
 {
 Identifier = MN_INSERT_SINGLE ;
 HelpId = HID_EDITENG_SPELLER_ADDWORD;
-Text [ en-US ] = ~Add ;
+Text [ en-US ] = ~Add to Dictionary ;
 };
 MenuItem
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-12-25 Thread Libreoffice Gerrit user
 editeng/source/accessibility/AccessibleContextBase.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 0897b55183dc76f5c9833fb2649116f560a1f034
Author: Peter Foley pefol...@verizon.net
Date:   Tue Dec 25 09:44:16 2012 -0500

build fix for editeng OUString cleanup

Change-Id: I82fdca86631d5b364224d151d85da5baef1a54a8

diff --git a/editeng/source/accessibility/AccessibleContextBase.cxx 
b/editeng/source/accessibility/AccessibleContextBase.cxx
index 895f655..14cb720 100644
--- a/editeng/source/accessibility/AccessibleContextBase.cxx
+++ b/editeng/source/accessibility/AccessibleContextBase.cxx
@@ -504,8 +504,8 @@ uno::Sequence OUString  SAL_CALL
 {
 ThrowIfDisposed ();
 static const OUString sServiceNames[2] = {
-com.sun.star.accessibility.Accessible,
-com.sun.star.accessibility.AccessibleContext
+OUString(com.sun.star.accessibility.Accessible),
+OUString(com.sun.star.accessibility.AccessibleContext)
 };
 return uno::SequenceOUString (sServiceNames, 2);
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-12-24 Thread Libreoffice Gerrit user
 editeng/source/accessibility/AccessibleComponentBase.cxx|   10 -
 editeng/source/accessibility/AccessibleContextBase.cxx  |   39 ++---
 editeng/source/accessibility/AccessibleEditableTextPara.cxx |   88 ++--
 editeng/source/accessibility/AccessibleHyperlink.cxx|6 
 editeng/source/accessibility/AccessibleHyperlink.hxx|6 
 5 files changed, 70 insertions(+), 79 deletions(-)

New commits:
commit 0e028d6ceb2a3900af06f50efecfdc0565b5e7b2
Author: Olivier Hallot olivier.hal...@alta.org.br
Date:   Fri Dec 21 23:10:24 2012 -0200

OUString cleanups in editeng/accessibility

Change-Id: Ie280c48cabec4ef9e0c623f8966a65027b975fe4
Reviewed-on: https://gerrit.libreoffice.org/1468
Reviewed-by: Thomas Arnhold tho...@arnhold.org
Tested-by: Thomas Arnhold tho...@arnhold.org
Reviewed-by: Norbert Thiebaud nthieb...@gmail.com
Tested-by: Norbert Thiebaud nthieb...@gmail.com

diff --git a/editeng/source/accessibility/AccessibleComponentBase.cxx 
b/editeng/source/accessibility/AccessibleComponentBase.cxx
index 79ef598..f796c33 100644
--- a/editeng/source/accessibility/AccessibleComponentBase.cxx
+++ b/editeng/source/accessibility/AccessibleComponentBase.cxx
@@ -184,19 +184,17 @@ sal_Int32 SAL_CALL AccessibleComponentBase::getBackground 
(void)
 
 
 
-::rtl::OUString SAL_CALL AccessibleComponentBase::getTitledBorderText (void)
+OUString SAL_CALL AccessibleComponentBase::getTitledBorderText (void)
 throw (::com::sun::star::uno::RuntimeException)
 {
-return ::rtl::OUString ();
+return OUString();
 }
 
 
-
-
-::rtl::OUString SAL_CALL AccessibleComponentBase::getToolTipText (void)
+OUString SAL_CALL AccessibleComponentBase::getToolTipText (void)
 throw (::com::sun::star::uno::RuntimeException)
 {
-return ::rtl::OUString ();
+return OUString();
 }
 
 
diff --git a/editeng/source/accessibility/AccessibleContextBase.cxx 
b/editeng/source/accessibility/AccessibleContextBase.cxx
index 8a11762..895f655 100644
--- a/editeng/source/accessibility/AccessibleContextBase.cxx
+++ b/editeng/source/accessibility/AccessibleContextBase.cxx
@@ -33,7 +33,6 @@
 
 #include utility
 
-using namespace ::rtl;
 using namespace ::com::sun::star;
 using namespace ::com::sun::star::accessibility;
 using ::com::sun::star::uno::Reference;
@@ -231,7 +230,7 @@ uno::ReferenceXAccessible SAL_CALL
 {
 ThrowIfDisposed ();
 throw lang::IndexOutOfBoundsException (
-::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(no child with index ) + 
nIndex),
+no child with index  + OUString(nIndex),
 NULL);
 }
 
@@ -296,7 +295,7 @@ sal_Int16 SAL_CALL
 
 
 
-::rtl::OUString SAL_CALL
+OUString SAL_CALL
AccessibleContextBase::getAccessibleDescription (void)
 throw (::com::sun::star::uno::RuntimeException)
 {
@@ -423,8 +422,7 @@ lang::Locale SAL_CALL
 
 //=  XAccessibleEventListener  
 
-void SAL_CALL
-AccessibleContextBase::addAccessibleEventListener (
+void SAL_CALL AccessibleContextBase::addAccessibleEventListener (
 const uno::ReferenceXAccessibleEventListener  rxListener)
 throw (uno::RuntimeException)
 {
@@ -447,8 +445,7 @@ void SAL_CALL
 
 
 
-void SAL_CALL
-AccessibleContextBase::removeAccessibleEventListener (
+void SAL_CALL AccessibleContextBase::removeAccessibleEventListener (
 const uno::ReferenceXAccessibleEventListener  rxListener )
 throw (uno::RuntimeException)
 {
@@ -473,8 +470,7 @@ void SAL_CALL
 
 //=  XServiceInfo  
 
-::rtl::OUString SAL_CALL
-   AccessibleContextBase::getImplementationName (void)
+OUString SAL_CALL AccessibleContextBase::getImplementationName (void)
 throw (::com::sun::star::uno::RuntimeException)
 {
 ThrowIfDisposed ();
@@ -491,7 +487,7 @@ sal_Bool SAL_CALL
 ThrowIfDisposed ();
 //  Iterate over all supported service names and return true if on of them
 //  matches the given name.
-uno::Sequence ::rtl::OUString aSupportedServices (
+uno::Sequence OUString  aSupportedServices (
 getSupportedServiceNames ());
 for (int i=0; iaSupportedServices.getLength(); i++)
 if (sServiceName == aSupportedServices[i])
@@ -502,16 +498,14 @@ sal_Bool SAL_CALL
 
 
 
-uno::Sequence ::rtl::OUString SAL_CALL
+uno::Sequence OUString  SAL_CALL
AccessibleContextBase::getSupportedServiceNames (void)
 throw (::com::sun::star::uno::RuntimeException)
 {
 ThrowIfDisposed ();
 static const OUString sServiceNames[2] = {
-OUString(
-com.sun.star.accessibility.Accessible),
-OUString(
-com.sun.star.accessibility.AccessibleContext)
+com.sun.star.accessibility.Accessible,
+com.sun.star.accessibility.AccessibleContext
 };
 return uno::SequenceOUString (sServiceNames, 2);
 }
@@ -564,7 +558,7 @@ void SAL_CALL AccessibleContextBase::disposing (void)
 
 
 void 

[Libreoffice-commits] .: editeng/source

2012-12-24 Thread Libreoffice Gerrit user
 editeng/source/accessibility/AccessibleParaManager.cxx|2 -
 editeng/source/accessibility/AccessibleSelectionBase.cxx  |1 
 editeng/source/accessibility/AccessibleStaticTextBase.cxx |   25 ++
 3 files changed, 13 insertions(+), 15 deletions(-)

New commits:
commit cd546700457c61b209deae7a4b75b5932127c1d0
Author: Olivier Hallot olivier.hal...@alta.org.br
Date:   Fri Dec 21 23:19:35 2012 -0200

OUString cleanup in editeng/acessibility

Change-Id: I5f8b73aab1e67a5ac071427574fa943a844fbf96
Reviewed-on: https://gerrit.libreoffice.org/1469
Reviewed-by: Thomas Arnhold tho...@arnhold.org
Tested-by: Thomas Arnhold tho...@arnhold.org
Reviewed-by: Norbert Thiebaud nthieb...@gmail.com
Tested-by: Norbert Thiebaud nthieb...@gmail.com

diff --git a/editeng/source/accessibility/AccessibleParaManager.cxx 
b/editeng/source/accessibility/AccessibleParaManager.cxx
index f215062..c3c24d6 100644
--- a/editeng/source/accessibility/AccessibleParaManager.cxx
+++ b/editeng/source/accessibility/AccessibleParaManager.cxx
@@ -185,7 +185,7 @@ namespace accessibility
 uno::Reference XAccessible  xChild( static_cast 
::cppu::OWeakObject*  (pChild), uno::UNO_QUERY );
 
 if( !xChild.is() )
-throw 
uno::RuntimeException(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(Child 
creation failed)), xFrontEnd);
+throw uno::RuntimeException(Child creation failed, 
xFrontEnd);
 
 aChild = WeakPara::HardRefType( xChild, pChild );
 
diff --git a/editeng/source/accessibility/AccessibleSelectionBase.cxx 
b/editeng/source/accessibility/AccessibleSelectionBase.cxx
index 0a97ebd..4f9936e 100644
--- a/editeng/source/accessibility/AccessibleSelectionBase.cxx
+++ b/editeng/source/accessibility/AccessibleSelectionBase.cxx
@@ -20,7 +20,6 @@
 
 #include editeng/AccessibleSelectionBase.hxx
 
-using namespace ::rtl;
 using namespace ::com::sun::star;
 using namespace ::com::sun::star::accessibility;
 
diff --git a/editeng/source/accessibility/AccessibleStaticTextBase.cxx 
b/editeng/source/accessibility/AccessibleStaticTextBase.cxx
index 6843dfe..7a5f57a 100644
--- a/editeng/source/accessibility/AccessibleStaticTextBase.cxx
+++ b/editeng/source/accessibility/AccessibleStaticTextBase.cxx
@@ -300,8 +300,7 @@ namespace accessibility
 DBG_CHKTHIS( AccessibleStaticTextBase_Impl, NULL );
 
 if( !mpTextParagraph )
-throw lang::DisposedException (
-::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(object has been 
already disposed)), mxThis );
+throw lang::DisposedException (object has been already disposed, 
mxThis );
 
 // TODO: Have a differnt method on AccessibleEditableTextPara
 // that does not care about state changes
@@ -353,7 +352,7 @@ namespace accessibility
 DBG_CHKTHIS( AccessibleStaticTextBase_Impl, NULL );
 
 if( nFlatIndex  0 )
-throw 
lang::IndexOutOfBoundsException(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(AccessibleStaticTextBase_Impl::Index2Internal:
 character index out of bounds)),
+throw 
lang::IndexOutOfBoundsException(AccessibleStaticTextBase_Impl::Index2Internal: 
character index out of bounds,
   mxThis);
 // gratuitously accepting larger indices here, 
AccessibleEditableTextPara will throw eventually
 
@@ -386,7 +385,7 @@ namespace accessibility
 }
 
 // not found? Out of bounds
-throw 
lang::IndexOutOfBoundsException(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(AccessibleStaticTextBase_Impl::Index2Internal:
 character index out of bounds)),
+throw 
lang::IndexOutOfBoundsException(AccessibleStaticTextBase_Impl::Index2Internal: 
character index out of bounds,
   mxThis);
 }
 
@@ -643,7 +642,7 @@ namespace accessibility
 return mpImpl-GetParagraph( aPos.nPara ).getCharacter( aPos.nIndex );
 }
 
-uno::Sequence beans::PropertyValue  SAL_CALL 
AccessibleStaticTextBase::getCharacterAttributes( sal_Int32 nIndex, const 
::com::sun::star::uno::Sequence ::rtl::OUString  aRequestedAttributes ) 
throw (lang::IndexOutOfBoundsException, uno::RuntimeException)
+uno::Sequence beans::PropertyValue  SAL_CALL 
AccessibleStaticTextBase::getCharacterAttributes( sal_Int32 nIndex, const 
::com::sun::star::uno::Sequence OUString  aRequestedAttributes ) throw 
(lang::IndexOutOfBoundsException, uno::RuntimeException)
 {
 SolarMutexGuard aGuard;
 
@@ -709,7 +708,7 @@ namespace accessibility
 return -1;
 }
 
-::rtl::OUString SAL_CALL AccessibleStaticTextBase::getSelectedText() throw 
(uno::RuntimeException)
+OUString SAL_CALL AccessibleStaticTextBase::getSelectedText() throw 
(uno::RuntimeException)
 {
 SolarMutexGuard aGuard;
 
@@ -718,7 +717,7 @@ namespace accessibility
 
 // #104481# Return the empty string for 'no 

[Libreoffice-commits] .: editeng/source

2012-12-24 Thread Libreoffice Gerrit user
 editeng/source/accessibility/AccessibleImageBullet.cxx |   38 -
 1 file changed, 19 insertions(+), 19 deletions(-)

New commits:
commit 071b7eea798acfbe765c462c5d5b9a45abd13f63
Author: Olivier Hallot olivier.hal...@alta.org.br
Date:   Sat Dec 22 18:00:14 2012 -0200

OUString cleanup in editeng/acessibility

Change-Id: I253444e2859685cae88aa65fae755af2674bd4b4
Reviewed-on: https://gerrit.libreoffice.org/1471
Reviewed-by: Norbert Thiebaud nthieb...@gmail.com
Tested-by: Norbert Thiebaud nthieb...@gmail.com

diff --git a/editeng/source/accessibility/AccessibleImageBullet.cxx 
b/editeng/source/accessibility/AccessibleImageBullet.cxx
index 8d97477..177ccef 100644
--- a/editeng/source/accessibility/AccessibleImageBullet.cxx
+++ b/editeng/source/accessibility/AccessibleImageBullet.cxx
@@ -127,7 +127,7 @@ namespace accessibility
 DBG_CHKTHIS( AccessibleImageBullet, NULL );
 (void)i;
 
-throw 
lang::IndexOutOfBoundsException(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(No 
children available)),
+throw lang::IndexOutOfBoundsException(No children available,
   uno::Reference uno::XInterface 
   ( static_cast 
::cppu::OWeakObject*  (this) ) ); // static_cast: disambiguate hierarchy
 }
@@ -153,7 +153,7 @@ namespace accessibility
 return AccessibleRole::GRAPHIC;
 }
 
-::rtl::OUString SAL_CALL  
AccessibleImageBullet::getAccessibleDescription() throw (uno::RuntimeException)
+OUString SAL_CALL  AccessibleImageBullet::getAccessibleDescription() throw 
(uno::RuntimeException)
 {
 DBG_CHKTHIS( AccessibleImageBullet, NULL );
 
@@ -163,7 +163,7 @@ namespace accessibility
 return EE_RESSTR(RID_SVXSTR_A11Y_IMAGEBULLET_DESCRIPTION);
 }
 
-::rtl::OUString SAL_CALL  AccessibleImageBullet::getAccessibleName() throw 
(uno::RuntimeException)
+OUString SAL_CALL  AccessibleImageBullet::getAccessibleName() throw 
(uno::RuntimeException)
 {
 DBG_CHKTHIS( AccessibleImageBullet, NULL );
 
@@ -321,7 +321,7 @@ namespace accessibility
 }
 }
 
-throw 
uno::RuntimeException(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(Cannot 
access parent)),
+throw uno::RuntimeException(Cannot access parent,
 uno::Reference uno::XInterface 
 ( static_cast XAccessible*  (this) ) );  
 // disambiguate hierarchy
 }
@@ -341,7 +341,7 @@ namespace accessibility
 {
 DBG_CHKTHIS( AccessibleImageBullet, NULL );
 
-throw 
uno::RuntimeException(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(Not 
focusable)),
+throw uno::RuntimeException(Not focusable,
 uno::Reference uno::XInterface 
 ( static_cast XAccessible*  (this) ) );  
 // disambiguate hierarchy
 }
@@ -369,20 +369,20 @@ namespace accessibility
 return static_castsal_Int32( aColor.GetColor() );
 }
 
-::rtl::OUString SAL_CALL AccessibleImageBullet::getImplementationName 
(void) throw (uno::RuntimeException)
+OUString SAL_CALL AccessibleImageBullet::getImplementationName (void) 
throw (uno::RuntimeException)
 {
 DBG_CHKTHIS( AccessibleImageBullet, NULL );
 
-return ::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM 
(AccessibleImageBullet));
+return OUString(AccessibleImageBullet);
 }
 
-sal_Bool SAL_CALL AccessibleImageBullet::supportsService (const 
::rtl::OUString sServiceName) throw (uno::RuntimeException)
+sal_Bool SAL_CALL AccessibleImageBullet::supportsService (const OUString 
sServiceName) throw (uno::RuntimeException)
 {
 DBG_CHKTHIS( AccessibleImageBullet, NULL );
 
 //  Iterate over all supported service names and return true if on of 
them
 //  matches the given name.
-uno::Sequence ::rtl::OUString aSupportedServices (
+uno::Sequence OUString  aSupportedServices (
 getSupportedServiceNames ());
 for (int i=0; iaSupportedServices.getLength(); i++)
 if (sServiceName == aSupportedServices[i])
@@ -390,19 +390,19 @@ namespace accessibility
 return sal_False;
 }
 
-uno::Sequence ::rtl::OUString SAL_CALL 
AccessibleImageBullet::getSupportedServiceNames (void) throw 
(uno::RuntimeException)
+uno::Sequence OUString  SAL_CALL 
AccessibleImageBullet::getSupportedServiceNames (void) throw 
(uno::RuntimeException)
 {
 DBG_CHKTHIS( AccessibleImageBullet, NULL );
 
-const ::rtl::OUString sServiceName (RTL_CONSTASCII_USTRINGPARAM 
(com.sun.star.accessibility.AccessibleContext));
-return uno::Sequence ::rtl::OUString  (sServiceName, 1);
+const OUString sServiceName 
(com.sun.star.accessibility.AccessibleContext);
+return uno::Sequence OUString  (sServiceName, 1);
 }
 
-::rtl::OUString SAL_CALL 

[Libreoffice-commits] .: editeng/source

2012-12-21 Thread Libreoffice Gerrit user
 editeng/source/editeng/editview.cxx |2 -
 editeng/source/items/borderline.cxx |   40 
 editeng/source/items/frmitems.cxx   |1 
 3 files changed, 1 insertion(+), 42 deletions(-)

New commits:
commit 8e25844811ebcf313f4ae9d0caae1f97fd230a7d
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sat Dec 22 08:27:57 2012 +0100

Some cppcheck cleaning in editeng

Change-Id: I94bf89867cec9cc24f8b96635c515d9477b561a9

diff --git a/editeng/source/editeng/editview.cxx 
b/editeng/source/editeng/editview.cxx
index 461a1f7..7d07aea 100644
--- a/editeng/source/editeng/editview.cxx
+++ b/editeng/source/editeng/editview.cxx
@@ -998,9 +998,9 @@ void EditView::ExecuteSpellPopup( const Point rPosPixel, 
Link* pCallBack )
 
 Reference XDictionaryList   xDicList( SvxGetDictionaryList() );
 Sequence Reference XDictionaryaDics;
-const Reference XDictionary   *pDic = NULL;
 if (xDicList.is())
 {
+const Reference XDictionary   *pDic = NULL;
 // add the default positive dictionary to dic-list (if not already 
done).
 // This is to ensure that there is at least one dictionary to which
 // words could be added.
diff --git a/editeng/source/items/borderline.cxx 
b/editeng/source/items/borderline.cxx
index 646a348..9e1d8f6 100644
--- a/editeng/source/items/borderline.cxx
+++ b/editeng/source/items/borderline.cxx
@@ -123,62 +123,45 @@ ConvertBorderStyleFromWord(int const nWordLineStyle)
 case  9:
 case 20:
 return SOLID;
-break;
 case  6:
 return DOTTED;
-break;
 case  7:
 return DASHED;
-break;
 case 22:
 return FINE_DASHED;
-break;
 // then the shading beams which we represent by a double line
 case 23:
 return DOUBLE;
-break;
 // then the double lines, for which we have good matches
 case  3:
 case 10: // Don't have triple so use double
 case 21: // Don't have double wave: use double instead
 return DOUBLE;
-break;
 case 11:
 return THINTHICK_SMALLGAP;
-break;
 case 12:
 case 13: // Don't have thin thick thin, so use thick thin
 return THICKTHIN_SMALLGAP;
-break;
 case 14:
 return THINTHICK_MEDIUMGAP;
-break;
 case 15:
 case 16: // Don't have thin thick thin, so use thick thin
 return THICKTHIN_MEDIUMGAP;
-break;
 case 17:
 return THINTHICK_LARGEGAP;
-break;
 case 18:
 case 19: // Don't have thin thick thin, so use thick thin
 return THICKTHIN_LARGEGAP;
-break;
 case 24:
 return EMBOSSED;
-break;
 case 25:
 return ENGRAVED;
-break;
 case 26:
 return OUTSET;
-break;
 case 27:
 return INSET;
-break;
 default:
 return NONE;
-break;
 }
 }
 
@@ -205,13 +188,10 @@ ConvertBorderWidthFromWord(SvxBorderStyle const eStyle, 
double const fWidth,
 {
 case 2:
 return (fWidth * 2.0); // thick
-break;
 case 5: // fdo#55526: map 0 hairline width to  0
 return (fWidth  1.0) ? fWidth : 1.0;
-break;
 default:
 return fWidth;
-break;
 }
 break;
 
@@ -222,48 +202,38 @@ ConvertBorderWidthFromWord(SvxBorderStyle const eStyle, 
double const fWidth,
 // Display a minimum effective border width of 1pt
 case FINE_DASHED:
 return (fWidth  0  fWidth  20) ? 20 : fWidth;
-break;
 
 // Double lines
 case DOUBLE:
 return fWidth * 3.0;
-break;
 
 case THINTHICK_MEDIUMGAP:
 case THICKTHIN_MEDIUMGAP:
 case EMBOSSED:
 case ENGRAVED:
 return fWidth * 2.0;
-break;
 
 case THINTHICK_SMALLGAP:
 return fWidth + THINTHICK_SMALLGAP_line2 + THINTHICK_SMALLGAP_gap;
-break;
 
 case THINTHICK_LARGEGAP:
 return fWidth + THINTHICK_LARGEGAP_line1 + 
THINTHICK_LARGEGAP_line2;
-break;
 
 case THICKTHIN_SMALLGAP:
 return fWidth + THICKTHIN_SMALLGAP_line1 + THICKTHIN_SMALLGAP_gap;
-break;
 
 case THICKTHIN_LARGEGAP:
 return fWidth + THICKTHIN_LARGEGAP_line1 + 
THICKTHIN_LARGEGAP_line2;
-break;
 
 case OUTSET:
 return (fWidth * 2.0) + OUTSET_line1;
-break;
 
 case INSET:
 return (fWidth * 2.0) + INSET_line2;
-break;
 

[Libreoffice-commits] .: editeng/source

2012-12-12 Thread Libreoffice Gerrit user
 editeng/source/misc/svxacorr.cxx |4 
 1 file changed, 4 insertions(+)

New commits:
commit a61928d2f7512d573e598b23c7fd3cf341f97780
Author: Michael Meeks michael.me...@suse.com
Date:   Wed Dec 12 22:16:53 2012 +

fdo#58060 - use empty-string for LANGUAGE_DONTKNOW ie. acorr_.dat

Change-Id: I399dcc41484c8b86f13c80bc1e92af919c9aeaff

diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index 0d8ff52..b4fa5d0 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -1870,6 +1870,10 @@ String SvxAutoCorrect::GetAutoCorrFileName( LanguageType 
eLang,
 sal_Bool bNewFile, sal_Bool bTst ) 
const
 {
 String sRet, sExt( LanguageTag( eLang ).getBcp47() );
+
+// fdo#58060 user added dictionary - saved as acorr_.dat
+if (eLang == LANGUAGE_DONTKNOW)
+sExt = String();
 sExt.Insert('_', 0);
 sExt.AppendAscii( .dat );
 if( bNewFile )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-12-11 Thread Libreoffice Gerrit user
 editeng/source/misc/svxacorr.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 0efcfcdf40661023d8b516d6461d164e4ab7
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Dec 11 09:12:03 2012 +

cppcheck: size() == 0 - empty()

Change-Id: I23b6b8fdbb9c4f367a590a21e40e7bbaa76f7fce

diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index aaee953..4bba278 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -2669,7 +2669,7 @@ void SvxAutocorrWordList::DeleteAndDestroyAll()
 // returns true if inserted
 bool SvxAutocorrWordList::Insert(SvxAutocorrWord *pWord)
 {
-if ( maSet.size() == 0 ) // use the hash
+if ( maSet.empty() ) // use the hash
 {
 rtl::OUString aShort( pWord-GetShort() );
 bool bThere = maHash.find( aShort ) != maHash.end();
@@ -2697,7 +2697,7 @@ SvxAutocorrWord 
*SvxAutocorrWordList::FindAndRemove(SvxAutocorrWord *pWord)
 {
 SvxAutocorrWord *pMatch = NULL;
 
-if ( maSet.size() == 0 ) // use the hash
+if ( maSet.empty() ) // use the hash
 {
 SvxAutocorrWordList_Hash::iterator it = maHash.find( pWord-GetShort() 
);
 if( it != maHash.end() )
@@ -2724,7 +2724,7 @@ SvxAutocorrWordList::Content 
SvxAutocorrWordList::getSortedContent() const
 Content aContent;
 
 // convert from hash to set permanantly
-if ( maSet.size() == 0 )
+if ( maSet.empty() )
 {
 // This beasty has some O(N log(N)) in a terribly slow ICU collate fn.
 for( SvxAutocorrWordList_Hash::const_iterator it = maHash.begin(); it 
!= maHash.end(); ++it )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source offapi/com svtools/inc svtools/source

2012-12-04 Thread Libreoffice Gerrit user
 editeng/source/items/borderline.cxx   |   11 ++-
 editeng/source/items/frmitems.cxx |2 +-
 offapi/com/sun/star/table/BorderLineStyle.idl |8 
 svtools/inc/svtools/ctrlbox.hxx   |1 +
 svtools/source/control/ctrlbox.cxx|7 +++
 5 files changed, 27 insertions(+), 2 deletions(-)

New commits:
commit 7d632ff29e601c2e680c4a689997fbf552592a4b
Author: Eilidh McAdam eil...@lanedo.com
Date:   Thu Nov 15 00:29:58 2012 +

Support added for fine dashing on table borders.

When reading in docx documents, fine dashing provides a better visual
match for some border types.
Added in this patch:
 - FINE_DASHED in BorderLineStyle UNO enum and in internals
 - Import of docx table borders using this border style

Change-Id: I39cfa18c915ec94d8e4ecfc6a2ca637076d1e468
Reviewed-on: https://gerrit.libreoffice.org/1123
Reviewed-by: Fridrich Strba fridr...@documentfoundation.org
Tested-by: Fridrich Strba fridr...@documentfoundation.org

diff --git a/editeng/source/items/borderline.cxx 
b/editeng/source/items/borderline.cxx
index 1904916..646a348 100644
--- a/editeng/source/items/borderline.cxx
+++ b/editeng/source/items/borderline.cxx
@@ -128,9 +128,11 @@ ConvertBorderStyleFromWord(int const nWordLineStyle)
 return DOTTED;
 break;
 case  7:
-case 22:
 return DASHED;
 break;
+case 22:
+return FINE_DASHED;
+break;
 // then the shading beams which we represent by a double line
 case 23:
 return DOUBLE;
@@ -216,6 +218,10 @@ ConvertBorderWidthFromWord(SvxBorderStyle const eStyle, 
double const fWidth,
 case DOTTED:
 case DASHED:
 return fWidth;
+
+// Display a minimum effective border width of 1pt
+case FINE_DASHED:
+return (fWidth  0  fWidth  20) ? 20 : fWidth;
 break;
 
 // Double lines
@@ -270,6 +276,7 @@ ConvertBorderWidthToWord(SvxBorderStyle const eStyle, 
double const fWidth)
 case SOLID:
 case DOTTED:
 case DASHED:
+case FINE_DASHED:
 return fWidth;
 break;
 
@@ -335,6 +342,7 @@ BorderWidthImpl SvxBorderLine::getWidthImpl( SvxBorderStyle 
nStyle )
 case SOLID:
 case DOTTED:
 case DASHED:
+case FINE_DASHED:
 aImpl = BorderWidthImpl( CHANGE_LINE1, 1.0 );
 break;
 
@@ -514,6 +522,7 @@ void SvxBorderLine::GuessLinesWidths( SvxBorderStyle 
nStyle, sal_uInt16 nOut, sa
 case SOLID:
 case DOTTED:
 case DASHED:
+case FINE_DASHED:
 ::std::swap( nOut, nIn);
 break;
 default:
diff --git a/editeng/source/items/frmitems.cxx 
b/editeng/source/items/frmitems.cxx
index 6f410c6..5ee12a8 100644
--- a/editeng/source/items/frmitems.cxx
+++ b/editeng/source/items/frmitems.cxx
@@ -1794,7 +1794,7 @@ sal_Bool
 SvxBoxItem::LineToSvxLine(const ::com::sun::star::table::BorderLine2 rLine, 
SvxBorderLine rSvxLine, sal_Bool bConvert)
 {
 SvxBorderStyle const nStyle =
-(rLine.LineStyle  0 || INSET  rLine.LineStyle)
+(rLine.LineStyle  0 || BORDER_LINE_STYLE_MAX  rLine.LineStyle)
 ? SOLID // default
 : rLine.LineStyle;
 
diff --git a/offapi/com/sun/star/table/BorderLineStyle.idl 
b/offapi/com/sun/star/table/BorderLineStyle.idl
index 0007118..7613272 100644
--- a/offapi/com/sun/star/table/BorderLineStyle.idl
+++ b/offapi/com/sun/star/table/BorderLineStyle.idl
@@ -90,6 +90,14 @@ constants BorderLineStyle
 /** Inset border line.
  */
 const short INSET = 13;
+
+/** Finely dashed border line.
+ */
+const short FINE_DASHED = 14;
+
+/** Maximum valid border line style value.
+ */
+const short BORDER_LINE_STYLE_MAX = 14;
 };
 
 
diff --git a/svtools/inc/svtools/ctrlbox.hxx b/svtools/inc/svtools/ctrlbox.hxx
index a6b26b8..2d6e6a2 100644
--- a/svtools/inc/svtools/ctrlbox.hxx
+++ b/svtools/inc/svtools/ctrlbox.hxx
@@ -44,6 +44,7 @@ typedef ::std::vector FontInfo   ImplFontList;
 #define STYLE_SOLID( ( sal_uInt16 ) 0 )
 #define STYLE_DOTTED   ( ( sal_uInt16 ) 1 )
 #define STYLE_DASHED   ( ( sal_uInt16 ) 2 )
+#define STYLE_FINE_DASHED  ( ( sal_uInt16 ) 14 )
 #define STYLE_NONE ( ( sal_uInt16 ) -1)
 
 #define CHANGE_LINE1   ( ( sal_uInt16 ) 1 )
diff --git a/svtools/source/control/ctrlbox.cxx 
b/svtools/source/control/ctrlbox.cxx
index 017422f..dd739d4 100644
--- a/svtools/source/control/ctrlbox.cxx
+++ b/svtools/source/control/ctrlbox.cxx
@@ -615,6 +615,13 @@ namespace svtools
 aPattern.push_back( 20 );
 }
 break;
+case STYLE_FINE_DASHED:
+if ( eUnit == MAP_PIXEL )
+{
+  

[Libreoffice-commits] .: editeng/source

2012-11-27 Thread Libreoffice Gerrit user
 editeng/source/items/textitem.cxx |5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

New commits:
commit cb523b62a4a63b70054d7a16385faf5f3a76be28
Author: Eike Rathke er...@redhat.com
Date:   Tue Nov 27 21:42:32 2012 +0100

SvxLanguageItem::PutValue: empty locale is not LANGUAGE_NONE

Change-Id: I8d788f53b54a7429124a653417d422db300238d7

diff --git a/editeng/source/items/textitem.cxx 
b/editeng/source/items/textitem.cxx
index 25f998b..d5846a0 100644
--- a/editeng/source/items/textitem.cxx
+++ b/editeng/source/items/textitem.cxx
@@ -2634,10 +2634,7 @@ bool SvxLanguageItem::PutValue( const uno::Any rVal, 
sal_uInt8 nMemberId )
 if(!(rVal = aLocale))
 return sal_False;
 
-if (!aLocale.Language.isEmpty() || !aLocale.Country.isEmpty())
-SetValue( LanguageTag( aLocale ).getLanguageType( false));
-else
-SetValue(LANGUAGE_NONE);
+SetValue( LanguageTag( aLocale ).getLanguageType( false));
 }
 break;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source sc/source

2012-11-24 Thread Libreoffice Gerrit user
 editeng/source/rtf/rtfitem.cxx |6 ++--
 sc/source/core/data/attrib.cxx |   20 +++---
 sc/source/ui/view/preview.cxx  |   55 -
 3 files changed, 42 insertions(+), 39 deletions(-)

New commits:
commit fa6be97997f6142884accef5a88d6491bb1b3b76
Author: Peter Baumgarten peter10...@gmail.com
Date:   Sat Nov 24 11:30:28 2012 +

Translation of German comments

Change-Id: I729049b8fa9c26926423c6990a06377e6884e840

diff --git a/editeng/source/rtf/rtfitem.cxx b/editeng/source/rtf/rtfitem.cxx
index b1ff8ec..dde178b 100644
--- a/editeng/source/rtf/rtfitem.cxx
+++ b/editeng/source/rtf/rtfitem.cxx
@@ -220,7 +220,8 @@ void SvxRTFParser::ReadAttr( int nToken, SfxItemSet* pSet )
 
 int bChkStkPos = !bNewGroup  !aAttrStack.empty();
 
-while( bWeiter  IsParserWorking() )  // as long as known Attribute are 
recognized
+while( bWeiter  IsParserWorking() )  // as long as known 
+Attribute are recognized
 {
 switch( nToken )
 {
@@ -293,7 +294,8 @@ void SvxRTFParser::ReadAttr( int nToken, SfxItemSet* pSet )
 else
 {
 nStyleNo = -1 == nTokenValue ? 0 : sal_uInt16(nTokenValue);
-// setze am akt. auf dem AttrStack stehenden Style die
+/* setze am akt. auf dem AttrStack stehenden Style die
+   I sit on akt. which is on the immiediate sytle 
AttrStack */
 // StyleNummer
 SvxRTFItemStackType* pAkt = aAttrStack.empty() ? 0 : 
aAttrStack.back();
 if( !pAkt )
diff --git a/sc/source/core/data/attrib.cxx b/sc/source/core/data/attrib.cxx
index d666229..310aa1d 100644
--- a/sc/source/core/data/attrib.cxx
+++ b/sc/source/core/data/attrib.cxx
@@ -65,7 +65,7 @@ TYPEINIT1(ScCondFormatItem,SfxPoolItem);
 //
 
 //
-//  allgemeine Hilfsfunktionen
+//  General Help Function
 //
 
 bool ScHasPriority( const ::editeng::SvxBorderLine* pThis, const 
::editeng::SvxBorderLine* pOther )
@@ -493,7 +493,7 @@ SfxItemPresentation ScRangeItem::GetPresentation
 }
 
 // ---
-//  ScTableListItem - Liste von Tabellen(-nummern)
+//  ScTableListItem - List from Tables (-numbers)
 // ---
 
 ScTableListItem::ScTableListItem( const ScTableListItem rCpy )
@@ -618,7 +618,7 @@ SfxItemPresentation ScTableListItem::GetPresentation
 
 
 // ---
-//  ScPageHFItem - Daten der Kopf-/Fusszeilen
+//  ScPageHFItem - Dates from the Head and Foot lines
 // ---
 
 ScPageHFItem::ScPageHFItem( sal_uInt16 nWhichP )
@@ -691,7 +691,7 @@ bool ScPageHFItem::PutValue( const uno::Any rVal, 
sal_uInt8 /* nMemberId */ )
 
 if ( !pLeftArea || !pCenterArea || !pRightArea )
 {
-// keine Texte auf NULL stehen lassen
+// no Text with Null are left
 ScEditEngineDefaulter aEngine( EditEngine::CreatePool(), 
true );
 if (!pLeftArea)
 pLeftArea = aEngine.CreateTextObject();
@@ -745,7 +745,7 @@ SfxPoolItem* ScPageHFItem::Clone( SfxItemPool* ) const
 
 static void lcl_SetSpace( String rStr, const ESelection rSel )
 {
-// Text durch ein Leerzeichen ersetzen, damit Positionen stimmen:
+// Text replaced by a space to ensure they are positions:
 
 xub_StrLen nLen = rSel.nEndPos-rSel.nStartPos;
 rStr.Erase( rSel.nStartPos, nLen-1 );
@@ -837,7 +837,7 @@ SfxPoolItem* ScPageHFItem::Create( SvStream rStream, 
sal_uInt16 nVer ) const
 }
 }
 
-if ( nVer  1 ) // alte Feldbefehle umsetzen
+if ( nVer  1 ) //old field command conversions
 {
 sal_uInt16 i;
 const String rDel = ScGlobal::GetRscString( STR_HFCMD_DELIMITER );
@@ -874,7 +874,7 @@ SfxPoolItem* ScPageHFItem::Create( SvStream rStream, 
sal_uInt16 nVer ) const
 }
 }
 else if ( nVer  2 )
-{   // nichts tun, SvxFileField nicht gegen SvxExtFileField austauschen
+{   // not to do, SvxFileField is not exchanged for SvxExtFileField
 }
 
 ScPageHFItem* pItem = new ScPageHFItem( Which() );
@@ -923,7 +923,7 @@ void ScPageHFItem::SetArea( EditTextObject *pNew, int nArea 
)
 }
 
 //---
-//  ScViewObjectModeItem - Darstellungsmodus von ViewObjekten
+//  ScViewObjectModeItem - Display Mode of View Objects
 //---
 
 ScViewObjectModeItem::ScViewObjectModeItem( sal_uInt16 nWhichP )
@@ -979,7 +979,7 @@ SfxItemPresentation ScViewObjectModeItem::GetPresentation
 

[Libreoffice-commits] .: editeng/source

2012-11-24 Thread Libreoffice Gerrit user
 editeng/source/rtf/rtfitem.cxx |3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit b82ecd0448ca97df79243b2d22547324d08dc130
Author: Ivan Timofeev timofeev@gmail.com
Date:   Sat Nov 24 18:52:06 2012 +0400

fix build

Change-Id: Iebbb690f3beacca026d584636b9ef505dfe9ac89

diff --git a/editeng/source/rtf/rtfitem.cxx b/editeng/source/rtf/rtfitem.cxx
index dde178b..a8b100c 100644
--- a/editeng/source/rtf/rtfitem.cxx
+++ b/editeng/source/rtf/rtfitem.cxx
@@ -220,8 +220,7 @@ void SvxRTFParser::ReadAttr( int nToken, SfxItemSet* pSet )
 
 int bChkStkPos = !bNewGroup  !aAttrStack.empty();
 
-while( bWeiter  IsParserWorking() )  // as long as known 
-Attribute are recognized
+while( bWeiter  IsParserWorking() )  // as long as known Attribute are 
recognized
 {
 switch( nToken )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source sw/source

2012-11-24 Thread Libreoffice Gerrit user
 editeng/source/editeng/editview.cxx |8 
 editeng/source/editeng/impedit4.cxx |4 ++--
 editeng/source/misc/unolingu.cxx|2 +-
 sw/source/ui/lingu/olmenu.cxx   |8 
 4 files changed, 11 insertions(+), 11 deletions(-)

New commits:
commit 117184c3e14bb01cc4482bb2a47569008358e6fc
Author: Eike Rathke er...@redhat.com
Date:   Sat Nov 24 19:03:56 2012 +0100

kill a few more SvxCreateLocale()  Co remainders

Change-Id: I2b70bf76a495b9edb79b5b3c4ae1b06abed30f54

diff --git a/editeng/source/editeng/editview.cxx 
b/editeng/source/editeng/editview.cxx
index d40ac27..461a1f7 100644
--- a/editeng/source/editeng/editview.cxx
+++ b/editeng/source/editeng/editview.cxx
@@ -130,10 +130,10 @@ static LanguageType lcl_CheckLanguage(
 // en-US
 aLangList[3] = LANGUAGE_ENGLISH_US;
 #ifdef DEBUG
-lang::Locale a0( SvxCreateLocale( aLangList[0] ) );
-lang::Locale a1( SvxCreateLocale( aLangList[1] ) );
-lang::Locale a2( SvxCreateLocale( aLangList[2] ) );
-lang::Locale a3( SvxCreateLocale( aLangList[3] ) );
+lang::Locale a0( LanguageTag( aLangList[0] ).getLocale() );
+lang::Locale a1( LanguageTag( aLangList[1] ).getLocale() );
+lang::Locale a2( LanguageTag( aLangList[2] ).getLocale() );
+lang::Locale a3( LanguageTag( aLangList[3] ).getLocale() );
 #endif
 
 sal_Int32 nCount = sizeof (aLangList) / sizeof (aLangList[0]);
diff --git a/editeng/source/editeng/impedit4.cxx 
b/editeng/source/editeng/impedit4.cxx
index d885d8d..d66e1b5 100644
--- a/editeng/source/editeng/impedit4.cxx
+++ b/editeng/source/editeng/impedit4.cxx
@@ -1500,7 +1500,7 @@ sal_Bool ImpEditEngine::HasConvertibleTextPortion( 
LanguageType nSrcLang )
 ++nStart;
 LanguageType nLangFound = pEditEngine-GetLanguage( k, nStart );
 #ifdef DEBUG
-lang::Locale aLocale( SvxCreateLocale( nLangFound ) );
+lang::Locale aLocale( LanguageTag( nLangFound ).getLocale() );
 #endif
 bHasConvTxt =   (nSrcLang == nLangFound) ||
 (editeng::HangulHanjaConversion::IsChinese( 
nLangFound ) 
@@ -1692,7 +1692,7 @@ void ImpEditEngine::ImpConvert( rtl::OUString rConvTxt, 
LanguageType rConvTxtL
 sal_uInt16 nLangIdx = nEnd  nStart ? nStart + 1 : nStart;
 LanguageType nLangFound = pEditEngine-GetLanguage( 
aCurStart.nPara, nLangIdx );
 #ifdef DEBUG
-lang::Locale aLocale( SvxCreateLocale( nLangFound ) );
+lang::Locale aLocale( LanguageTag( nLangFound ).getLocale() );
 #endif
 sal_Bool bLangOk =  (nLangFound == nSrcLang) ||
 (editeng::HangulHanjaConversion::IsChinese( 
nLangFound ) 
diff --git a/editeng/source/misc/unolingu.cxx b/editeng/source/misc/unolingu.cxx
index 5c78f07..7fa98c1 100644
--- a/editeng/source/misc/unolingu.cxx
+++ b/editeng/source/misc/unolingu.cxx
@@ -741,7 +741,7 @@ uno::Reference XDictionary  LinguMgr::GetStandard()
 uno::Reference XStorable   xStor( xDic, UNO_QUERY );
 OSL_ENSURE( xDic.is()  xDic-getDictionaryType() == 
DictionaryType_POSITIVE,
 wrong dictionary type);
-OSL_ENSURE( xDic.is()  SvxLocaleToLanguage( xDic-getLocale() ) == 
LANGUAGE_NONE,
+OSL_ENSURE( xDic.is()  LanguageTag( xDic-getLocale() 
).getLanguageType() == LANGUAGE_NONE,
 wrong dictionary language);
 OSL_ENSURE( !xStor.is() || (xStor-hasLocation()  !xStor-isReadonly()),
 dictionary not editable );
diff --git a/sw/source/ui/lingu/olmenu.cxx b/sw/source/ui/lingu/olmenu.cxx
index f5d24ca..f337aa8 100644
--- a/sw/source/ui/lingu/olmenu.cxx
+++ b/sw/source/ui/lingu/olmenu.cxx
@@ -157,10 +157,10 @@ static LanguageType lcl_CheckLanguage(
 // en-US
 aLangList[3] = LANGUAGE_ENGLISH_US;
 #if OSL_DEBUG_LEVEL  1
-lang::Locale a0( SvxCreateLocale( aLangList[0] ) );
-lang::Locale a1( SvxCreateLocale( aLangList[1] ) );
-lang::Locale a2( SvxCreateLocale( aLangList[2] ) );
-lang::Locale a3( SvxCreateLocale( aLangList[3] ) );
+lang::Locale a0( LanguageTag( aLangList[0] ).getLocale() );
+lang::Locale a1( LanguageTag( aLangList[1] ).getLocale() );
+lang::Locale a2( LanguageTag( aLangList[2] ).getLocale() );
+lang::Locale a3( LanguageTag( aLangList[3] ).getLocale() );
 #endif
 
 sal_Int32 nCount = SAL_N_ELEMENTS(aLangList);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-11-16 Thread Libreoffice Gerrit user
 editeng/source/editeng/editeng.cxx  |2 +-
 editeng/source/editeng/editview.cxx |4 ++--
 editeng/source/items/textitem.cxx   |6 +++---
 editeng/source/misc/svxacorr.cxx|2 +-
 editeng/source/misc/unolingu.cxx|   10 +-
 editeng/source/outliner/outlvw.cxx  |4 ++--
 6 files changed, 14 insertions(+), 14 deletions(-)

New commits:
commit 378e437fbe313e87b7e56f8f0a1fc4009470679c
Author: Eike Rathke er...@redhat.com
Date:   Fri Nov 16 23:22:11 2012 +0100

use LanguageTag

Change-Id: If056193c803f70f8707373ed7ff7b1abbf953852

diff --git a/editeng/source/editeng/editeng.cxx 
b/editeng/source/editeng/editeng.cxx
index f437564..8b191ea 100644
--- a/editeng/source/editeng/editeng.cxx
+++ b/editeng/source/editeng/editeng.cxx
@@ -1362,7 +1362,7 @@ sal_Bool EditEngine::PostKeyEvent( const KeyEvent 
rKeyEvent, EditView* pEditVie
 String aComplete;
 
 LanguageType eLang = 
pImpEditEngine-GetLanguage( EditPaM( aStart.GetNode(), aStart.GetIndex()+1));
-lang::Locale aLocale( 
MsLangId::convertLanguageToLocale( eLang));
+lang::Locale aLocale( LanguageTag( 
eLang).getLocale());
 
 if 
(!pImpEditEngine-xLocaleDataWrapper.isInitialized())
 pImpEditEngine-xLocaleDataWrapper.init( 
SvtSysLocale().GetLocaleData().getComponentContext(), aLocale, eLang);
diff --git a/editeng/source/editeng/editview.cxx 
b/editeng/source/editeng/editview.cxx
index 1ed54fc..86084a7 100644
--- a/editeng/source/editeng/editview.cxx
+++ b/editeng/source/editeng/editview.cxx
@@ -27,7 +27,7 @@
 #include com/sun/star/i18n/WordType.hpp
 #include vcl/metric.hxx
 
-#include i18npool/mslangid.hxx
+#include i18npool/languagetag.hxx
 #include svl/languageoptions.hxx
 #include svtools/ctrltool.hxx
 #include svtools/langtab.hxx
@@ -102,7 +102,7 @@ static LanguageType lcl_CheckLanguage(
 nLang = nTmpLang;
 }
 if (nLang == LANGUAGE_NONE) // language not found by looking up the 
sytem language...
-nLang = MsLangId::convertLocaleToLanguageWithFallback( aLocale );
+nLang = LanguageTag( aLocale ).makeFallback().getLanguageType();
 if (nLang == LANGUAGE_SYSTEM)
 nLang = nTmpLang;
 if (nLang == LANGUAGE_DONTKNOW)
diff --git a/editeng/source/items/textitem.cxx 
b/editeng/source/items/textitem.cxx
index 401ca6f..25f998b 100644
--- a/editeng/source/items/textitem.cxx
+++ b/editeng/source/items/textitem.cxx
@@ -37,7 +37,7 @@
 #include tools/tenccvt.hxx
 
 #include rtl/ustring.hxx
-#include i18npool/mslangid.hxx
+#include i18npool/languagetag.hxx
 #include svl/itemset.hxx
 
 #include svtools/langtab.hxx
@@ -2607,7 +2607,7 @@ bool SvxLanguageItem::QueryValue( uno::Any rVal, 
sal_uInt8 nMemberId ) const
 rVal = (sal_Int16)(GetValue());
 break;
 case MID_LANG_LOCALE:
-lang::Locale aRet( MsLangId::convertLanguageToLocale( GetValue(), 
false));
+lang::Locale aRet( LanguageTag( GetValue()).getLocale( false));
 rVal = aRet;
 break;
 }
@@ -2635,7 +2635,7 @@ bool SvxLanguageItem::PutValue( const uno::Any rVal, 
sal_uInt8 nMemberId )
 return sal_False;
 
 if (!aLocale.Language.isEmpty() || !aLocale.Country.isEmpty())
-SetValue(MsLangId::convertLocaleToLanguage( aLocale ));
+SetValue( LanguageTag( aLocale ).getLanguageType( false));
 else
 SetValue(LANGUAGE_NONE);
 }
diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index 3f2298a..d8ae94c 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -1905,7 +1905,7 @@ sal_Bool 
SvxAutoCorrect::FindInCplSttExceptList(LanguageType eLang,
 String SvxAutoCorrect::GetAutoCorrFileName( LanguageType eLang,
 sal_Bool bNewFile, sal_Bool bTst ) 
const
 {
-String sRet, sExt( MsLangId::convertLanguageToIsoString( eLang ) );
+String sRet, sExt( LanguageTag( eLang ).getBcp47() );
 sExt.Insert('_', 0);
 sExt.AppendAscii( .dat );
 if( bNewFile )
diff --git a/editeng/source/misc/unolingu.cxx b/editeng/source/misc/unolingu.cxx
index 022ab11..4648636 100644
--- a/editeng/source/misc/unolingu.cxx
+++ b/editeng/source/misc/unolingu.cxx
@@ -40,7 +40,7 @@
 
 #include comphelper/processfactory.hxx
 #include cppuhelper/implbase1.hxx // helper for implementations
-#include i18npool/mslangid.hxx
+#include i18npool/languagetag.hxx
 #include unotools/lingucfg.hxx
 #include unotools/ucbhelper.hxx
 #include unotools/localfilehelper.hxx
@@ -131,7 +131,7 @@ void ThesDummy_Impl::GetCfgLocales()
 for (sal_Int32 i = 0;  i  nLen;  ++i)
 {
 pLocale[i] = SvxCreateLocale(
-MsLangId::convertIsoStringToLanguage( 

[Libreoffice-commits] .: editeng/source

2012-10-12 Thread Libreoffice Gerrit user
 editeng/source/editeng/impedit3.cxx |5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

New commits:
commit fa694a21b806ed7837c1337ec49a4b299c478393
Author: Jan Holesovsky ke...@suse.cz
Date:   Fri Oct 12 17:41:49 2012 +0200

fdo#55931 Fix renderding of subscript/superscript with Autofit Text.

We are using font metrics to compute the stretch ratio for autofit; but that
collides with nPropr property of SvxFont - it is then counted twice, ie. in
the case of nPropr == 25, we actually behave as if it was much less; and
worse, only in the horizontal direction.

Change-Id: Idba62f1e3f40802651b93f1344e376048866b1b6

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 5878822..2e00179 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -2651,10 +2651,7 @@ void ImpEditEngine::SeekCursor( ContentNode* pNode, 
sal_uInt16 nPos, SvxFont rF
 // ugly on the screen!
 OutputDevice* pDev = pOut ? pOut : GetRefDevice();
 rFont.SetPhysFont( pDev );
-FontMetric aMetric( pDev-GetFontMetric() );
-// For the height do not consider the metrics, because it will go
-// wrong at Superscript/Subscript.
-Size aRealSz( aMetric.GetSize().Width(), rFont.GetSize().Height() );
+Size aRealSz( rFont.GetSize().Width(), rFont.GetSize().Height() );
 if ( aStatus.DoStretch() )
 {
 if ( nStretchY != 100 )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-10-01 Thread Libreoffice Gerrit user
 editeng/source/editeng/editdoc.cxx |   15 +++
 editeng/source/editeng/editdoc.hxx |9 ++---
 2 files changed, 17 insertions(+), 7 deletions(-)

New commits:
commit f3f706274172879324884e845629ef74b2e2e443
Author: Thorsten Behrens tbehr...@suse.com
Date:   Mon Oct 1 19:33:34 2012 +0200

Outline methods with conditionals in editeng

diff --git a/editeng/source/editeng/editdoc.cxx 
b/editeng/source/editeng/editdoc.cxx
index cb13c91..560d7ca 100644
--- a/editeng/source/editeng/editdoc.cxx
+++ b/editeng/source/editeng/editdoc.cxx
@@ -1034,6 +1034,21 @@ sal_Bool operator != ( const EditLine r1,  const 
EditLine r2  )
 return !( r1 == r2 );
 }
 
+void EditLine::SetHeight( sal_uInt16 nH, sal_uInt16 nTxtH, sal_uInt16 nCrsrH )
+{
+nHeight = nH;
+nTxtHeight = ( nTxtH ? nTxtH : nH );
+nCrsrHeight = ( nCrsrH ? nCrsrH : nTxtHeight );
+}
+
+void EditLine::SetStartPosX( long start )
+{
+if (start  0)
+nStartPosX = start;
+else
+nStartPosX = 0;
+}
+
 Size EditLine::CalcTextSize( ParaPortion rParaPortion )
 {
 Size aSz;
diff --git a/editeng/source/editeng/editdoc.hxx 
b/editeng/source/editeng/editdoc.hxx
index c595ea8..2c303ee 100644
--- a/editeng/source/editeng/editdoc.hxx
+++ b/editeng/source/editeng/editdoc.hxx
@@ -513,11 +513,7 @@ public:
 sal_uInt16  GetEndPortion() const   { return nEndPortion; }
 sal_uInt16 GetEndPortion() { return nEndPortion; }
 
-voidSetHeight( sal_uInt16 nH, sal_uInt16 nTxtH = 0, sal_uInt16 
nCrsrH = 0 )
-{   nHeight = nH;
-nTxtHeight = ( nTxtH ? nTxtH : nH );
-nCrsrHeight = ( nCrsrH ? nCrsrH : nTxtHeight );
-}
+voidSetHeight( sal_uInt16 nH, sal_uInt16 nTxtH = 0, sal_uInt16 
nCrsrH = 0 );
 sal_uInt16  GetHeight() const   { return nHeight; }
 sal_uInt16  GetTxtHeight() const{ return nTxtHeight; }
 sal_uInt16  GetCrsrHeight() const   { return nCrsrHeight; }
@@ -534,8 +530,7 @@ public:
 sal_uInt16  GetLen() const  { return nEnd - 
nStart; }
 
 sal_uInt16  GetStartPosX() const{ return nStartPosX; }
-voidSetStartPosX( long start )  { if (start  0) 
nStartPosX = start; else nStartPosX = 0; }
-
+voidSetStartPosX( long start );
 SizeCalcTextSize( ParaPortion rParaPortion );
 
 boolIsInvalid() const   { return bInvalid; }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-09-20 Thread Libreoffice Gerrit user
 editeng/source/editeng/editview.cxx |2 +-
 editeng/source/editeng/impedit2.cxx |2 +-
 editeng/source/editeng/impedit3.cxx |4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 45bae63f0f56060e2a609ebb2610502a3722dfee
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 14:19:46 2012 +0300

Add some static goodness here, too, to avoid duplicate symbols

Change-Id: Ie5e7aecd4f1c0e5b4dda9250ae755bf9210e048f

diff --git a/editeng/source/editeng/editview.cxx 
b/editeng/source/editeng/editview.cxx
index bd13494..9cb8900 100644
--- a/editeng/source/editeng/editview.cxx
+++ b/editeng/source/editeng/editview.cxx
@@ -85,7 +85,7 @@ DBG_NAME( EditView )
 
 
 // From SW = Create common method
-LanguageType lcl_CheckLanguage(
+static LanguageType lcl_CheckLanguage(
 const OUString rText,
 Reference XSpellChecker1  xSpell,
 Reference linguistic2::XLanguageGuessing  xLangGuess,
diff --git a/editeng/source/editeng/impedit2.cxx 
b/editeng/source/editeng/impedit2.cxx
index a85b163..3c9990e 100644
--- a/editeng/source/editeng/impedit2.cxx
+++ b/editeng/source/editeng/impedit2.cxx
@@ -1675,7 +1675,7 @@ sal_Bool ImpEditEngine::IsInputSequenceCheckingRequired( 
sal_Unicode nChar, cons
 return bIsSequenceChecking;
 }
 
- bool lcl_HasStrongLTR ( const String rTxt, xub_StrLen nStart, xub_StrLen 
nEnd )
+static  bool lcl_HasStrongLTR ( const String rTxt, xub_StrLen nStart, 
xub_StrLen nEnd )
  {
  for ( xub_StrLen nCharIdx = nStart; nCharIdx  nEnd; ++nCharIdx )
  {
diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 027302f..8e20ae0 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -254,7 +254,7 @@ Point lcl_ImplCalcRotatedPos( Point rPos, Point rOrigin, 
double nSin, double nCo
 return aTranslatedPos;
 }
 
-sal_Bool lcl_IsLigature( xub_Unicode cCh, xub_Unicode cNextCh ) // For 
Kashidas from sw/source/core/text/porlay.txt
+static sal_Bool lcl_IsLigature( xub_Unicode cCh, xub_Unicode cNextCh ) // For 
Kashidas from sw/source/core/text/porlay.txt
 {
 // Lam + Alef
 return ( 0x644 == cCh  0x627 == cNextCh ) ||
@@ -262,7 +262,7 @@ sal_Bool lcl_IsLigature( xub_Unicode cCh, xub_Unicode 
cNextCh ) // For Kashidas
( 0x628 == cCh  0x631 == cNextCh );
 }
 
-sal_Bool lcl_ConnectToPrev( xub_Unicode cCh, xub_Unicode cPrevCh )  // For 
Kashidas from sw/source/core/text/porlay.txt
+static sal_Bool lcl_ConnectToPrev( xub_Unicode cCh, xub_Unicode cPrevCh )  // 
For Kashidas from sw/source/core/text/porlay.txt
 {
 // Alef, Dal, Thal, Reh, Zain, and Waw do not connect to the left
 sal_Bool bRet = 0x627 != cPrevCh  0x62F != cPrevCh  0x630 != cPrevCh 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-08-22 Thread Libreoffice Gerrit user
 editeng/source/editeng/impedit3.cxx |   30 +-
 1 file changed, 17 insertions(+), 13 deletions(-)

New commits:
commit 4f177fbbab1619b9a2f9afa04d882c5fd415fcc0
Author: Caolán McNamara caol...@redhat.com
Date:   Wed Aug 22 14:38:50 2012 +0100

Resolves: rhbz#841092 don't crash on empty portions (a11y?)

Change-Id: Ibd08fa09370580bff12f19f867219098f0d4980e

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index a0aedb1..027302f 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -2477,22 +2477,26 @@ void ImpEditEngine::RecalcTextPortion( ParaPortion* 
pParaPortion, sal_uInt16 nSt
 pTP-GetLen() = pTP-GetLen() + nNewChars;
 }
 
-// No HYPHENATOR portion is allowed to get stuck right at the end...
-DBG_ASSERT( pParaPortion-GetTextPortions().Count(), 
RecalcTextPortions: Nothing left! );
-sal_uInt16 nLastPortion = pParaPortion-GetTextPortions().Count() - 1;
-pTP = pParaPortion-GetTextPortions()[nLastPortion];
-if ( pTP-GetKind() == PORTIONKIND_HYPHENATOR )
+sal_uInt16 nPortionCount = pParaPortion-GetTextPortions().Count();
+assert( nPortionCount );
+if (nPortionCount)
 {
-// Discard portion; if possible, correct the ones before,
-// if the Hyphenator portion has swallowed one character...
-if ( nLastPortion  pTP-GetLen() )
+// No HYPHENATOR portion is allowed to get stuck right at the 
end...
+sal_uInt16 nLastPortion = nPortionCount - 1;
+pTP = pParaPortion-GetTextPortions()[nLastPortion];
+if ( pTP-GetKind() == PORTIONKIND_HYPHENATOR )
 {
-TextPortion* pPrev = 
pParaPortion-GetTextPortions()[nLastPortion - 1];
-DBG_ASSERT( pPrev-GetKind() == PORTIONKIND_TEXT, Portion?! 
);
-pPrev-SetLen( pPrev-GetLen() + pTP-GetLen() );
-pPrev-GetSize().Width() = (-1);
+// Discard portion; if possible, correct the ones before,
+// if the Hyphenator portion has swallowed one character...
+if ( nLastPortion  pTP-GetLen() )
+{
+TextPortion* pPrev = 
pParaPortion-GetTextPortions()[nLastPortion - 1];
+DBG_ASSERT( pPrev-GetKind() == PORTIONKIND_TEXT, 
Portion?! );
+pPrev-SetLen( pPrev-GetLen() + pTP-GetLen() );
+pPrev-GetSize().Width() = (-1);
+}
+pParaPortion-GetTextPortions().Remove( nLastPortion );
 }
-pParaPortion-GetTextPortions().Remove( nLastPortion );
 }
 }
 #if OSL_DEBUG_LEVEL  2
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-08-09 Thread Caolán McNamara
 editeng/source/editeng/editdoc.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 069caf79d716ba73d3281beb13fa2af67cac24f9
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Aug 9 10:34:50 2012 +0100

valgrind: use after free

i.e. since b35980d9b

seems remove, remove, delete was old pattern, new patterns ended up
as remove+delete, remove. Reorder to remove, remove+delete

Change-Id: I54ec8d0296e751110c516516465be2ac0615f6a1

diff --git a/editeng/source/editeng/editdoc.cxx 
b/editeng/source/editeng/editdoc.cxx
index b4a08e5..cb13c91 100644
--- a/editeng/source/editeng/editdoc.cxx
+++ b/editeng/source/editeng/editdoc.cxx
@@ -2688,8 +2688,8 @@ void CharAttribList::OptimizeRanges( SfxItemPool 
rItemPool )
 if (*rNext.GetItem() == *rAttr.GetItem())
 {
 rAttr.GetEnd() = rNext.GetEnd();
-aAttribs.erase(aAttribs.begin()+nNext);
 rItemPool.Remove(*rNext.GetItem());
+aAttribs.erase(aAttribs.begin()+nNext);
 }
 break;  // only 1 attr with same which can start here.
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-07-31 Thread Stephan Bergmann
 editeng/source/lookuptree/Node.cxx |5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 9d75ea50e7e64b7d282655e986564acb640bf04e
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Jul 31 09:15:34 2012 +0200

error: attempt to increment a past-the-end iterator

...when running editeng/CppunitTest_editeng_lookuptree.mk

Change-Id: Ida1cbb16965138a42bec9e675b630bcbf2f6617e

diff --git a/editeng/source/lookuptree/Node.cxx 
b/editeng/source/lookuptree/Node.cxx
index 6d5e320..2492a88 100644
--- a/editeng/source/lookuptree/Node.cxx
+++ b/editeng/source/lookuptree/Node.cxx
@@ -56,7 +56,10 @@ void Node::removeChild(Node* pChild)
 {
 i = m_lChildren.erase( i );
 }
-++i;
+else
+{
+++i;
+}
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-07-12 Thread Caolán McNamara
 editeng/source/misc/svxacorr.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 058116026965ad31164fe60d5cd8a6d5fbbe8f65
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Jul 12 15:07:13 2012 +0100

make this a const_iterator for windows buildbot

Change-Id: I6e242de572595fdf39553d76932bc0e953cd3a34

diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index 31a1cda..68473fd 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -1658,7 +1658,7 @@ static const SvxAutocorrWord* lcl_SearchWordsInList(
 {
 const SvxAutocorrWordList* pAutoCorrWordList = 
pList-GetAutocorrWordList();
 TransliterationWrapper rCmp = GetIgnoreTranslWrapper();
-for( SvxAutocorrWordList::iterator it = pAutoCorrWordList-begin(); it != 
pAutoCorrWordList-end(); ++it )
+for( SvxAutocorrWordList::const_iterator it = pAutoCorrWordList-begin(); 
it != pAutoCorrWordList-end(); ++it )
 {
 const SvxAutocorrWord* pFnd = *it;
 const String rChk = pFnd-GetShort();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-07-12 Thread Caolán McNamara
 editeng/source/misc/SvXMLAutoCorrectExport.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 1d2ba733d758cbf7a6878debc8d183960996bde6
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Jul 12 17:13:35 2012 +0100

throw some const love at SvxAutocorrWordList

Change-Id: I41d59d386b8f1fd5dc1fb9744a649085c66143f9

diff --git a/editeng/source/misc/SvXMLAutoCorrectExport.cxx 
b/editeng/source/misc/SvXMLAutoCorrectExport.cxx
index 6719b39..2552f2f 100644
--- a/editeng/source/misc/SvXMLAutoCorrectExport.cxx
+++ b/editeng/source/misc/SvXMLAutoCorrectExport.cxx
@@ -61,9 +61,9 @@ sal_uInt32 SvXMLAutoCorrectExport::exportDoc(enum 
XMLTokenEnum /*eClass*/)
_GetNamespaceMap().GetNameByKey ( XML_NAMESPACE_BLOCKLIST ) 
);
 {
 SvXMLElementExport aRoot (*this, XML_NAMESPACE_BLOCKLIST, 
XML_BLOCK_LIST, sal_True, sal_True);
-for ( SvxAutocorrWordList::iterator it = pAutocorr_List-begin(); it 
!= pAutocorr_List-end(); ++it )
+for ( SvxAutocorrWordList::const_iterator it = 
pAutocorr_List-begin(); it != pAutocorr_List-end(); ++it )
 {
-SvxAutocorrWord* p = *it;
+const SvxAutocorrWord* p = *it;
 
 AddAttribute( XML_NAMESPACE_BLOCKLIST,
   XML_ABBREVIATED_NAME,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-07-05 Thread Michael Stahl
 editeng/source/items/borderline.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 927f1e56d57ef9238a4574699ae36626b3daaf22
Author: Michael Stahl mst...@redhat.com
Date:   Thu Jul 5 18:28:41 2012 +0200

ConvertBorderWidthToWord: fix wrong INSET/OUSET conversion:

This should do the inverse of ConvertBorderWidthFromWord.

Change-Id: If0b2a8a83a7faa6600a07ecfcb13a124d7cdeab6

diff --git a/editeng/source/items/borderline.cxx 
b/editeng/source/items/borderline.cxx
index 225d914..704bbfa 100644
--- a/editeng/source/items/borderline.cxx
+++ b/editeng/source/items/borderline.cxx
@@ -224,11 +224,11 @@ ConvertBorderWidthToWord(SvxBorderStyle const eStyle, 
double const fWidth)
 break;
 
 case OUTSET:
-return (fWidth / 2.0) - OUTSET_line1;
+return (fWidth - OUTSET_line1) / 2.0;
 break;
 
 case INSET:
-return (fWidth / 2.0) - INSET_line2;
+return (fWidth - INSET_line2) / 2.0;
 break;
 
 default:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-07-02 Thread Kohei Yoshida
 editeng/source/editeng/impedit3.cxx |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 6cf78c72be5f8f1b0a037b03a6fc63cbace0a2d9
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Mon Jul 2 13:27:52 2012 -0400

fdo#51582: Avoid double deletion of a text portion.

Change-Id: Ia1c31cc8122c99a7e37bc285f61f8c5d2779b577

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 3a5ac2b..504d774 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -896,18 +896,19 @@ sal_Bool ImpEditEngine::CreateLines( sal_uInt16 nPara, 
sal_uInt32 nStartPosY )
 {
 // Throw away a Portion, if necessary correct the one before,
 // if the Hyph portion has swallowed a character ...
+sal_uInt16 nTmpLen = pPortion-GetLen();
 pParaPortion-GetTextPortions().Remove( nTmpPortion );
-if ( nTmpPortion  pPortion-GetLen() )
+if (nTmpPortion  nTmpLen)
 {
 nTmpPortion--;
 TextPortion* pPrev = 
pParaPortion-GetTextPortions()[nTmpPortion];
 DBG_ASSERT( pPrev-GetKind() == PORTIONKIND_TEXT, 
Portion?! );
 nTmpWidth -= pPrev-GetSize().Width();
 nTmpPos = nTmpPos - pPrev-GetLen();
-pPrev-SetLen( pPrev-GetLen() + pPortion-GetLen() );
+pPrev-SetLen(pPrev-GetLen() + nTmpLen);
 pPrev-GetSize().Width() = (-1);
 }
-delete pPortion;
+
 DBG_ASSERT( nTmpPortion  
pParaPortion-GetTextPortions().Count(), No more Portions left! );
 pPortion = pParaPortion-GetTextPortions()[nTmpPortion];
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-06-13 Thread Stephan Bergmann
 editeng/source/editeng/editdoc.cxx |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit a4242df3ae36d9158317298fe01b5cfa0aae4b32
Author: Stephan Bergmann sberg...@redhat.com
Date:   Wed Jun 13 14:09:38 2012 +0200

Take care of short strings

Change-Id: I469b27a45bb5676570b59a829419f3959210a1a6

diff --git a/editeng/source/editeng/editdoc.cxx 
b/editeng/source/editeng/editdoc.cxx
index fe747c3..08722a5 100644
--- a/editeng/source/editeng/editdoc.cxx
+++ b/editeng/source/editeng/editdoc.cxx
@@ -68,6 +68,7 @@
 #include tools/shl.hxx
 #include com/sun/star/i18n/ScriptType.hpp
 
+#include cassert
 #include limits
 
 #include boost/bind.hpp
@@ -2064,6 +2065,8 @@ XubString EditDoc::GetText( LineEnd eEnd ) const
 pCur += nSepSize;
 }
 }
+assert(pCur - newStr-buffer = newStr-length);
+newStr-length = pCur - newStr-buffer;
 return rtl::OUString(newStr, SAL_NO_ACQUIRE);
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-06-13 Thread Stephan Bergmann
 editeng/source/editeng/editdoc.cxx |5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

New commits:
commit 5c47001d38af7b61db2089f5b3b8dc97e9f8d891
Author: Stephan Bergmann sberg...@redhat.com
Date:   Wed Jun 13 15:26:36 2012 +0200

Better fix for those short strings

...that are not intended to be short after all; just the length
calculation turned out to be wrong.

Change-Id: Ic3a81c935161564f980ddd0241575fff2f76b467

diff --git a/editeng/source/editeng/editdoc.cxx 
b/editeng/source/editeng/editdoc.cxx
index 08722a5..0ef70bb 100644
--- a/editeng/source/editeng/editdoc.cxx
+++ b/editeng/source/editeng/editdoc.cxx
@@ -2044,7 +2044,7 @@ XubString EditDoc::GetText( LineEnd eEnd ) const
 sal_Int32 nSepSize = aSep.getLength();
 
 if ( nSepSize )
-nLen += nNodes * nSepSize;
+nLen += (nNodes - 1) * nSepSize;
 if ( nLen  0xFFFb / sizeof(xub_Unicode) )
 {
 OSL_FAIL( Text too large for String );
@@ -2065,8 +2065,7 @@ XubString EditDoc::GetText( LineEnd eEnd ) const
 pCur += nSepSize;
 }
 }
-assert(pCur - newStr-buffer = newStr-length);
-newStr-length = pCur - newStr-buffer;
+assert(pCur - newStr-buffer == newStr-length);
 return rtl::OUString(newStr, SAL_NO_ACQUIRE);
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-06-13 Thread Stephan Bergmann
 editeng/source/editeng/editdoc.cxx |5 -
 1 file changed, 5 deletions(-)

New commits:
commit 59f5d8f54bd485f828aa1266b2bc11a9b05bd1e7
Author: Stephan Bergmann sberg...@redhat.com
Date:   Wed Jun 13 16:37:39 2012 +0200

Remove unnecessary size-too-large check

...it is done in UniString(rtl::OUString const) ctor anyway

Change-Id: I890ab21a6c04fe0ddc0a9b2c9a356359984e516e

diff --git a/editeng/source/editeng/editdoc.cxx 
b/editeng/source/editeng/editdoc.cxx
index 0ef70bb..59f4524 100644
--- a/editeng/source/editeng/editdoc.cxx
+++ b/editeng/source/editeng/editdoc.cxx
@@ -2045,11 +2045,6 @@ XubString EditDoc::GetText( LineEnd eEnd ) const
 
 if ( nSepSize )
 nLen += (nNodes - 1) * nSepSize;
-if ( nLen  0xFFFb / sizeof(xub_Unicode) )
-{
-OSL_FAIL( Text too large for String );
-return rtl::OUString();
-}
 
 rtl_uString* newStr = comphelper::string::rtl_uString_alloc(nLen);
 xub_Unicode* pCur = newStr-buffer;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-06-06 Thread Caolán McNamara
 editeng/source/editeng/impedit2.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 7093b2418d036b47ed706464956f330cfb89707a
Author: Caolán McNamara caol...@redhat.com
Date:   Wed Jun 6 16:23:36 2012 +0100

valgrind: uninitialized value

Change-Id: I405733a77a721f3d5562782e654a1942fe3b073e

diff --git a/editeng/source/editeng/impedit2.cxx 
b/editeng/source/editeng/impedit2.cxx
index 246ad29..be1e766 100644
--- a/editeng/source/editeng/impedit2.cxx
+++ b/editeng/source/editeng/impedit2.cxx
@@ -99,6 +99,7 @@ ImpEditEngine::ImpEditEngine( EditEngine* pEE, SfxItemPool* 
pItemPool ) :
 bAddExtLeading(false),
 bIsFormatting(false),
 bFormatted(false),
+bInSelection(false),
 bIsInUndo(false),
 bUpdate(true),
 bUndoEnabled(true),
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-06-03 Thread Muthu Subramanian
 editeng/source/editeng/editdoc.hxx  |1 
 editeng/source/editeng/impedit3.cxx |   70 ++--
 2 files changed, 69 insertions(+), 2 deletions(-)

New commits:
commit 0eaed74b10714c005ea28db4ecb4cb92f56ca0d8
Author: Muthu Subramanian sumu...@suse.com
Date:   Mon Jun 4 10:58:18 2012 +0530

Hyperlinks/Fields wrapping feature.

* Known issue: If the hyperlink wraps the next content is always from
  the new line! :(
* Pending: Creation of unit test case for this.

diff --git a/editeng/source/editeng/editdoc.hxx 
b/editeng/source/editeng/editdoc.hxx
index 8dfe4f6..a97fcec 100644
--- a/editeng/source/editeng/editdoc.hxx
+++ b/editeng/source/editeng/editdoc.hxx
@@ -375,6 +375,7 @@ struct ExtraPortionInfo
 sal_BoolbCompressed;
 
 sal_Int32*pOrgDXArray;
+::std::vector sal_Int32  lineBreaksList;
 
 
 ExtraPortionInfo();
diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 451ba16..9bac785 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -1009,7 +1009,31 @@ sal_Bool ImpEditEngine::CreateLines( sal_uInt16 nPara, 
sal_uInt32 nStartPosY )
 pPortion-GetSize() = aTmpFont.QuickGetTextSize( 
GetRefDevice(), aFieldValue, 0, aFieldValue.getLength(), 0 );
 // So no scrolling for oversized fields
 if ( pPortion-GetSize().Width()  nXWidth )
+{
+sal_Int32 nWidthOrg = 
pPortion-GetSize().Width();
+sal_Int32 nChars= 
aFieldValue.Len();
+sal_Int32 nApproxWC = nXWidth / ( 
nWidthOrg / nChars );
+ExtraPortionInfo *pExtraInfo= 
pPortion-GetExtraInfos();
+if( !nApproxWC ) nApproxWC++;
+if( pExtraInfo == NULL )
+{
+pExtraInfo = new ExtraPortionInfo();
+pExtraInfo-nOrgWidth = nXWidth;
+pPortion-SetExtraInfos( pExtraInfo );
+}
+else
+{
+pExtraInfo-lineBreaksList.clear();
+}
+
 pPortion-GetSize().Width() = nXWidth;
+
+while( nChars  0 )
+{
+pExtraInfo-lineBreaksList.push_back( 
aFieldValue.Len() - nChars );
+nChars -= nApproxWC;
+}
+}
 }
 nTmpWidth += pPortion-GetSize().Width();
 EditLine::CharPosArrayType rArray = 
pLine-GetCharPosArray();
@@ -2922,6 +2946,8 @@ void ImpEditEngine::Paint( OutputDevice* pOutDev, 
Rectangle aClipRec, Point aSta
 // Over the Portions of the line ...
 // --
 sal_uInt16 nIndex = pLine-GetStart();
+bool bParsingFields = false;
+::std::vector sal_Int32 ::iterator itSubLines;
 for ( sal_uInt16 y = pLine-GetStartPortion(); y = 
pLine-GetEndPortion(); y++ )
 {
 DBG_ASSERT( pPortion-GetTextPortions().Count(), Line 
without Textportion in Paint! );
@@ -3098,12 +3124,48 @@ void ImpEditEngine::Paint( OutputDevice* pOutDev, 
Rectangle aClipRec, Point aSta
 aText = 
((EditCharAttribField*)pAttr)-GetFieldValue();
 nTextStart = 0;
 nTextLen = aText.Len();
+ExtraPortionInfo *pExtraInfo = 
pTextPortion-GetExtraInfos();
+// Do not split the Fields into different 
lines while editing
+if( bStripOnly  !bParsingFields  
pExtraInfo  pExtraInfo-lineBreaksList.size() )
+{
+bParsingFields = true;
+itSubLines = 
pExtraInfo-lineBreaksList.begin();
+}
+if( bParsingFields )
+{
+if( itSubLines != 
pExtraInfo-lineBreaksList.begin() )
+{
+if ( !IsVertical() )
+{
+aStartPos.Y() += 
pLine-GetMaxAscent();
+   

[Libreoffice-commits] .: editeng/source

2012-06-03 Thread Julien Nabet
 editeng/source/editeng/impedit3.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit c3b12e24ce5e62e05d524a338e1c17f8e7a53f53
Author: Julien Nabet serval2...@yahoo.fr
Date:   Mon Jun 4 07:58:01 2012 +0200

Fix call of getLength

Change-Id: I5fb5bc5016bd2779643ecfcf2df40e00c4741a2c

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 9bac785..83e5689 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -1011,7 +1011,7 @@ sal_Bool ImpEditEngine::CreateLines( sal_uInt16 nPara, 
sal_uInt32 nStartPosY )
 if ( pPortion-GetSize().Width()  nXWidth )
 {
 sal_Int32 nWidthOrg = 
pPortion-GetSize().Width();
-sal_Int32 nChars= 
aFieldValue.Len();
+sal_Int32 nChars= 
aFieldValue.getLength();
 sal_Int32 nApproxWC = nXWidth / ( 
nWidthOrg / nChars );
 ExtraPortionInfo *pExtraInfo= 
pPortion-GetExtraInfos();
 if( !nApproxWC ) nApproxWC++;
@@ -1030,7 +1030,7 @@ sal_Bool ImpEditEngine::CreateLines( sal_uInt16 nPara, 
sal_uInt32 nStartPosY )
 
 while( nChars  0 )
 {
-pExtraInfo-lineBreaksList.push_back( 
aFieldValue.Len() - nChars );
+pExtraInfo-lineBreaksList.push_back( 
aFieldValue.getLength() - nChars );
 nChars -= nApproxWC;
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source offapi/com sc/source sd/source svx/source

2012-05-11 Thread Kohei Yoshida
 editeng/source/items/flditem.cxx|1 -
 editeng/source/uno/unofield.cxx |   10 --
 editeng/source/xml/xmltxtexp.cxx|2 +-
 offapi/com/sun/star/text/textfield/Type.idl |2 --
 sc/source/ui/unoobj/fielduno.cxx|5 +
 sd/source/ui/unoidl/unomodel.cxx|2 +-
 svx/source/unodraw/unomod.cxx   |2 +-
 7 files changed, 4 insertions(+), 20 deletions(-)

New commits:
commit ee5fc5d25fe102c30daf7d181b8181d40b85a4f3
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Fri May 11 13:03:07 2012 -0400

Removing EXTENDED_DATE. There is really no such thing.

This can be safely substituted by the normal DATE.

Change-Id: I I6b4ccfa342a8d7b638b013cdce64a0b7ff477ec0

diff --git a/editeng/source/items/flditem.cxx b/editeng/source/items/flditem.cxx
index d5aec5b..bcc578d 100644
--- a/editeng/source/items/flditem.cxx
+++ b/editeng/source/items/flditem.cxx
@@ -63,7 +63,6 @@ SvxFieldData* SvxFieldData::Create(const 
uno::Referencetext::XTextContent xTe
 case text::textfield::Type::TIME:
 case text::textfield::Type::EXTENDED_TIME:
 case text::textfield::Type::DATE:
-case text::textfield::Type::EXTENDED_DATE:
 {
 sal_Bool bIsDate = false;
 xPropSet-getPropertyValue(UNO_TC_PROP_IS_DATE) = bIsDate;
diff --git a/editeng/source/uno/unofield.cxx b/editeng/source/uno/unofield.cxx
index 83cad2d..a156c89 100644
--- a/editeng/source/uno/unofield.cxx
+++ b/editeng/source/uno/unofield.cxx
@@ -146,7 +146,6 @@ const SfxItemPropertySet* ImplGetFieldItemPropertySet( 
sal_Int32 mnId )
 
 switch( mnId )
 {
-case text::textfield::Type::EXTENDED_DATE:
 case text::textfield::Type::EXTENDED_TIME:
 case text::textfield::Type::DATE:
 return aExDateTimeFieldPropertySet_Impl;
@@ -266,7 +265,6 @@ SvxUnoTextField::SvxUnoTextField( sal_Int32 nServiceId ) 
throw()
 
 switch( nServiceId )
 {
-case text::textfield::Type::EXTENDED_DATE:
 case text::textfield::Type::DATE:
 mpImpl-mbBoolean2 = sal_True;
 mpImpl-mnInt32 = SVXDATEFORMAT_STDSMALL;
@@ -329,7 +327,6 @@ SvxUnoTextField::SvxUnoTextField( uno::Reference 
text::XTextRange  xAnchor, co
 switch( mnServiceId )
 {
 case text::textfield::Type::DATE:
-case text::textfield::Type::EXTENDED_DATE:
 {
 mpImpl-mbBoolean2 = sal_True;
 // #i35416# for variable date field, don't use invalid 
-00-00 date,
@@ -403,7 +400,6 @@ SvxFieldData* SvxUnoTextField::CreateFieldData() const 
throw()
 case text::textfield::Type::TIME:
 case text::textfield::Type::EXTENDED_TIME:
 case text::textfield::Type::DATE:
-case text::textfield::Type::EXTENDED_DATE:
 {
 if( mpImpl-mbBoolean2 ) // IsDate?
 {
@@ -630,8 +626,6 @@ OUString SAL_CALL SvxUnoTextField::getPresentation( 
sal_Bool bShowCommand )
 return OUString(Footer);
 case text::textfield::Type::PRESENTATION_DATE_TIME:
 return OUString(DateTime);
-case text::textfield::Type::EXTENDED_DATE:
-return OUString(ExtDate);
 default:
 return OUString(Unknown);
 }
@@ -879,10 +873,6 @@ uno::Sequence OUString  SAL_CALL 
SvxUnoTextField::getSupportedServiceNames()
 pServices[2] = com.sun.star.presentation.TextField.DateTime;
 pServices[3] = com.sun.star.presentation.textfield.DateTime;
 break;
-case text::textfield::Type::EXTENDED_DATE:
-pServices[2] = com.sun.star.text.TextField.DateTime;
-pServices[3] = com.sun.star.text.textfield.DateTime;
-break;
 default:
 aSeq.realloc(0);
 }
diff --git a/editeng/source/xml/xmltxtexp.cxx b/editeng/source/xml/xmltxtexp.cxx
index 15ea264..a0455ff 100644
--- a/editeng/source/xml/xmltxtexp.cxx
+++ b/editeng/source/xml/xmltxtexp.cxx
@@ -245,7 +245,7 @@ uno::Reference uno::XInterface  SAL_CALL 
SvxSimpleUnoModel::createInstance( co
 
RTL_CONSTASCII_STRINGPARAM(com.sun.star.text.TextField.DateTime)))
)
 {
-return (::cppu::OWeakObject * )new SvxUnoTextField( 
text::textfield::Type::EXTENDED_DATE );
+return (::cppu::OWeakObject * )new SvxUnoTextField( 
text::textfield::Type::DATE );
 }
 
 return SvxUnoTextCreateTextField( aServiceSpecifier );
diff --git a/offapi/com/sun/star/text/textfield/Type.idl 
b/offapi/com/sun/star/text/textfield/Type.idl
index a9a2653..cfa864a 100644
--- a/offapi/com/sun/star/text/textfield/Type.idl
+++ b/offapi/com/sun/star/text/textfield/Type.idl
@@ -60,8 +60,6 @@ constants Type
 const long PRESENTATION_HEADER= 11;
 const long PRESENTATION_FOOTER= 12;
 const long PRESENTATION_DATE_TIME = 13;
-
-const long EXTENDED_DATE = 14;
 };
 
 }; }; }; }; };
diff --git 

[Libreoffice-commits] .: editeng/source filter/source sfx2/source svtools/inc svtools/source svx/source toolkit/source tools/source unotools/source vcl/source

2012-05-02 Thread Julien Nabet
 editeng/source/outliner/outliner.cxx  |2 +-
 filter/source/graphicfilter/icgm/chart.cxx|2 +-
 filter/source/graphicfilter/icgm/elements.cxx |2 +-
 filter/source/msfilter/svdfppt.cxx|4 ++--
 sfx2/source/view/viewsh.cxx   |2 +-
 svtools/inc/svtools/treelist.hxx  |2 +-
 svtools/source/contnr/imivctl1.cxx|2 +-
 svtools/source/contnr/treelist.cxx|2 +-
 svtools/source/filter/filter.cxx  |2 +-
 svtools/source/graphic/grfmgr2.cxx|2 +-
 svx/source/form/fmexpl.cxx|2 +-
 svx/source/gallery2/gallery1.cxx  |4 ++--
 toolkit/source/awt/vclxgraphics.cxx   |2 +-
 tools/source/stream/strmunx.cxx   |2 +-
 unotools/source/config/options.cxx|2 +-
 vcl/source/app/idlemgr.cxx|2 +-
 vcl/source/window/accmgr.cxx  |2 +-
 vcl/source/window/toolbox.cxx |2 +-
 18 files changed, 20 insertions(+), 20 deletions(-)

New commits:
commit 46c01657d505a021673449ff0f874bc080d23ba6
Author: Julien Nabet serval2...@yahoo.fr
Date:   Wed May 2 21:57:04 2012 +0200

!= instead of  for comparison with end iterator

Change-Id: I7baa34c1cf7161fb55914d41b451b842ad4ab0c8

diff --git a/editeng/source/outliner/outliner.cxx 
b/editeng/source/outliner/outliner.cxx
index b1690e9..ffefcfb 100644
--- a/editeng/source/outliner/outliner.cxx
+++ b/editeng/source/outliner/outliner.cxx
@@ -1352,7 +1352,7 @@ OutlinerView* Outliner::RemoveView( OutlinerView* pView )
 {
 DBG_CHKTHIS(Outliner,0);
 
-for ( ViewList::iterator it = aViewList.begin(); it  aViewList.end(); 
++it )
+for ( ViewList::iterator it = aViewList.begin(); it != aViewList.end(); 
++it )
 {
 if ( *it == pView )
 {
diff --git a/filter/source/graphicfilter/icgm/chart.cxx 
b/filter/source/graphicfilter/icgm/chart.cxx
index 7e34fdf..4651a04 100644
--- a/filter/source/graphicfilter/icgm/chart.cxx
+++ b/filter/source/graphicfilter/icgm/chart.cxx
@@ -71,7 +71,7 @@ void CGMChart::DeleteTextEntry( TextEntry* pTextEntry )
 }
 delete pTextEntry;
 ::std::vector TextEntry* ::iterator it;
-for ( it = maTextEntryList.begin(); it  maTextEntryList.end(); ++it )
+for ( it = maTextEntryList.begin(); it != maTextEntryList.end(); ++it )
 {
 if ( *it == pTextEntry )
 {
diff --git a/filter/source/graphicfilter/icgm/elements.cxx 
b/filter/source/graphicfilter/icgm/elements.cxx
index a3f5892..07fef99 100644
--- a/filter/source/graphicfilter/icgm/elements.cxx
+++ b/filter/source/graphicfilter/icgm/elements.cxx
@@ -357,7 +357,7 @@ Bundle* CGMElements::InsertBundle( BundleList rList, 
Bundle rBundle )
 Bundle* pBundle = GetBundle( rList, rBundle.GetIndex() );
 if ( pBundle )
 {
-for ( BundleList::iterator it = rList.begin(); it  rList.end(); ++it 
) {
+for ( BundleList::iterator it = rList.begin(); it != rList.end(); ++it 
) {
 if ( *it == pBundle ) {
 rList.erase( it );
 delete pBundle;
diff --git a/filter/source/msfilter/svdfppt.cxx 
b/filter/source/msfilter/svdfppt.cxx
index 7f36b99..d61ab39 100644
--- a/filter/source/msfilter/svdfppt.cxx
+++ b/filter/source/msfilter/svdfppt.cxx
@@ -6721,12 +6721,12 @@ PPTTextObj::PPTTextObj( SvStream rIn, 
SdrPowerPointImport rSdrPowerPointImport
 {
 // sorting fields ( hi  lo )
 ::std::vector PPTFieldEntry* ::iterator 
it = FieldList.begin();
-for( ; it  FieldList.end(); ++it ) {
+for( ; it != FieldList.end(); ++it ) {
 if ( (*it)-nPos  pEntry-nPos ) {
 break;
 }
 }
-if ( it  FieldList.end() ) {
+if ( it != FieldList.end() ) {
 FieldList.insert( it, pEntry );
 } else {
 FieldList.push_back( pEntry );
diff --git a/sfx2/source/view/viewsh.cxx b/sfx2/source/view/viewsh.cxx
index b961592..d4cb20b 100644
--- a/sfx2/source/view/viewsh.cxx
+++ b/sfx2/source/view/viewsh.cxx
@@ -433,7 +433,7 @@ SAL_DLLPRIVATE void SfxViewShell::IPClientGone_Impl( 
SfxInPlaceClient *pIPClient
 {
 SfxInPlaceClientList* pClientList = GetIPClientList_Impl(sal_True);
 
-for( SfxInPlaceClientList::iterator it = pClientList-begin(); it  
pClientList-end(); ++it )
+for( SfxInPlaceClientList::iterator it = pClientList-begin(); it != 
pClientList-end(); ++it )
 {
 if ( *it == pIPClient )
 {
diff --git 

[Libreoffice-commits] .: editeng/source

2012-04-28 Thread Julien Nabet
 editeng/source/editeng/impedit4.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 953ba47e75a449c464b4ca488073360cf622553d
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sat Apr 28 22:56:44 2012 +0200

Some cppcheck cleaning

Change-Id: Idf2e6e3e2870429541b3b5c225470cc80a13c102

diff --git a/editeng/source/editeng/impedit4.cxx 
b/editeng/source/editeng/impedit4.cxx
index 7316d7e..f2373fa 100644
--- a/editeng/source/editeng/impedit4.cxx
+++ b/editeng/source/editeng/impedit4.cxx
@@ -2160,7 +2160,7 @@ void ImpEditEngine::ApplyChangedSentence(EditView 
rEditView,
 }
 else
 {
-DBG_ASSERT( pSpellInfo-aLastSpellContentSelections.size()  0, 
aLastSpellContentSelections should not be empty here );
+DBG_ASSERT( !pSpellInfo-aLastSpellContentSelections.empty(), 
aLastSpellContentSelections should not be empty here );
 
 //select the complete sentence
 SpellContentSelections::const_iterator aCurrentEndPosition = 
pSpellInfo-aLastSpellContentSelections.end();
@@ -2217,7 +2217,7 @@ void ImpEditEngine::ApplyChangedSentence(EditView 
rEditView,
 
 void ImpEditEngine::PutSpellingToSentenceStart( EditView rEditView )
 {
-if( pSpellInfo  pSpellInfo-aLastSpellContentSelections.size() )
+if( pSpellInfo  !pSpellInfo-aLastSpellContentSelections.empty() )
 {
 rEditView.pImpEditView-SetEditSelection( 
pSpellInfo-aLastSpellContentSelections.begin()-Min() );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-26 Thread Michael Meeks
 editeng/source/misc/svxacorr.cxx |   14 ++
 1 file changed, 14 insertions(+)

New commits:
commit b269dced445494891a8e6e8d1d62b931a31dddbd
Author: Michael Meeks michael.me...@suse.com
Date:   Thu Apr 26 11:59:02 2012 +0100

fdo#46805 - special-case appending items to autocorrect lists

diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index f459af0..ce5c71c 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -68,6 +68,7 @@
 #include com/sun/star/ucb/NameClash.hpp
 #include xmloff/xmltoken.hxx
 #include vcl/help.hxx
+#include rtl/logfile.hxx
 
 #define CHAR_HARDBLANK  ((sal_Unicode)0x00A0)
 
@@ -233,15 +234,27 @@ void SvxAutocorrWordList::DeleteAndDestroy( sal_uInt16 
nP, sal_uInt16 nL )
 }
 
 
+// Keep the list sorted ...
 sal_Bool SvxAutocorrWordList::Seek_Entry( const SvxAutocorrWordPtr aE, 
sal_uInt16* pP ) const
 {
 register sal_uInt16 nO  = SvxAutocorrWordList_SAR::Count(),
 nM,
 nU = 0;
+
 if( nO  0 )
 {
 CollatorWrapper rCmp = ::GetCollatorWrapper();
 nO--;
+
+// quick check of the end of the list
+if (rCmp.compareString( aE-GetShort(),
+(*((SvxAutocorrWordPtr*)pData + 
nO))-GetShort() )  0)
+{
+if( pP ) *pP = nO + 1;
+return sal_False;
+}
+
+// Incredibly crude sort algorithm, should use some partitioning 
search.
 while( nU = nO )
 {
 nM = nU + ( nO - nU ) / 2;
@@ -2169,6 +2182,7 @@ SvxAutocorrWordList* 
SvxAutoCorrectLanguageLists::LoadAutocorrWordList()
 OSL_ENSURE( xXMLParser.is(), XMLReader::Read: 
com.sun.star.xml.sax.Parser service missing );
 if( xXMLParser.is() )
 {
+RTL_LOGFILE_PRODUCT_CONTEXT( aLog, AutoCorrect Import );
 uno::Reference xml::sax::XDocumentHandler  xFilter = new 
SvXMLAutoCorrectImport( xServiceFactory, pAutocorr_List, rAutoCorrect, xStg );
 
 // connect parser and filter
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-20 Thread Muthu Subramanian
 editeng/source/items/frmitems.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 1489254311e75a379a129bceeab49643ebea04e0
Author: Dezsi Szabolcs dezsisz...@hotmail.com
Date:   Fri Apr 20 14:10:56 2012 +0530

fdo#47436: Fixed crash while opening odt file.

diff --git a/editeng/source/items/frmitems.cxx 
b/editeng/source/items/frmitems.cxx
index efdcadb..d4b876b 100644
--- a/editeng/source/items/frmitems.cxx
+++ b/editeng/source/items/frmitems.cxx
@@ -1951,7 +1951,8 @@ bool SvxBoxItem::PutValue( const uno::Any rVal, 
sal_uInt8 nMemberId )
 for (int n(0); n != SAL_N_ELEMENTS(aBorders); ++n)
 {
 editeng::SvxBorderLine* pLine = const_cast 
editeng::SvxBorderLine* ( GetLine( aBorders[n] ) );
-pLine-SetStyle( eBorderStyle );
+if( pLine )
+pLine-SetStyle( eBorderStyle );
 }
 return sal_True;
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-16 Thread Stephan Bergmann
 editeng/source/editeng/editdbg.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 25de2a561b8f2618c2fa42dab39f57c727ce03e5
Author: Stephan Bergmann sberg...@redhat.com
Date:   Mon Apr 16 15:52:46 2012 +0200

Corret size_t format specifier

diff --git a/editeng/source/editeng/editdbg.cxx 
b/editeng/source/editeng/editdbg.cxx
index 98b4e9c..79c2c14 100644
--- a/editeng/source/editeng/editdbg.cxx
+++ b/editeng/source/editeng/editdbg.cxx
@@ -461,7 +461,7 @@ void EditDbg::ShowEditEngineData( EditEngine* pEE, sal_Bool 
bInfoBox )
 fprintf( fp, \nMaxAutoPaperSize: %li x %li, 
pEE-GetMaxAutoPaperSize().Width(), pEE-GetMaxAutoPaperSize().Height() );
 fprintf( fp, \nMinAutoPaperSize: %li x %li, 
pEE-GetMinAutoPaperSize().Width(), pEE-GetMinAutoPaperSize().Height() );
 fprintf( fp, \nUpdate: %i, pEE-GetUpdateMode() );
-fprintf( fp, \nNumber of Views: % SAL_PRIuUINT64, pEE-GetViewCount() );
+fprintf( fp, \nNumber of Views: % SAL_PRI_SIZET i, pEE-GetViewCount() 
);
 for ( sal_uInt16 nView = 0; nView  pEE-GetViewCount(); nView++ )
 {
 EditView* pV = pEE-GetView( nView );
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-14 Thread Julien Nabet
 editeng/source/editeng/editdbg.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit daae05ccf52e37afc41dc1976afcc1f7c6fd978c
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sat Apr 14 10:05:34 2012 +0200

WaE fix

diff --git a/editeng/source/editeng/editdbg.cxx 
b/editeng/source/editeng/editdbg.cxx
index af2e37f..98b4e9c 100644
--- a/editeng/source/editeng/editdbg.cxx
+++ b/editeng/source/editeng/editdbg.cxx
@@ -461,7 +461,7 @@ void EditDbg::ShowEditEngineData( EditEngine* pEE, sal_Bool 
bInfoBox )
 fprintf( fp, \nMaxAutoPaperSize: %li x %li, 
pEE-GetMaxAutoPaperSize().Width(), pEE-GetMaxAutoPaperSize().Height() );
 fprintf( fp, \nMinAutoPaperSize: %li x %li, 
pEE-GetMinAutoPaperSize().Width(), pEE-GetMinAutoPaperSize().Height() );
 fprintf( fp, \nUpdate: %i, pEE-GetUpdateMode() );
-fprintf( fp, \nNumber of Views: %i, pEE-GetViewCount() );
+fprintf( fp, \nNumber of Views: % SAL_PRIuUINT64, pEE-GetViewCount() );
 for ( sal_uInt16 nView = 0; nView  pEE-GetViewCount(); nView++ )
 {
 EditView* pV = pEE-GetView( nView );
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-14 Thread Julien Nabet
 editeng/source/editeng/impedit3.cxx |5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

New commits:
commit 95cc5de63b20c5986fe8f3913da86002eabd7cb1
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sat Apr 14 18:06:08 2012 +0200

Resolves: fdo#48683 nice crash when hitting DEL

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 0d6877b..5ea5d71 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -2436,7 +2436,6 @@ void ImpEditEngine::RecalcTextPortion( ParaPortion* 
pParaPortion, sal_uInt16 nSt
 // Remove portion;
 sal_uInt8 nType = pTP-GetKind();
 pParaPortion-GetTextPortions().Remove( nPortion );
-delete pTP;
 if ( nType == PORTIONKIND_LINEBREAK )
 {
 TextPortion* pNext = pParaPortion-GetTextPortions()[ nPortion 
];
@@ -2444,7 +2443,6 @@ void ImpEditEngine::RecalcTextPortion( ParaPortion* 
pParaPortion, sal_uInt16 nSt
 {
 // Remove dummy portion
 pParaPortion-GetTextPortions().Remove( nPortion );
-delete pNext;
 }
 }
 }
@@ -2462,7 +2460,6 @@ void ImpEditEngine::RecalcTextPortion( ParaPortion* 
pParaPortion, sal_uInt16 nSt
 {
 // Discard portion; if possible, correct the ones before,
 // if the Hyphenator portion has swallowed one character...
-pParaPortion-GetTextPortions().Remove( nLastPortion );
 if ( nLastPortion  pTP-GetLen() )
 {
 TextPortion* pPrev = 
pParaPortion-GetTextPortions()[nLastPortion - 1];
@@ -2470,7 +2467,7 @@ void ImpEditEngine::RecalcTextPortion( ParaPortion* 
pParaPortion, sal_uInt16 nSt
 pPrev-SetLen( pPrev-GetLen() + pTP-GetLen() );
 pPrev-GetSize().Width() = (-1);
 }
-delete pTP;
+pParaPortion-GetTextPortions().Remove( nLastPortion );
 }
 }
 #if OSL_DEBUG_LEVEL  2
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-11 Thread Kohei Yoshida
 editeng/source/editeng/editdoc.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit a78a7ee9f7b1db56a6fa7e082f410db5a8db2f18
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Apr 11 15:36:00 2012 -0400

Do this special case search only when the array is large enough.

Otherwise the normal linear search should be sufficient.  Eventually
we need to use a better algorithm here than this special case handling...

diff --git a/editeng/source/editeng/editdoc.cxx 
b/editeng/source/editeng/editdoc.cxx
index f797d24..c791aa2 100644
--- a/editeng/source/editeng/editdoc.cxx
+++ b/editeng/source/editeng/editdoc.cxx
@@ -688,7 +688,7 @@ size_t FastGetPos(const _Array rArray, const _Val* p, 
size_t rLastPos)
 // Through certain filter code-paths we do a lot of appends, which in
 // turn call GetPos - creating some N^2 nightmares. If we have a
 // non-trivially large list, do a few checks from the end first.
-if (rLastPos  16)
+if (rLastPos  16  nArrayLen  16)
 {
 size_t nEnd;
 if (rLastPos  nArrayLen - 2)
@@ -698,7 +698,7 @@ size_t FastGetPos(const _Array rArray, const _Val* p, 
size_t rLastPos)
 
 for (size_t nIdx = rLastPos - 2; nIdx  nEnd; ++nIdx)
 {
-if (rArray[nIdx] == p)
+if (rArray.at(nIdx) == p)
 {
 rLastPos = nIdx;
 return nIdx;
@@ -707,7 +707,7 @@ size_t FastGetPos(const _Array rArray, const _Val* p, 
size_t rLastPos)
 }
 // The world's lamest linear search from svarray ...
 for (size_t nIdx = 0; nIdx  nArrayLen; ++nIdx)
-if (rArray[nIdx] == p)
+if (rArray.at(nIdx) == p)
 return rLastPos = nIdx;
 
 // 0x is used to signify not found condition. We need to change this.
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source slideshow/source xmlsecurity/source

2012-04-07 Thread Julien Nabet
 editeng/source/editeng/impedit3.cxx   |2 +-
 slideshow/source/engine/slideshowimpl.cxx |2 +-
 xmlsecurity/source/helper/documentsignaturehelper.cxx |2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit a16060116346533923c607bd406d3fea511b41ec
Author: Julien Nabet serval2...@yahoo.fr
Date:   Sun Apr 8 01:01:31 2012 +0200

Fix Prefer prefix ++/-- operators

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index a380c08..46cfe04 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -2349,7 +2349,7 @@ void ImpEditEngine::CreateTextPortions( ParaPortion* 
pParaPortion, sal_uInt16 r
 DBG_ASSERT( (nInvPos != aPositions.end()), InvPos ?! );
 
 ::std::set sal_uInt32 ::iterator i = nInvPos;
-i++;
+++i;
 while ( i != aPositions.end() )
 {
 TextPortion* pNew = new TextPortion( static_castsal_uInt16(*i++) - 
static_castsal_uInt16(*nInvPos++) );
diff --git a/slideshow/source/engine/slideshowimpl.cxx 
b/slideshow/source/engine/slideshowimpl.cxx
index ca91a6c..7718019 100644
--- a/slideshow/source/engine/slideshowimpl.cxx
+++ b/slideshow/source/engine/slideshowimpl.cxx
@@ -902,7 +902,7 @@ PolygonMap::iterator SlideShowImpl::findPolygons( 
uno::Referencedrawing::XDrawP
 if(aIter-first == xDrawPage)
 bFound = true;
 else
-aIter++;
+++aIter;
 }
 
 return aIter;
diff --git a/xmlsecurity/source/helper/documentsignaturehelper.cxx 
b/xmlsecurity/source/helper/documentsignaturehelper.cxx
index 7523bf1..8b10f03 100644
--- a/xmlsecurity/source/helper/documentsignaturehelper.cxx
+++ b/xmlsecurity/source/helper/documentsignaturehelper.cxx
@@ -374,7 +374,7 @@ bool DocumentSignatureHelper::checkIfAllFilesAreSigned(
 
 //find the file in the element list
 typedef ::std::vector ::rtl::OUString ::const_iterator CIT;
-for (CIT aIter = sElementList.begin(); aIter  sElementList.end(); 
aIter++)
+for (CIT aIter = sElementList.begin(); aIter != 
sElementList.end(); ++aIter)
 {
 ::rtl::OUString sElementListURI = *aIter;
 if (alg == OOo2Document)
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-04 Thread Kohei Yoshida
 editeng/source/editeng/editdbg.cxx |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit e0fd2c61e47c994d050d9afcb9578e096e1489b6
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Apr 4 16:22:32 2012 -0400

dbgutil fix.

diff --git a/editeng/source/editeng/editdbg.cxx 
b/editeng/source/editeng/editdbg.cxx
index 9142ec1..c1ef702 100644
--- a/editeng/source/editeng/editdbg.cxx
+++ b/editeng/source/editeng/editdbg.cxx
@@ -334,13 +334,14 @@ void EditDbg::ShowEditEngineData( EditEngine* pEE, 
sal_Bool bInfoBox )
 fprintf( fp, 
\n
 );
 for ( sal_uInt16 nPortion = 0; nPortion  
pEE-pImpEditEngine-GetParaPortions(). Count(); nPortion++)
 {
-
 ParaPortion* pPPortion = 
pEE-pImpEditEngine-GetParaPortions()[nPortion];
-fprintf( fp, \nParagraph %i: Length = %i, Invalid = %i\nText = '%s', 
nPortion, pPPortion-GetNode()-Len(), pPPortion-IsInvalid(), 
rtl::OUStringToOString( *pPPortion-GetNode(), RTL_TEXTENCODING_ASCII_US 
).getStr() );
+fprintf( fp, \nParagraph %i: Length = %i, Invalid = %i\nText = '%s',
+ nPortion, pPPortion-GetNode()-Len(), pPPortion-IsInvalid(),
+ rtl::OUStringToOString(pPPortion-GetNode()-GetString(), 
RTL_TEXTENCODING_UTF8).getStr() );
 fprintf( fp, \nVorlage: );
 SfxStyleSheet* pStyle = pPPortion-GetNode()-GetStyleSheet();
 if ( pStyle )
-fprintf( fp,  %s, rtl::OUStringToOString( pStyle-GetName(), 
RTL_TEXTENCODING_ASCII_US ).getStr() );
+fprintf( fp,  %s, rtl::OUStringToOString( pStyle-GetName(), 
RTL_TEXTENCODING_UTF8).getStr() );
 fprintf( fp, \nParagraph attribute: );
 DbgOutItemSet( fp, 
pPPortion-GetNode()-GetContentAttribs().GetItems(), sal_False, sal_False );
 
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-03 Thread Kohei Yoshida
 editeng/source/editeng/editdbg.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 2c4e811049e9f2cad823e372a01a5afc09ead25c
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Apr 3 11:44:06 2012 -0400

Fix the dbgutil build.

diff --git a/editeng/source/editeng/editdbg.cxx 
b/editeng/source/editeng/editdbg.cxx
index 48f6e00..9142ec1 100644
--- a/editeng/source/editeng/editdbg.cxx
+++ b/editeng/source/editeng/editdbg.cxx
@@ -384,7 +384,7 @@ void EditDbg::ShowEditEngineData( EditEngine* pEE, sal_Bool 
bInfoBox )
 sal_uLong n = 0;
 for ( z = 0; z  nTextPortions; z++ )
 {
-TextPortion* pPortion = pPPortion-GetTextPortions().GetObject( z 
);
+TextPortion* pPortion = pPPortion-GetTextPortions()[z];
 aPortionStr.append(' ');
 aPortionStr.append(static_castsal_Int32(pPortion-GetLen()));
 aPortionStr.append('(');
@@ -409,7 +409,7 @@ void EditDbg::ShowEditEngineData( EditEngine* pEE, sal_Bool 
bInfoBox )
 sal_uInt16 nLine;
 for ( nLine = 0; nLine  pPPortion-GetLines().Count(); nLine++ )
 {
-EditLine* pLine = pPPortion-GetLines().GetObject( nLine );
+EditLine* pLine = pPPortion-GetLines()[nLine];
 
 rtl::OString 
aLine(rtl::OUStringToOString(pPPortion-GetNode()-Copy(pLine-GetStart(), 
pLine-GetEnd() - pLine-GetStart()), RTL_TEXTENCODING_ASCII_US));
 fprintf( fp, \nLine %i\t%s, nLine, aLine.getStr() );
@@ -417,7 +417,7 @@ void EditDbg::ShowEditEngineData( EditEngine* pEE, sal_Bool 
bInfoBox )
 // then the internal data ...
 for ( nLine = 0; nLine  pPPortion-GetLines().Count(); nLine++ )
 {
-EditLine* pLine = pPPortion-GetLines().GetObject( nLine );
+EditLine* pLine = pPPortion-GetLines()[nLine];
 fprintf( fp, \nZeile %i:\tStart: %i,\tEnd: %i, nLine, 
pLine-GetStart(), pLine-GetEnd() );
 fprintf( fp, \t\tPortions: %i - %i.\tHight: %i, Ascent=%i, 
pLine-GetStartPortion(), pLine-GetEndPortion(), pLine-GetHeight(), 
pLine-GetMaxAscent() );
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-03 Thread Kohei Yoshida
 editeng/source/editeng/impedit2.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 97bcf0b645fa5ce14cfd806548c6805952fd3e76
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Apr 3 11:51:18 2012 -0400

WaE: for Norbert's Mac OS tinderbox.

diff --git a/editeng/source/editeng/impedit2.cxx 
b/editeng/source/editeng/impedit2.cxx
index 6a01a0f..ec74f29 100644
--- a/editeng/source/editeng/impedit2.cxx
+++ b/editeng/source/editeng/impedit2.cxx
@@ -3965,7 +3965,7 @@ long ImpEditEngine::GetXPos(
 // End of Portion
 if ( pPortion-GetKind() == PORTIONKIND_TAB )
 {
-if ( (nTextPortion+1)  
pParaPortion-GetTextPortions().Count() )
+if ( static_castsize_t(nTextPortion+1)  
pParaPortion-GetTextPortions().Count() )
 {
 const TextPortion* pNextPortion = 
pParaPortion-GetTextPortions()[nTextPortion+1];
 if ( pNextPortion-GetKind() != PORTIONKIND_TAB )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-03 Thread Kohei Yoshida
 editeng/source/editeng/impedit3.cxx |2 +-
 editeng/source/editeng/impedit4.cxx |8 
 2 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 881fa8d5cbb937b033f87f7da813419895b31b28
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Apr 3 13:15:57 2012 -0400

More dbgutil build fix.

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 091c329..83396c9 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -1697,7 +1697,7 @@ void ImpEditEngine::CreateAndInsertEmptyLine( 
ParaPortion* pParaPortion, sal_uIn
 {
 // -2: The new one is already inserted.
 #ifdef DBG_UTIL
-EditLine* pLastLine = pParaPortion-GetLines().GetObject( 
pParaPortion-GetLines().Count()-2 );
+EditLine* pLastLine = 
pParaPortion-GetLines()[pParaPortion-GetLines().Count()-2];
 DBG_ASSERT( pLastLine, soft wrap no line?! );
 DBG_ASSERT( pLastLine-GetEnd() == pParaPortion-GetNode()-Len(), 
different anyway? );
 #endif
diff --git a/editeng/source/editeng/impedit4.cxx 
b/editeng/source/editeng/impedit4.cxx
index dd45a57..63b9402 100644
--- a/editeng/source/editeng/impedit4.cxx
+++ b/editeng/source/editeng/impedit4.cxx
@@ -1167,9 +1167,9 @@ EditTextObject* ImpEditEngine::CreateBinTextObject( 
EditSelection aSel, SfxItemP
 sal_uInt16 nTest;
 int nTPLen = 0, nTxtLen = 0;
 for ( nTest = pParaPortion-GetTextPortions().Count(); nTest; )
-nTPLen += pParaPortion-GetTextPortions().GetObject( --nTest 
)-GetLen();
+nTPLen += pParaPortion-GetTextPortions()[--nTest]-GetLen();
 for ( nTest = pParaPortion-GetLines().Count(); nTest; )
-nTxtLen += pParaPortion-GetLines().GetObject( --nTest 
)-GetLen();
+nTxtLen += pParaPortion-GetLines()[--nTest]-GetLen();
 DBG_ASSERT( ( nTPLen == pParaPortion-GetNode()-Len() )  ( 
nTxtLen == pParaPortion-GetNode()-Len() ), CreateBinTextObject: ParaPortion 
not completely formatted! );
 #endif
 }
@@ -1363,9 +1363,9 @@ EditSelection ImpEditEngine::InsertBinTextObject( 
BinTextObject rTextObject, Ed
 sal_uInt16 nTest;
 int nTPLen = 0, nTxtLen = 0;
 for ( nTest = pParaPortion-GetTextPortions().Count(); nTest; )
-nTPLen += pParaPortion-GetTextPortions().GetObject( 
--nTest )-GetLen();
+nTPLen += 
pParaPortion-GetTextPortions()[--nTest]-GetLen();
 for ( nTest = pParaPortion-GetLines().Count(); nTest; )
-nTxtLen += pParaPortion-GetLines().GetObject( --nTest 
)-GetLen();
+nTxtLen += pParaPortion-GetLines()[--nTest]-GetLen();
 DBG_ASSERT( ( nTPLen == pParaPortion-GetNode()-Len() )  ( 
nTxtLen == pParaPortion-GetNode()-Len() ), InsertBinTextObject: ParaPortion 
not completely formatted! );
 #endif
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-03 Thread Kohei Yoshida
 editeng/source/editeng/impedit.cxx |2 --
 1 file changed, 2 deletions(-)

New commits:
commit e7e882715872303dc8feabc899ceab572d1c2337
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Apr 3 13:53:15 2012 -0400

Unused local variables.

diff --git a/editeng/source/editeng/impedit.cxx 
b/editeng/source/editeng/impedit.cxx
index b01c830..d3268ce 100644
--- a/editeng/source/editeng/impedit.cxx
+++ b/editeng/source/editeng/impedit.cxx
@@ -491,7 +491,6 @@ void ImpEditView::ResetOutputArea( const Rectangle rRec )
 
 void ImpEditView::RecalcOutputArea()
 {
-Rectangle aOldArea( aOutArea );
 Point aNewTopLeft( aOutArea.TopLeft() );
 Size aNewSz( aOutArea.GetSize() );
 
@@ -886,7 +885,6 @@ Pair ImpEditView::Scroll( long ndX, long ndY, sal_uInt8 
nRangeCheck )
 #endif
 
 Rectangle aNewVisArea( GetVisDocArea() );
-Size aPaperSz( pEditEngine-pImpEditEngine-GetPaperSize() );
 
 // Vertical:
 if ( !IsVertical() )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-03 Thread Kohei Yoshida
 editeng/source/editeng/impedit3.cxx |   11 ---
 1 file changed, 8 insertions(+), 3 deletions(-)

New commits:
commit e5d378b1ba04c470e0c6c16b48af97ab5818e2bb
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Apr 3 18:26:29 2012 -0400

Check for empty-ness of vector before accessing the first element.

I hope this will keep Stephan's tinderbox happy.  If not...

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 83396c9..4e9c393 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -1241,7 +1241,9 @@ sal_Bool ImpEditEngine::CreateLines( sal_uInt16 nPara, 
sal_uInt32 nStartPosY )
 if ( bCompressedChars  pPortion  ( pPortion-GetLen()  1 )  
pPortion-GetExtraInfos()  pPortion-GetExtraInfos()-bCompressed )
 {
 // I need the manipulated DXArray for determining the break 
postion...
-sal_Int32* pDXArray = pLine-GetCharPosArray()[0] + 
(nPortionStart - pLine-GetStart());
+sal_Int32* pDXArray = NULL;
+if (!pLine-GetCharPosArray().empty())
+pDXArray = pLine-GetCharPosArray()[0] + (nPortionStart - 
pLine-GetStart());
 ImplCalcAsianCompression(
 pNode, pPortion, nPortionStart, pDXArray, 1, true);
 }
@@ -2997,7 +2999,8 @@ void ImpEditEngine::Paint( OutputDevice* pOutDev, 
Rectangle aClipRec, Point aSta
 aText = *pPortion-GetNode();
 nTextStart = nIndex;
 nTextLen = pTextPortion-GetLen();
-pDXArray = pLine-GetCharPosArray()[0]+( 
nIndex-pLine-GetStart() );
+if (!pLine-GetCharPosArray().empty())
+pDXArray = 
pLine-GetCharPosArray()[0]+( nIndex-pLine-GetStart() );
 
 // Paint control characters (#i55716#)
 if ( aStatus.MarkFields() )
@@ -4457,7 +4460,9 @@ void ImpEditEngine::ImplExpandCompressedPortions( 
EditLine* pLine, ParaPortion*
 size_t nTxtPortion = pParaPortion-GetTextPortions().GetPos( 
pTP );
 sal_uInt16 nTxtPortionStart = 
pParaPortion-GetTextPortions().GetStartPos( nTxtPortion );
 DBG_ASSERT( nTxtPortionStart = pLine-GetStart(), Portion 
doesn't belong to the line!!! );
-sal_Int32* pDXArray = pLine-GetCharPosArray()[0]+( 
nTxtPortionStart-pLine-GetStart() );
+sal_Int32* pDXArray = NULL;
+if (!pLine-GetCharPosArray().empty())
+pDXArray = pLine-GetCharPosArray()[0]+( 
nTxtPortionStart-pLine-GetStart() );
 if ( pTP-GetExtraInfos()-pOrgDXArray )
 memcpy( pDXArray, pTP-GetExtraInfos()-pOrgDXArray, 
(pTP-GetLen()-1)*sizeof(sal_Int32) );
 ImplCalcAsianCompression( pParaPortion-GetNode(), pTP, 
nTxtPortionStart, pDXArray, (sal_uInt16)nCompressPercent, sal_True );
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-04-03 Thread Kohei Yoshida
 editeng/source/editeng/impedit2.cxx |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 71da75abda60172ee9b8f42c1bf5de2f48a731b9
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Apr 3 18:45:55 2012 -0400

I need to check for empty-ness here too.

diff --git a/editeng/source/editeng/impedit2.cxx 
b/editeng/source/editeng/impedit2.cxx
index ec74f29..ecb4a89 100644
--- a/editeng/source/editeng/impedit2.cxx
+++ b/editeng/source/editeng/impedit2.cxx
@@ -4021,7 +4021,9 @@ long ImpEditEngine::GetXPos(
 if ( nType == CHAR_PUNCTUATIONRIGHT )
 {
 sal_uInt16 n = nIndex - nTextPortionStart;
-const sal_Int32* pDXArray = 
pLine-GetCharPosArray()[0]+( nTextPortionStart-pLine-GetStart() );
+const sal_Int32* pDXArray = NULL;
+if (!pLine-GetCharPosArray().empty())
+pDXArray = pLine-GetCharPosArray()[0]+( 
nTextPortionStart-pLine-GetStart() );
 sal_Int32 nCharWidth = ( ( (n+1)  
pPortion-GetLen() ) ? pDXArray[n] : pPortion-GetSize().Width() )
 - ( n ? 
pDXArray[n-1] : 0 );
 if ( (n+1)  pPortion-GetLen() )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-03-30 Thread Lubos Lunak
 editeng/source/editeng/editattr.cxx |   10 --
 editeng/source/editeng/editattr.hxx |4 +---
 editeng/source/editeng/editdoc.cxx  |2 +-
 3 files changed, 2 insertions(+), 14 deletions(-)

New commits:
commit d52d301660df47527a69c12bca49bb4e0199b4f5
Author: Luboš Luňák l.lu...@suse.cz
Date:   Fri Mar 30 16:36:01 2012 +0200

avoid sc crashes

Boost is reportedly 'expertly designed C++ library', so why do I
find the idea of a container containing pointers but exposing them
by value to be rather retarded? I don't know why exactly this
crashes, but I expect std::sort() either either does unwanted
copies that break things or the copies slice off inherited classes.

diff --git a/editeng/source/editeng/editattr.cxx 
b/editeng/source/editeng/editattr.cxx
index 171eb36..2e79644 100644
--- a/editeng/source/editeng/editattr.cxx
+++ b/editeng/source/editeng/editattr.cxx
@@ -66,12 +66,6 @@ EditAttrib::EditAttrib( const SfxPoolItem rAttr )
 pItem = rAttr;
 }
 
-EditAttrib::EditAttrib(const EditAttrib r) :
-pItem(r.pItem)
-{
-DBG_CTOR( EE_EditAttrib, 0 );
-}
-
 EditAttrib::~EditAttrib()
 {
 DBG_DTOR( EE_EditAttrib, 0 );
@@ -88,10 +82,6 @@ EditCharAttrib::EditCharAttrib( const SfxPoolItem rAttr, 
sal_uInt16 nS, sal_uIn
 DBG_ASSERT( ( rAttr.Which()  EE_FEATURE_START ) || ( rAttr.Which()  
EE_FEATURE_END ) || ( nE == (nS+1) ), EditCharAttrib CTOR: Invalid feature! );
 }
 
-EditCharAttrib::EditCharAttrib(const EditCharAttrib r) :
-EditAttrib(r),
-nStart(r.nStart), nEnd(r.nEnd), bFeature(r.bFeature), bEdge(r.bEdge) {}
-
 void EditCharAttrib::SetFont( SvxFont, OutputDevice* )
 {
 }
diff --git a/editeng/source/editeng/editattr.hxx 
b/editeng/source/editeng/editattr.hxx
index 0ad35a2..5855e47 100644
--- a/editeng/source/editeng/editattr.hxx
+++ b/editeng/source/editeng/editattr.hxx
@@ -68,7 +68,7 @@ class SfxVoidItem;
 // -
 // class EditAttrib
 // -
-class EditAttrib
+class EditAttrib : public boost::noncopyable
 {
 private:
 EditAttrib();
@@ -77,7 +77,6 @@ protected:
 const SfxPoolItem*  pItem;
 
 EditAttrib( const SfxPoolItem rAttr );
-EditAttrib(const EditAttrib r);
 virtual ~EditAttrib();
 
 public:
@@ -104,7 +103,6 @@ protected:
 
 public:
 EditCharAttrib( const SfxPoolItem rAttr, sal_uInt16 nStart, sal_uInt16 
nEnd );
-EditCharAttrib(const EditCharAttrib r);
 
 sal_uInt16 GetStart()  { return nStart; }
 sal_uInt16 GetEnd(){ return nEnd; }
diff --git a/editeng/source/editeng/editdoc.cxx 
b/editeng/source/editeng/editdoc.cxx
index 6b35b47..86a709b 100644
--- a/editeng/source/editeng/editdoc.cxx
+++ b/editeng/source/editeng/editdoc.cxx
@@ -2005,7 +2005,7 @@ void CharAttribList::InsertAttrib( EditCharAttrib* 
pAttrib )
 
 void CharAttribList::ResortAttribs()
 {
-std::sort(aAttribs.begin(), aAttribs.end(), LessByStart());
+aAttribs.sort(LessByStart());
 }
 
 void CharAttribList::OptimizeRanges( SfxItemPool rItemPool )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-03-30 Thread Lubos Lunak
 editeng/source/editeng/editdoc.cxx  |9 -
 editeng/source/editeng/editdoc.hxx  |4 
 editeng/source/editeng/editdoc2.cxx |8 
 editeng/source/editeng/impedit2.cxx |7 ---
 4 files changed, 24 insertions(+), 4 deletions(-)

New commits:
commit 46da0b6837bffa7d8a72624d2587159737e3d7e2
Author: Luboš Luňák l.lu...@suse.cz
Date:   Fri Mar 30 23:32:21 2012 +0200

further sc/editeng unbreaking

this boost::ptr_vector idea is seriously broken

diff --git a/editeng/source/editeng/editdoc.cxx 
b/editeng/source/editeng/editdoc.cxx
index 51f0112..c05e379 100644
--- a/editeng/source/editeng/editdoc.cxx
+++ b/editeng/source/editeng/editdoc.cxx
@@ -1816,7 +1816,7 @@ void EditDoc::InsertAttrib( ContentNode* pNode, 
sal_uInt16 nStart, sal_uInt16 nE
 if ( pAttr )
 {
 // Remove attribute
-rAttrList.Remove(pAttr);
+rAttrList.Release(pAttr);
 }
 
 // check whether 'the same' attribute exist at this place.
@@ -2137,6 +2137,13 @@ void CharAttribList::Remove(size_t nPos)
 aAttribs.erase(aAttribs.begin()+nPos);
 }
 
+void CharAttribList::Release(const EditCharAttrib* p)
+{
+AttribsType::iterator it = std::find_if(aAttribs.begin(), aAttribs.end(), 
FindByAddress(p));
+if (it != aAttribs.end())
+aAttribs.release(it).release();
+}
+
 void CharAttribList::SetHasEmptyAttribs(bool b)
 {
 bHasEmptyAttribs = b;
diff --git a/editeng/source/editeng/editdoc.hxx 
b/editeng/source/editeng/editdoc.hxx
index bf00c76..609e2bd 100644
--- a/editeng/source/editeng/editdoc.hxx
+++ b/editeng/source/editeng/editdoc.hxx
@@ -253,6 +253,7 @@ public:
 
 void Remove(const EditCharAttrib* p);
 void Remove(size_t nPos);
+void Release(const EditCharAttrib* p);
 
 #if OSL_DEBUG_LEVEL  2
 // Debug:
@@ -315,7 +316,10 @@ public:
 ContentNode* operator[](size_t nPos);
 
 void Insert(size_t nPos, ContentNode* p);
+/// deletes
 void Remove(size_t nPos);
+/// does not delete
+void Release(size_t nPos);
 size_t Count() const;
 void Clear();
 };
diff --git a/editeng/source/editeng/editdoc2.cxx 
b/editeng/source/editeng/editdoc2.cxx
index b22c10b..fc65c9c 100644
--- a/editeng/source/editeng/editdoc2.cxx
+++ b/editeng/source/editeng/editdoc2.cxx
@@ -419,6 +419,14 @@ void ContentList::Remove(size_t nPos)
 maContents.erase(maContents.begin() + nPos);
 }
 
+void ContentList::Release(size_t nPos)
+{
+if (nPos = maContents.size())
+return;
+
+maContents.release(maContents.begin() + nPos).release();
+}
+
 size_t ContentList::Count() const
 {
 return maContents.size();
diff --git a/editeng/source/editeng/impedit2.cxx 
b/editeng/source/editeng/impedit2.cxx
index a4654f0..9bab5db 100644
--- a/editeng/source/editeng/impedit2.cxx
+++ b/editeng/source/editeng/impedit2.cxx
@@ -2192,7 +2192,7 @@ EditSelection ImpEditEngine::ImpMoveParagraphs( Range 
aOldPositions, sal_uInt16
 {
 // always aOldPositions.Min(), since Remove().
 ParaPortion* pTmpPortion = 
GetParaPortions().Release(aOldPositions.Min());
-aEditDoc.Remove( (sal_uInt16)aOldPositions.Min() );
+aEditDoc.Release( (sal_uInt16)aOldPositions.Min() );
 aTmpPortionList.Append(pTmpPortion);
 }
 
@@ -2237,7 +2237,8 @@ EditSelection ImpEditEngine::ImpMoveParagraphs( Range 
aOldPositions, sal_uInt16
 if ( pRecalc4 )
 CalcHeight( pRecalc4 );
 
-aTmpPortionList.Reset();
+while( aTmpPortionList.Count()  0 )
+aTmpPortionList.Release( aTmpPortionList.Count() - 1 );
 
 #if OSL_DEBUG_LEVEL  2
 GetParaPortions().DbgCheck(aEditDoc);
@@ -2484,7 +2485,7 @@ void ImpEditEngine::ImpRemoveParagraph( sal_uInt16 nPara )
 aDeletedNodes.Insert( pInf, aDeletedNodes.Count() );
 
 // The node is managed by the undo and possibly destroyed!
-aEditDoc.Remove( nPara );
+aEditDoc.Release( nPara );
 GetParaPortions().Remove( nPara );
 
 if ( IsCallParaInsertedOrDeleted() )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-03-29 Thread Kohei Yoshida
 editeng/source/editeng/impedit2.cxx |5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

New commits:
commit b512c3ca29735f15383c5f6215348fccfaddd40e
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Thu Mar 29 02:00:02 2012 -0400

No need to delete ParaPortion outside of the container any more.

diff --git a/editeng/source/editeng/impedit2.cxx 
b/editeng/source/editeng/impedit2.cxx
index 04cebce..e7d2bbc 100644
--- a/editeng/source/editeng/impedit2.cxx
+++ b/editeng/source/editeng/impedit2.cxx
@@ -2477,18 +2477,15 @@ void ImpEditEngine::ImpRemoveParagraph( sal_uInt16 
nPara )
 {
 ContentNode* pNode = aEditDoc.SaveGetObject( nPara );
 ContentNode* pNextNode = aEditDoc.SaveGetObject( nPara+1 );
-ParaPortion* pPortion = GetParaPortions().SaveGetObject( nPara );
 
 OSL_ENSURE( pNode, Blind Node in ImpRemoveParagraph );
-OSL_ENSURE( pPortion, Blind Portion in ImpRemoveParagraph(2) );
 
 DeletedNodeInfo* pInf = new DeletedNodeInfo( (sal_uLong)pNode, nPara );
 aDeletedNodes.Insert( pInf, aDeletedNodes.Count() );
 
 // The node is managed by the undo and possibly destroyed!
-/* delete */ aEditDoc.Remove( nPara );
+aEditDoc.Remove( nPara );
 GetParaPortions().Remove( nPara );
-delete pPortion;
 
 if ( IsCallParaInsertedOrDeleted() )
 {
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-03-29 Thread Stephan Bergmann
 editeng/source/editeng/editeng.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit a159dfbea39dd3470837e73bf2df4cd2a0f3e806
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Mar 29 11:04:03 2012 +0200

-Werror=sign-compare

...solved by recasting the offending expression.

diff --git a/editeng/source/editeng/editeng.cxx 
b/editeng/source/editeng/editeng.cxx
index 2b246c9..4f9184f 100644
--- a/editeng/source/editeng/editeng.cxx
+++ b/editeng/source/editeng/editeng.cxx
@@ -1320,7 +1320,7 @@ EditTextObject* EditEngine::CreateTextObject( sal_uInt16 
nPara, sal_uInt16 nPara
 {
 DBG_CHKTHIS( EditEngine, 0 );
 DBG_ASSERT( nPara  pImpEditEngine-GetEditDoc().Count(), 
CreateTextObject: Startpara out of Range );
-DBG_ASSERT( nPara+nParas-1  pImpEditEngine-GetEditDoc().Count(), 
CreateTextObject: Endpara out of Range );
+DBG_ASSERT( nParas = pImpEditEngine-GetEditDoc().Count() - nPara, 
CreateTextObject: Endpara out of Range );
 
 ContentNode* pStartNode = pImpEditEngine-GetEditDoc().SafeGetObject( 
nPara );
 ContentNode* pEndNode = pImpEditEngine-GetEditDoc().SafeGetObject( 
nPara+nParas-1 );
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source filter/source sc/source starmath/source svtools/inc toolkit/source

2012-03-29 Thread Michael Meeks
 editeng/source/editeng/editdoc.hxx  |1 -
 editeng/source/misc/svxacorr.cxx|1 -
 filter/source/xsltdialog/xmlfileview.hxx|1 -
 sc/source/filter/html/htmlpars.cxx  |1 -
 starmath/source/math_pch.cxx|1 -
 svtools/inc/svtools/treelist.hxx|2 +-
 toolkit/source/controls/unocontrol.cxx  |1 -
 toolkit/source/controls/unocontrolmodel.cxx |1 -
 8 files changed, 1 insertion(+), 8 deletions(-)

New commits:
commit 82f5a3b5df000966d344b26fe36e776bc947c6ef
Author: Noel Grandin n...@peralex.com
Date:   Thu Mar 29 16:49:07 2012 +0200

Remove unnecessary tools/table.hxx #includes

diff --git a/editeng/source/editeng/editdoc.hxx 
b/editeng/source/editeng/editdoc.hxx
index 59010d6..bf00c76 100644
--- a/editeng/source/editeng/editdoc.hxx
+++ b/editeng/source/editeng/editdoc.hxx
@@ -37,7 +37,6 @@
 #include svl/itemset.hxx
 #include svl/style.hxx
 #include svl/itempool.hxx
-#include tools/table.hxx
 
 #include vector
 #include deque
diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index ac726c8..651b9eb 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -29,7 +29,6 @@
 #include com/sun/star/io/XStream.hpp
 #include com/sun/star/lang/Locale.hpp
 #include tools/urlobj.hxx
-#include tools/table.hxx
 #include i18npool/mslangid.hxx
 #include vcl/svapp.hxx
 #include sot/storinfo.hxx
diff --git a/filter/source/xsltdialog/xmlfileview.hxx 
b/filter/source/xsltdialog/xmlfileview.hxx
index 4ff36a4..5623fab 100644
--- a/filter/source/xsltdialog/xmlfileview.hxx
+++ b/filter/source/xsltdialog/xmlfileview.hxx
@@ -32,7 +32,6 @@
 #include svl/lstner.hxx
 #include vcl/timer.hxx
 
-#include tools/table.hxx
 #include svtools/xtextedt.hxx
 #include vcl/timer.hxx
 #include vcl/wrkwin.hxx
diff --git a/sc/source/filter/html/htmlpars.cxx 
b/sc/source/filter/html/htmlpars.cxx
index aaae381..019d057 100644
--- a/sc/source/filter/html/htmlpars.cxx
+++ b/sc/source/filter/html/htmlpars.cxx
@@ -58,7 +58,6 @@
 #include vcl/svapp.hxx
 #include tools/urlobj.hxx
 #include tools/tenccvt.hxx
-#include tools/table.hxx
 
 #include htmlpars.hxx
 #include global.hxx
diff --git a/starmath/source/math_pch.cxx b/starmath/source/math_pch.cxx
index ee17d65..945025c 100644
--- a/starmath/source/math_pch.cxx
+++ b/starmath/source/math_pch.cxx
@@ -349,7 +349,6 @@
 #include vcl/metric.hxx
 #include sfx2/inimgr.hxx
 #include node.hxx
-#include tools/table.hxx
 #include sfx2/docfac.hxx
 #include svl/ownlist.hxx
 #include sfx2/objsh.hxx
diff --git a/svtools/inc/svtools/treelist.hxx b/svtools/inc/svtools/treelist.hxx
index b2a7173..c505e27 100644
--- a/svtools/inc/svtools/treelist.hxx
+++ b/svtools/inc/svtools/treelist.hxx
@@ -29,12 +29,12 @@
 #ifndef _SVTREELIST_HXX
 #define _SVTREELIST_HXX
 
+#include limits.h
 #include svtools/svtdllapi.h
 #include tools/solar.h
 #include vector
 #include boost/ptr_container/ptr_map.hpp
 
-#include tools/table.hxx
 #include tools/link.hxx
 #include tools/string.hxx
 #include tools/debug.hxx
diff --git a/toolkit/source/controls/unocontrol.cxx 
b/toolkit/source/controls/unocontrol.cxx
index b845b59..cec8c40 100644
--- a/toolkit/source/controls/unocontrol.cxx
+++ b/toolkit/source/controls/unocontrol.cxx
@@ -39,7 +39,6 @@
 #include rtl/memory.h
 #include rtl/uuid.h
 #include osl/mutex.hxx
-#include tools/table.hxx
 #include tools/date.hxx
 #include tools/debug.hxx
 #include tools/diagnose_ex.h
diff --git a/toolkit/source/controls/unocontrolmodel.cxx 
b/toolkit/source/controls/unocontrolmodel.cxx
index ac7f3ed..8afc19e 100644
--- a/toolkit/source/controls/unocontrolmodel.cxx
+++ b/toolkit/source/controls/unocontrolmodel.cxx
@@ -43,7 +43,6 @@
 #include rtl/memory.h
 #include rtl/uuid.h
 #include tools/diagnose_ex.h
-#include tools/table.hxx
 #include tools/date.hxx
 #include tools/time.hxx
 #include tools/debug.hxx
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-03-28 Thread Kohei Yoshida
 editeng/source/editeng/editdbg.cxx |   12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)

New commits:
commit 736bf5c26f4f220171d95f52b933669c1ff6b172
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Mar 28 14:00:06 2012 -0400

Keep tinderbox happy.

diff --git a/editeng/source/editeng/editdbg.cxx 
b/editeng/source/editeng/editdbg.cxx
index a1b4575..4070002 100644
--- a/editeng/source/editeng/editdbg.cxx
+++ b/editeng/source/editeng/editdbg.cxx
@@ -349,21 +349,21 @@ void EditDbg::ShowEditEngineData( EditEngine* pEE, 
sal_Bool bInfoBox )
 sal_uInt16 z;
 for ( z = 0; z  pPPortion-GetNode()-GetCharAttribs().Count(); z++ )
 {
-EditCharAttrib* pAttr = 
pPPortion-GetNode()-GetCharAttribs().GetAttribs().GetObject( z );
+const EditCharAttrib rAttr = 
pPPortion-GetNode()-GetCharAttribs().GetAttribs()[z];
 rtl::OStringBuffer aCharAttribs;
 aCharAttribs.append(RTL_CONSTASCII_STRINGPARAM(\nA));
 aCharAttribs.append(static_castsal_Int32(nPortion));
 aCharAttribs.append(RTL_CONSTASCII_STRINGPARAM(:  ));
-
aCharAttribs.append(static_castsal_Int32(pAttr-GetItem()-Which()));
+
aCharAttribs.append(static_castsal_Int32(rAttr.GetItem()-Which()));
 aCharAttribs.append('\t');
-aCharAttribs.append(static_castsal_Int32(pAttr-GetStart()));
+aCharAttribs.append(static_castsal_Int32(rAttr.GetStart()));
 aCharAttribs.append('\t');
-aCharAttribs.append(static_castsal_Int32(pAttr-GetEnd()));
-if ( pAttr-IsEmpty() )
+aCharAttribs.append(static_castsal_Int32(rAttr.GetEnd()));
+if ( rAttr.IsEmpty() )
 bZeroAttr = sal_True;
 fprintf(fp, %s = , aCharAttribs.getStr());
 
-rtl::OString aDebStr = DbgOutItem( rPool, *pAttr-GetItem() );
+rtl::OString aDebStr = DbgOutItem( rPool, *rAttr.GetItem() );
 fprintf( fp, %s, aDebStr.getStr() );
 }
 if ( bZeroAttr )
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source sc/source

2012-03-26 Thread Kohei Yoshida
 editeng/source/items/flditem.cxx |   16 
 sc/source/ui/app/scdll.cxx   |6 +++---
 2 files changed, 11 insertions(+), 11 deletions(-)

New commits:
commit df827af160fa255c56b9aaef644a2d9c4680977d
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Tue Mar 27 01:34:35 2012 -0400

Let's not use a macro for this...

diff --git a/editeng/source/items/flditem.cxx b/editeng/source/items/flditem.cxx
index a182a36..58f6ab1 100644
--- a/editeng/source/items/flditem.cxx
+++ b/editeng/source/items/flditem.cxx
@@ -948,14 +948,14 @@ SvClassManager SvxFieldItem::GetClassManager()
 if ( !pClassMgr )
 {
 pClassMgr = new SvClassManager;
-pClassMgr-SV_CLASS_REGISTER( SvxFieldData );
-pClassMgr-SV_CLASS_REGISTER( SvxURLField );
-pClassMgr-SV_CLASS_REGISTER( SvxDateField );
-pClassMgr-SV_CLASS_REGISTER( SvxPageField );
-pClassMgr-SV_CLASS_REGISTER( SvxTimeField );
-pClassMgr-SV_CLASS_REGISTER( SvxExtTimeField );
-pClassMgr-SV_CLASS_REGISTER( SvxExtFileField );
-pClassMgr-SV_CLASS_REGISTER( SvxAuthorField );
+pClassMgr-Register(SvxFieldData::StaticClassId(),
SvxFieldData::CreateInstance);
+pClassMgr-Register(SvxURLField::StaticClassId(), 
SvxURLField::CreateInstance);
+pClassMgr-Register(SvxDateField::StaticClassId(),
SvxDateField::CreateInstance);
+pClassMgr-Register(SvxPageField::StaticClassId(),
SvxPageField::CreateInstance);
+pClassMgr-Register(SvxTimeField::StaticClassId(),
SvxTimeField::CreateInstance);
+pClassMgr-Register(SvxExtTimeField::StaticClassId(), 
SvxExtTimeField::CreateInstance);
+pClassMgr-Register(SvxExtFileField::StaticClassId(), 
SvxExtFileField::CreateInstance);
+pClassMgr-Register(SvxAuthorField::StaticClassId(),  
SvxAuthorField::CreateInstance);
 }
 
 return *pClassMgr;
diff --git a/sc/source/ui/app/scdll.cxx b/sc/source/ui/app/scdll.cxx
index 25712d4..2d9af16 100644
--- a/sc/source/ui/app/scdll.cxx
+++ b/sc/source/ui/app/scdll.cxx
@@ -295,9 +295,9 @@ void ScDLL::Init()
 //  Edit-Engine-Felder, soweit nicht schon in OfficeApplication::Init
 
 SvClassManager rClassManager = SvxFieldItem::GetClassManager();
-rClassManager.SV_CLASS_REGISTER( SvxPagesField );
-rClassManager.SV_CLASS_REGISTER( SvxFileField );
-rClassManager.SV_CLASS_REGISTER( SvxTableField );
+rClassManager.Register(SvxPagesField::StaticClassId(), 
SvxPagesField::CreateInstance);
+rClassManager.Register(SvxFileField::StaticClassId(),  
SvxFileField::CreateInstance);
+rClassManager.Register(SvxTableField::StaticClassId(), 
SvxTableField::CreateInstance);
 
 SdrRegisterFieldClasses();  // SvDraw-Felder registrieren
 
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-03-13 Thread Caolán McNamara
 editeng/source/items/frmitems.cxx |   30 ++
 1 file changed, 14 insertions(+), 16 deletions(-)

New commits:
commit ffc3c27130faec6c5337943975fa6ebf7623de36
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Mar 13 12:58:07 2012 +

Conditional jump or move depends on uninitialised value

diff --git a/editeng/source/items/frmitems.cxx 
b/editeng/source/items/frmitems.cxx
index f38ba5d..c2dbce8 100644
--- a/editeng/source/items/frmitems.cxx
+++ b/editeng/source/items/frmitems.cxx
@@ -792,28 +792,26 @@ bool SvxLRSpaceItem::HasMetrics() const
 
 // class SvxULSpaceItem --
 
-SvxULSpaceItem::SvxULSpaceItem( const sal_uInt16 nId ) :
-
-SfxPoolItem( nId ),
-
-nUpper( 0 ),
-nLower( 0 ),
-nPropUpper( 100 ),
-nPropLower( 100 )
+SvxULSpaceItem::SvxULSpaceItem( const sal_uInt16 nId )
+: SfxPoolItem(nId)
+, nUpper(0)
+, nLower(0)
+, bContext(false)
+, nPropUpper(100)
+, nPropLower(100)
 {
 }
 
 // ---
 
 SvxULSpaceItem::SvxULSpaceItem( const sal_uInt16 nUp, const sal_uInt16 nLow,
-const sal_uInt16 nId ) :
-
-SfxPoolItem( nId ),
-
-nUpper( nUp  ),
-nLower( nLow ),
-nPropUpper( 100 ),
-nPropLower( 100 )
+const sal_uInt16 nId )
+: SfxPoolItem(nId)
+, nUpper(nUp)
+, nLower(nLow)
+, bContext(false)
+, nPropUpper(100)
+, nPropLower(100)
 {
 }
 
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-03-12 Thread Ivan Timofeev
 editeng/source/editeng/impedit.hxx  |6 ++--
 editeng/source/editeng/impedit3.cxx |   50 ++--
 editeng/source/editeng/impedit4.cxx |4 +-
 3 files changed, 25 insertions(+), 35 deletions(-)

New commits:
commit f2d0fcc26be481c2f872056fb3b8402169d124d8
Author: Bartosz Kosiorek gan...@poczta.onet.pl
Date:   Sat Mar 10 22:16:16 2012 +0100

Replace VARARR_SORT with std

diff --git a/editeng/source/editeng/impedit.hxx 
b/editeng/source/editeng/impedit.hxx
index 2076d29..5196703 100644
--- a/editeng/source/editeng/impedit.hxx
+++ b/editeng/source/editeng/impedit.hxx
@@ -905,7 +905,7 @@ public:
 LanguageTypeGetDefaultLanguage() const { return eDefLanguage; }
 
 
-LanguageTypeGetLanguage( const EditSelection rSelection ) const;
+LanguageTypeGetLanguage( const EditSelection rSelection ) const;
 LanguageTypeGetLanguage( const EditPaM rPaM, sal_uInt16* pEndPos 
= NULL ) const;
 ::com::sun::star::lang::Locale GetLocale( const EditPaM rPaM ) const;
 
@@ -945,12 +945,12 @@ public:
 //adds one or more portions of text to the SpellPortions depending on 
language changes
 voidAddPortionIterated(
 EditView rEditView,
-const EditSelection rSel,
+const EditSelection rSel,
 ::com::sun::star::uno::Reference 
::com::sun::star::linguistic2::XSpellAlternatives  xAlt,
 ::svx::SpellPortions rToFill);
 //adds one portion to the SpellPortions
 voidAddPortion(
-const EditSelection rSel,
+const EditSelection rSel,
 ::com::sun::star::uno::Reference 
::com::sun::star::linguistic2::XSpellAlternatives  xAlt,
 ::svx::SpellPortions rToFill,
 bool bIsField );
diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index d37d8d9..a7919a2 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -68,6 +68,7 @@
 
 #include editeng/unolingu.hxx
 
+#include set
 #include math.h
 #include vcl/svapp.hxx
 #include vcl/metric.hxx
@@ -86,9 +87,6 @@ using namespace ::com::sun::star::uno;
 using namespace ::com::sun::star::beans;
 using namespace ::com::sun::star::linguistic2;
 
-SV_DECL_VARARR_SORT( SortedPositions, sal_uInt32, 16 )
-SV_IMPL_VARARR_SORT( SortedPositions, sal_uInt32 );
-
 #define CH_HYPH '-'
 
 #define RESDIFF 10
@@ -2252,14 +2250,14 @@ sal_uInt16 ImpEditEngine::SplitTextPortion( 
ParaPortion* pPortion, sal_uInt16 nP
 return nSplitPortion;
 }
 
-void ImpEditEngine::CreateTextPortions( ParaPortion* pParaPortion, sal_uInt16 
rStart /* , sal_Bool bCreateBlockPortions */ )
+void ImpEditEngine::CreateTextPortions( ParaPortion* pParaPortion, sal_uInt16 
rStart )
 {
 sal_uInt16 nStartPos = rStart;
 ContentNode* pNode = pParaPortion-GetNode();
 DBG_ASSERT( pNode-Len(), CreateTextPortions should not be used for empty 
paragraphs! );
 
-SortedPositions aPositions;
-aPositions.Insert( (sal_uInt32) 0 );
+::std::set sal_uInt32  aPositions;
+aPositions.insert( 0 );
 
 sal_uInt16 nAttr = 0;
 EditCharAttrib* pAttrib = GetAttrib( pNode-GetCharAttribs().GetAttribs(), 
nAttr );
@@ -2267,23 +2265,23 @@ void ImpEditEngine::CreateTextPortions( ParaPortion* 
pParaPortion, sal_uInt16 r
 {
 // Insert Start and End into the Array...
 // The Insert method does not allow for duplicate values
-aPositions.Insert( pAttrib-GetStart() );
-aPositions.Insert( pAttrib-GetEnd() );
+aPositions.insert( pAttrib-GetStart() );
+aPositions.insert( pAttrib-GetEnd() );
 nAttr++;
 pAttrib = GetAttrib( pNode-GetCharAttribs().GetAttribs(), nAttr );
 }
-aPositions.Insert( pNode-Len() );
+aPositions.insert( pNode-Len() );
 
 if ( pParaPortion-aScriptInfos.empty() )
 ((ImpEditEngine*)this)-InitScriptTypes( GetParaPortions().GetPos( 
pParaPortion ) );
 
 const ScriptTypePosInfos rTypes = pParaPortion-aScriptInfos;
 for ( size_t nT = 0; nT  rTypes.size(); nT++ )
-aPositions.Insert( rTypes[nT].nStartPos );
+aPositions.insert( rTypes[nT].nStartPos );
 
 const WritingDirectionInfos rWritingDirections = 
pParaPortion-aWritingDirectionInfos;
 for ( size_t nD = 0; nD  rWritingDirections.size(); nD++ )
-aPositions.Insert( rWritingDirections[nD].nStartPos );
+aPositions.insert( rWritingDirections[nD].nStartPos );
 
 if ( mpIMEInfos  mpIMEInfos-nLen  mpIMEInfos-pAttribs  ( 
mpIMEInfos-aPos.GetNode() == pNode ) )
 {
@@ -2292,11 +2290,11 @@ void ImpEditEngine::CreateTextPortions( ParaPortion* 
pParaPortion, sal_uInt16 r
 {
 if ( mpIMEInfos-pAttribs[n] != nLastAttr )
 {
-aPositions.Insert( mpIMEInfos-aPos.GetIndex() + n );

[Libreoffice-commits] .: editeng/source

2012-03-01 Thread Tor Lillqvist
 editeng/source/editeng/editdoc.cxx  |   26 --
 editeng/source/editeng/impedit.hxx  |5 ++---
 editeng/source/editeng/impedit4.cxx |   24 
 3 files changed, 14 insertions(+), 41 deletions(-)

New commits:
commit a0cd57b926830e8798a5019b30776e94dd97c8a1
Author: Noel Grandin n...@peralex.com
Date:   Thu Mar 1 10:22:36 2012 +0200

Remove SvxFontTable and use std::vector instead

In removing tools/table.hxx usage, remove SvxFontTable and
convert to std::vectorSvxFontItem* because it was only
being used as a temporary list, not a map.

diff --git a/editeng/source/editeng/editdoc.cxx 
b/editeng/source/editeng/editdoc.cxx
index 50f96a2..98268d9 100644
--- a/editeng/source/editeng/editdoc.cxx
+++ b/editeng/source/editeng/editdoc.cxx
@@ -2161,32 +2161,6 @@ bool CharAttribList::DbgCheckAttribs() const
 }
 #endif
 
-SvxFontTable::SvxFontTable()
-{
-}
-
-SvxFontTable::~SvxFontTable()
-{
-SvxFontItem* pItem = First();
-while( pItem )
-{
-delete pItem;
-pItem = Next();
-}
-}
-
-sal_uLong SvxFontTable::GetId( const SvxFontItem rFontItem )
-{
-SvxFontItem* pItem = First();
-while ( pItem )
-{
-if ( *pItem == rFontItem )
-return GetCurKey();
-pItem = Next();
-}
-DBG_WARNING( Font not found: GetId() );
-return 0;
-}
 
 SvxColorList::SvxColorList()
 {
diff --git a/editeng/source/editeng/impedit.hxx 
b/editeng/source/editeng/impedit.hxx
index 1c008fb..d02a959 100644
--- a/editeng/source/editeng/impedit.hxx
+++ b/editeng/source/editeng/impedit.hxx
@@ -80,7 +80,6 @@ DBG_NAMEEX( EditEngine )
 
 class EditView;
 class EditEngine;
-class SvxFontTable;
 class SvxColorList;
 
 class SvxSearchItem;
@@ -640,9 +639,9 @@ private:
 sal_uInt32  WriteBin( SvStream rOutput, EditSelection aSel, 
sal_Bool bStoreUnicode = sal_False ) const;
 
 voidWriteItemAsRTF( const SfxPoolItem rItem, SvStream 
rOutput, sal_uInt16 nPara, sal_uInt16 nPos,
-SvxFontTable rFontTable, SvxColorList rColorList );
+std::vectorSvxFontItem* rFontTable, SvxColorList 
rColorList );
 sal_BoolWriteItemListAsRTF( ItemList rLst, SvStream rOutput, 
sal_uInt16 nPara, sal_uInt16 nPos,
-SvxFontTable rFontTable, SvxColorList rColorList );
+std::vectorSvxFontItem* rFontTable, SvxColorList 
rColorList );
 sal_Int32   LogicToTwips( sal_Int32 n );
 
 inline shortGetXValue( short nXValue ) const;
diff --git a/editeng/source/editeng/impedit4.cxx 
b/editeng/source/editeng/impedit4.cxx
index f76b0ca..07d8074 100644
--- a/editeng/source/editeng/impedit4.cxx
+++ b/editeng/source/editeng/impedit4.cxx
@@ -277,7 +277,7 @@ sal_uInt32 ImpEditEngine::WriteText( SvStream rOutput, 
EditSelection aSel )
 }
 
 sal_Bool ImpEditEngine::WriteItemListAsRTF( ItemList rLst, SvStream rOutput, 
sal_uInt16 nPara, sal_uInt16 nPos,
-SvxFontTable rFontTable, SvxColorList rColorList )
+std::vectorSvxFontItem* rFontTable, SvxColorList 
rColorList )
 {
 const SfxPoolItem* pAttrItem = rLst.First();
 while ( pAttrItem )
@@ -358,11 +358,11 @@ sal_uInt32 ImpEditEngine::WriteRTF( SvStream rOutput, 
EditSelection aSel )
 rtl_TextEncoding eDestEnc = RTL_TEXTENCODING_MS_1252;
 
 // Generate and write out Font table  ...
-SvxFontTable aFontTable;
+std::vectorSvxFontItem* aFontTable;
 // default font must be up front, so DEF font in RTF
-aFontTable.Insert( 0, new SvxFontItem( (const 
SvxFontItem)aEditDoc.GetItemPool().GetDefaultItem( EE_CHAR_FONTINFO ) ) );
-aFontTable.Insert( 1, new SvxFontItem( (const 
SvxFontItem)aEditDoc.GetItemPool().GetDefaultItem( EE_CHAR_FONTINFO_CJK ) ) );
-aFontTable.Insert( 2, new SvxFontItem( (const 
SvxFontItem)aEditDoc.GetItemPool().GetDefaultItem( EE_CHAR_FONTINFO_CTL ) ) );
+aFontTable.push_back( new SvxFontItem( (const 
SvxFontItem)aEditDoc.GetItemPool().GetDefaultItem( EE_CHAR_FONTINFO ) ) );
+aFontTable.push_back( new SvxFontItem( (const 
SvxFontItem)aEditDoc.GetItemPool().GetDefaultItem( EE_CHAR_FONTINFO_CJK ) ) );
+aFontTable.push_back( new SvxFontItem( (const 
SvxFontItem)aEditDoc.GetItemPool().GetDefaultItem( EE_CHAR_FONTINFO_CTL ) ) );
 for ( sal_uInt16 nScriptType = 0; nScriptType  3; nScriptType++ )
 {
 sal_uInt16 nWhich = EE_CHAR_FONTINFO;
@@ -376,14 +376,14 @@ sal_uInt32 ImpEditEngine::WriteRTF( SvStream rOutput, 
EditSelection aSel )
 while ( pFontItem )
 {
 bool bAlreadyExist = false;
-sal_uLong nTestMax = nScriptType ? aFontTable.Count() : 1;
+sal_uLong nTestMax = nScriptType ? aFontTable.size() : 1;
 for ( sal_uLong nTest = 0; !bAlreadyExist  ( nTest  nTestMax ); 
nTest++ )
 {
-bAlreadyExist = *aFontTable.Get( nTest ) == *pFontItem;
+   

[Libreoffice-commits] .: editeng/source

2012-02-05 Thread Ivan Timofeev
 editeng/source/editeng/edtspell.cxx |2 --
 1 file changed, 2 deletions(-)

New commits:
commit 1c85118414d91bc0b881f03ad7fac20454ec43a0
Author: Ivan Timofeev timofeev@gmail.com
Date:   Sun Feb 5 22:02:12 2012 +0400

fix debug build

diff --git a/editeng/source/editeng/edtspell.cxx 
b/editeng/source/editeng/edtspell.cxx
index eda0d61..a2340aa 100644
--- a/editeng/source/editeng/edtspell.cxx
+++ b/editeng/source/editeng/edtspell.cxx
@@ -505,7 +505,6 @@ bool WrongList::operator==(const WrongList rCompare) const
 return true;
 }
 
-#ifdef DBG_UTIL
 sal_Bool WrongList::DbgIsBuggy() const
 {
 // Check if the ranges overlap.
@@ -524,7 +523,6 @@ sal_Bool WrongList::DbgIsBuggy() const
 }
 return bError;
 }
-#endif
 
 //
 
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-02-02 Thread Eike Rathke
 editeng/source/items/borderline.cxx |   16 
 1 file changed, 16 insertions(+)

New commits:
commit 971f826a96f0938bfcfd8dfe26edd50d8e031bb4
Author: Eike Rathke er...@redhat.com
Date:   Thu Feb 2 19:11:37 2012 +0100

resolved fdo#42784 BorderLine with only InnerWidth set does not work

diff --git a/editeng/source/items/borderline.cxx 
b/editeng/source/items/borderline.cxx
index 8d073c6..b617ff4 100644
--- a/editeng/source/items/borderline.cxx
+++ b/editeng/source/items/borderline.cxx
@@ -298,6 +298,22 @@ void SvxBorderLine::GuessLinesWidths( SvxBorderStyle 
nStyle, sal_uInt16 nOut, sa
 else
 {
 SetStyle( nStyle );
+if (nOut == 0  nIn  0)
+{
+// If only inner width is given swap inner and outer widths for
+// single line styles, otherwise GuessWidth() marks this as invalid
+// and returns a 0 width.
+switch (nStyle)
+{
+case SOLID:
+case DOTTED:
+case DASHED:
+::std::swap( nOut, nIn);
+break;
+default:
+;   // nothing
+}
+}
 m_nWidth = m_aWidthImpl.GuessWidth( nOut, nIn, nDist );
 }
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source sw/source

2012-02-01 Thread Julien Nabet
 editeng/source/accessibility/AccessibleStaticTextBase.cxx |2 +-
 sw/source/core/doc/ftnidx.cxx |   10 ++
 2 files changed, 3 insertions(+), 9 deletions(-)

New commits:
commit 428c9d516d3408460f1b7f8ce2c753b9a829de87
Author: Julien Nabet serval2...@yahoo.fr
Date:   Wed Feb 1 19:36:28 2012 +0100

Some cppcheck cleaning

diff --git a/editeng/source/accessibility/AccessibleStaticTextBase.cxx 
b/editeng/source/accessibility/AccessibleStaticTextBase.cxx
index 9e234d9..734dc1b 100644
--- a/editeng/source/accessibility/AccessibleStaticTextBase.cxx
+++ b/editeng/source/accessibility/AccessibleStaticTextBase.cxx
@@ -344,7 +344,7 @@ namespace accessibility
 {
 // Keep 'invalid' values at the TextSegment
 if( aTextSegment.SegmentStart != -1 
-aTextSegment.SegmentStart != -1 )
+aTextSegment.SegmentEnd != -1 )
 {
 // #112814# Correct TextSegment by paragraph offset
 sal_Int32 nOffset(0);
diff --git a/sw/source/core/doc/ftnidx.cxx b/sw/source/core/doc/ftnidx.cxx
index 3ccd856..52f9873 100644
--- a/sw/source/core/doc/ftnidx.cxx
+++ b/sw/source/core/doc/ftnidx.cxx
@@ -192,10 +192,7 @@ void SwFtnIdxs::UpdateFtn( const SwNodeIndex rStt )
 
 if( nSectNo )
 {
-if( rFtn.IsEndNote() )
-pTxtFtn-SetNumber( nSectNo, rFtn.GetNumStr() );
-else
-pTxtFtn-SetNumber( nSectNo, rFtn.GetNumStr() );
+pTxtFtn-SetNumber( nSectNo, rFtn.GetNumStr() );
 }
 }
 }
@@ -281,10 +278,7 @@ void SwFtnIdxs::UpdateAllFtn()
 
 if( nSectNo )
 {
-if( rFtn.IsEndNote() )
-pTxtFtn-SetNumber( nSectNo, rFtn.GetNumStr() );
-else
-pTxtFtn-SetNumber( nSectNo, rFtn.GetNumStr() );
+pTxtFtn-SetNumber( nSectNo, rFtn.GetNumStr() );
 }
 }
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source oox/source sd/source

2012-01-14 Thread Matus Kukan
 editeng/source/editeng/editdbg.cxx   |2 +-
 oox/source/helper/propertymap.cxx|   30 +++---
 sd/source/filter/eppt/pptx-epptooxml.cxx |   12 ++--
 3 files changed, 22 insertions(+), 22 deletions(-)

New commits:
commit fcc8db1d2be9a25f79cbcc14043b495d240bb8bf
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Sat Jan 14 11:30:26 2012 +0100

fix for gcc 4.7: unable to find string literal operator

diff --git a/editeng/source/editeng/editdbg.cxx 
b/editeng/source/editeng/editdbg.cxx
index 9389001..fc9c562 100644
--- a/editeng/source/editeng/editdbg.cxx
+++ b/editeng/source/editeng/editdbg.cxx
@@ -454,7 +454,7 @@ void EditDbg::ShowEditEngineData( EditEngine* pEE, sal_Bool 
bInfoBox )
 fprintf( fp, 
\n\n
 );
 fprintf( fp, \n==   EditEngine  Views   
== );
 fprintf( fp, 
\n
 );
-fprintf( fp, \nControl: %SAL_PRIxUINT32, pEE-GetControlWord() );
+fprintf( fp, \nControl: % SAL_PRIxUINT32, pEE-GetControlWord() );
 fprintf( fp, \nRefMapMode: %i, 
pEE-pImpEditEngine-pRefDev-GetMapMode().GetMapUnit() );
 fprintf( fp, \nPaperSize: %li x %li, pEE-GetPaperSize().Width(), 
pEE-GetPaperSize().Height() );
 fprintf( fp, \nMaxAutoPaperSize: %li x %li, 
pEE-GetMaxAutoPaperSize().Width(), pEE-GetMaxAutoPaperSize().Height() );
diff --git a/oox/source/helper/propertymap.cxx 
b/oox/source/helper/propertymap.cxx
index b4094eb..af0d374 100644
--- a/oox/source/helper/propertymap.cxx
+++ b/oox/source/helper/propertymap.cxx
@@ -355,9 +355,9 @@ static void lclDumpAnyValue( Any value)
 } else if( value = aMatrix ) {
 fprintf (stderr,Matrix\n%f %f %f\n%f %f %f\n%f %f %f\n, 
aMatrix.Line1.Column1, aMatrix.Line1.Column2, aMatrix.Line1.Column3, 
aMatrix.Line2.Column1, aMatrix.Line2.Column2, aMatrix.Line2.Column3, 
aMatrix.Line3.Column1, aMatrix.Line3.Column2, aMatrix.Line3.Column3);
 } else if( value = intValue )
-fprintf (stderr,%SAL_PRIdINT32(hex: 
%SAL_PRIxUINT32)\n, intValue, intValue);
+fprintf (stderr,% SAL_PRIdINT32 (hex: % 
SAL_PRIxUINT32 )\n, intValue, intValue);
 else if( value = uintValue )
-fprintf (stderr,%SAL_PRIdINT32(hex: 
%SAL_PRIxUINT32)\n, uintValue, uintValue);
+fprintf (stderr,% SAL_PRIdINT32 (hex: % 
SAL_PRIxUINT32 )\n, uintValue, uintValue);
 else if( value = int16Value )
 fprintf (stderr,%d(hex: %x)\n, int16Value, 
int16Value);
 else if( value = uint16Value )
@@ -502,7 +502,7 @@ static const char* lclDumpAnyValueCode( Any value, int 
level = 0)
 return Any (str);
 } else if( value = strArray ) {
 printLevel (level);
-fprintf (stderr,Sequence OUString  aStringSequence 
(%SAL_PRIdINT32);\n, strArray.getLength());
+fprintf (stderr,Sequence OUString  aStringSequence (% 
SAL_PRIdINT32 );\n, strArray.getLength());
 for( int i=0; istrArray.getLength(); i++ ) {
 printLevel (level);
 fprintf (stderr,aStringSequence[%d] = CREATE_OUSTRING 
(\%s\);\n, i, USS( strArray[i] ) );
@@ -510,7 +510,7 @@ static const char* lclDumpAnyValueCode( Any value, int 
level = 0)
 return aStringSequence;
 } else if( value = propArray ) {
 printLevel (level);
-fprintf (stderr,Sequence PropertyValue  aPropSequence 
(%SAL_PRIdINT32);\n, propArray.getLength());
+fprintf (stderr,Sequence PropertyValue  aPropSequence (% 
SAL_PRIdINT32 );\n, propArray.getLength());
 for( int i=0; ipropArray.getLength(); i++ ) {
 printLevel (level);
 fprintf (stderr, {\n);
@@ -525,7 +525,7 @@ static const char* lclDumpAnyValueCode( Any value, int 
level = 0)
 return aPropSequence;
 } else if( value = propArrayArray ) {
 printLevel (level);
-fprintf (stderr,Sequence Sequence  PropertyValue   
aPropSequenceSequence (%SAL_PRIdINT32);\n, propArrayArray.getLength());
+fprintf (stderr,Sequence Sequence  PropertyValue   
aPropSequenceSequence (% SAL_PRIdINT32 );\n, propArrayArray.getLength());
 for( int i=0; ipropArrayArray.getLength(); i++ ) {
 printLevel (level);
 fprintf (stderr, {\n);
@@ -544,7 +544,7 @@ static const char* lclDumpAnyValueCode( Any value, int 
level = 0)
 }
 } else if( value = adjArray ) {
 printLevel (level);
-fprintf (stderr,Sequence EnhancedCustomShapeAdjustmentValue  
aAdjSequence (%SAL_PRIdINT32);\n, adjArray.getLength());
+fprintf (stderr,Sequence EnhancedCustomShapeAdjustmentValue  
aAdjSequence (% SAL_PRIdINT32 );\n, adjArray.getLength());

[Libreoffice-commits] .: editeng/source

2012-01-11 Thread August Sodora
 editeng/source/misc/svxacorr.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 0204bbffbc273782ca3148c7ba6768af4f5a77f9
Author: August Sodora aug...@gmail.com
Date:   Wed Jan 11 20:49:52 2012 -0500

Remove an unused SvStringsDtor

diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index b7df997..169c403 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -2155,7 +2155,6 @@ SvxAutocorrWordList* 
SvxAutoCorrectLanguageLists::LoadAutocorrWordList()
 else
 pAutocorr_List = new SvxAutocorrWordList( 16, 16 );
 
-SvStringsDtor aRemoveArr;
 try
 {
 uno::Reference  embed::XStorage  xStg = 
comphelper::OStorageHelper::GetStorageFromURL( sShareAutoCorrFile, 
embed::ElementModes::READ );
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-01-07 Thread Andras Timar
 editeng/source/items/svxitems.src |   12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)

New commits:
commit b95be347d45c90febdeca85d22b65c03fba283bc
Author: Andras Timar ati...@suse.com
Date:   Sat Jan 7 22:08:09 2012 +0100

make measurement units localizable

diff --git a/editeng/source/items/svxitems.src 
b/editeng/source/items/svxitems.src
index 222fc2c..6049ad6 100644
--- a/editeng/source/items/svxitems.src
+++ b/editeng/source/items/svxitems.src
@@ -598,27 +598,27 @@ String RID_OUTSET
 };
 String RID_SVXITEMS_METRIC_MM
 {
-Text = mm ;
+Text [ en-US ] = mm ;
 };
 String RID_SVXITEMS_METRIC_CM
 {
-Text = cm ;
+Text [ en-US ] = cm ;
 };
 String RID_SVXITEMS_METRIC_INCH
 {
-Text = inch ;
+Text [ en-US ] = inch ;
 };
 String RID_SVXITEMS_METRIC_POINT
 {
-Text = pt ;
+Text [ en-US ] = pt ;
 };
 String RID_SVXITEMS_METRIC_TWIP
 {
-Text = twip ;
+Text [ en-US ] = twip ;
 };
 String RID_SVXITEMS_METRIC_PIXEL
 {
-Text = pixel ;
+Text [ en-US ] = pixel ;
 };
  // GetValueText von BoolItems
 String RID_SVXITEMS_SHADOWED_TRUE
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2011-12-23 Thread August Sodora
 editeng/source/misc/svxacorr.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit c9f68b6d2356ef81560b5449670b81a1156a5da6
Author: August Sodora aug...@gmail.com
Date:   Fri Dec 23 19:44:48 2011 -0500

Fix the build so that this is recognized as an initializer list

diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index c2d937a..df191c0 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -1644,7 +1644,7 @@ sal_Bool SvxAutoCorrect::CreateLanguageFile( LanguageType 
eLang, sal_Bool bNewFi
 }
 else if( !bNewFile )
 {
-pLastFileTable[eLang] = { std::make_pair(eLang, nAktTime.GetTime()) };
+pLastFileTable[eLang] = { std::make_pair(eLang, nAktTime.GetTime()), };
 }
 return pLists != 0;
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2011-12-23 Thread David Tardon
 editeng/source/misc/svxacorr.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 93d3f6258f8f061dc53b5e6003de65479c2290ba
Author: David Tardon dtar...@redhat.com
Date:   Sat Dec 24 06:29:30 2011 +0100

make the line work as intended again

At least I suppose memory corruption is not intended behavior .-)

diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index df191c0..f28df9d 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -1644,7 +1644,7 @@ sal_Bool SvxAutoCorrect::CreateLanguageFile( LanguageType 
eLang, sal_Bool bNewFi
 }
 else if( !bNewFile )
 {
-pLastFileTable[eLang] = { std::make_pair(eLang, nAktTime.GetTime()), };
+(*pLastFileTable)[eLang] = nAktTime.GetTime();
 }
 return pLists != 0;
 }
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source oox/inc oox/source svtools/inc svtools/source writerfilter/source

2011-11-24 Thread Cédric Bosdonnat
 editeng/source/rtf/rtfgrf.cxx  |9 ---
 oox/inc/oox/helper/graphichelper.hxx   |8 ++-
 oox/source/helper/graphichelper.cxx|   65 -
 svtools/inc/svtools/filter.hxx |6 +-
 svtools/inc/svtools/wmf.hxx|   48 ++
 svtools/source/filter/filter.cxx   |8 +--
 svtools/source/filter/wmf/winmtf.hxx   |   14 +++--
 svtools/source/filter/wmf/winwmf.cxx   |   62 ---
 svtools/source/filter/wmf/wmf.cxx  |4 -
 svtools/source/graphic/provider.cxx|   42 +++-
 writerfilter/source/rtftok/rtfcontrolwords.cxx |2 
 writerfilter/source/rtftok/rtfdocumentimpl.cxx |   19 ++-
 writerfilter/source/rtftok/rtfdocumentimpl.hxx |1 
 13 files changed, 187 insertions(+), 101 deletions(-)

New commits:
commit f4f4a899f0d5520a497bbd07ccfef9086c54ca1c
Author: Cédric Bosdonnat cedric.bosdonnat@free.fr
Date:   Thu Nov 24 15:05:25 2011 +0100

RTF import: partly fixed the WMF images scaling.

diff --git a/editeng/source/rtf/rtfgrf.cxx b/editeng/source/rtf/rtfgrf.cxx
index 4ff9f35..2906caa 100644
--- a/editeng/source/rtf/rtfgrf.cxx
+++ b/editeng/source/rtf/rtfgrf.cxx
@@ -495,15 +495,8 @@ sal_Bool SvxRTFParser::ReadBmpData( Graphic rGrf, 
SvxRTFPictureType rPicType )
 }
 
 String sTmpStr;
-WMF_APMFILEHEADER aAPMHeader;
-aAPMHeader.left=0;
-aAPMHeader.top=0;
-aAPMHeader.right=rPicType.nWidth;
-aAPMHeader.bottom=rPicType.nHeight;
-
-WMF_APMFILEHEADER *pAPMHeader=(aAPMHeader.right0  
aAPMHeader.bottom0?aAPMHeader:NULL);
 pTmpFile-Seek( STREAM_SEEK_TO_BEGIN );
-bValidBmp = 0 == rGF.ImportGraphic( rGrf, sTmpStr, *pTmpFile, 
nImportFilter, NULL, 0, pAPMHeader );
+bValidBmp = 0 == rGF.ImportGraphic( rGrf, sTmpStr, *pTmpFile, 
nImportFilter );
 }
 }
 
diff --git a/oox/inc/oox/helper/graphichelper.hxx 
b/oox/inc/oox/helper/graphichelper.hxx
index f144cd1..d3507fa 100644
--- a/oox/inc/oox/helper/graphichelper.hxx
+++ b/oox/inc/oox/helper/graphichelper.hxx
@@ -37,6 +37,8 @@
 #include oox/helper/binarystreambase.hxx
 #include oox/helper/storagebase.hxx
 
+struct WMF_EXTERNALHEADER;
+
 namespace com { namespace sun { namespace star {
 namespace awt { struct Point; }
 namespace awt { struct Size; }
@@ -115,7 +117,8 @@ public:
 /** Imports a graphic from the passed input stream. */
 ::com::sun::star::uno::Reference ::com::sun::star::graphic::XGraphic 
 importGraphic(
-const ::com::sun::star::uno::Reference 
::com::sun::star::io::XInputStream  rxInStrm ) const;
+const ::com::sun::star::uno::Reference 
::com::sun::star::io::XInputStream  rxInStrm,
+const WMF_EXTERNALHEADER* pExtHeader = NULL ) 
const;
 
 /** Imports a graphic from the passed binary memory block. */
 ::com::sun::star::uno::Reference ::com::sun::star::graphic::XGraphic 
@@ -133,7 +136,8 @@ public:
 /** Creates a persistent graphic object from the passed input stream.
 @return  The URL of the created and internally cached graphic object. 
*/
 ::rtl::OUString importGraphicObject(
-const ::com::sun::star::uno::Reference 
::com::sun::star::io::XInputStream  rxInStrm ) const;
+const ::com::sun::star::uno::Reference 
::com::sun::star::io::XInputStream  rxInStrm,
+const WMF_EXTERNALHEADER* pExtHeader = NULL ) 
const;
 
 /** Creates a persistent graphic object from the passed binary memory 
block.
 @return  The URL of the created and internally cached graphic object. 
*/
diff --git a/oox/source/helper/graphichelper.cxx 
b/oox/source/helper/graphichelper.cxx
index 941cc80..fd3046e 100644
--- a/oox/source/helper/graphichelper.cxx
+++ b/oox/source/helper/graphichelper.cxx
@@ -39,6 +39,7 @@
 #include com/sun/star/graphic/XGraphicProvider.hpp
 #include com/sun/star/util/MeasureUnit.hpp
 #include comphelper/seqstream.hxx
+#include svtools/wmf.hxx
 #include oox/helper/containerhelper.hxx
 #include oox/helper/propertyset.hxx
 #include oox/token/tokens.hxx
@@ -47,7 +48,8 @@ namespace oox {
 
 // 
 
-using namespace ::com::sun::star::awt;
+
+using namespace ::com::sun::star;
 using namespace ::com::sun::star::beans;
 using namespace ::com::sun::star::frame;
 using namespace ::com::sun::star::graphic;
@@ -130,7 +132,7 @@ GraphicHelper::GraphicHelper( const Reference 
XComponentContext  rxContext, c
 maDeviceInfo.PixelPerMeterX = maDeviceInfo.PixelPerMeterY = 3500.0; // 
some default just in case
 if( xFrame.is() ) try
 {
-Reference XDevice  xDevice( xFrame-getContainerWindow(), 
UNO_QUERY_THROW );
+

[Libreoffice-commits] .: editeng/source sw/source

2011-11-15 Thread Michael Meeks
 editeng/source/uno/unotext.cxx|   34 --
 sw/source/filter/xml/xmltexti.cxx |2 +-
 2 files changed, 21 insertions(+), 15 deletions(-)

New commits:
commit aa0b7b033098bfc1bf3f06cd112691abc862bd20
Author: Michael Meeks michael.me...@suse.com
Date:   Sat Nov 12 11:15:19 2011 +

avoid crasher from unusual odt document fdo#42785

diff --git a/editeng/source/uno/unotext.cxx b/editeng/source/uno/unotext.cxx
index ac241d2..5933da3 100644
--- a/editeng/source/uno/unotext.cxx
+++ b/editeng/source/uno/unotext.cxx
@@ -183,6 +183,13 @@ void CheckSelection( struct ESelection rSel, 
SvxTextForwarder* pForwarder ) thr
 }
 }
 
+void CheckSelection( struct ESelection rSel, SvxEditSource *pEdit ) throw()
+{
+if (!pEdit)
+return;
+CheckSelection( rSel, pEdit-GetTextForwarder() );
+}
+
 // 
 // class SvxUnoTextRangeBase
 // 
@@ -275,8 +282,7 @@ void SvxUnoTextRangeBase::SetSelection( const ESelection 
rSelection ) throw()
 SolarMutexGuard aGuard;
 
 maSelection = rSelection;
-if (mpEditSource != NULL)
-CheckSelection( maSelection, mpEditSource-GetTextForwarder() );
+CheckSelection( maSelection, mpEditSource );
 }
 
 // Interface XTextRange ( XText )
@@ -291,7 +297,6 @@ uno::Reference text::XTextRange  SAL_CALL 
SvxUnoTextRangeBase::getStart(void)
 SvxTextForwarder* pForwarder = mpEditSource ? 
mpEditSource-GetTextForwarder() : NULL;
 if( pForwarder )
 {
-
 CheckSelection( maSelection, pForwarder );
 
 SvxUnoTextBase* pText = SvxUnoTextBase::getImplementation( getText() );
@@ -406,7 +411,6 @@ void SAL_CALL SvxUnoTextRangeBase::_setPropertyValue( const 
OUString PropertyNa
 SvxTextForwarder* pForwarder = mpEditSource ? 
mpEditSource-GetTextForwarder() : NULL;
 if( pForwarder )
 {
-
 CheckSelection( maSelection, pForwarder );
 
 const SfxItemPropertySimpleEntry* pMap = 
mpPropSet-getPropertyMapEntry(PropertyName );
@@ -1308,7 +1312,7 @@ uno::Sequence uno::Any  SAL_CALL 
SvxUnoTextRangeBase::getPropertyDefaults( con
 // internal
 void SvxUnoTextRangeBase::CollapseToStart(void) throw()
 {
-CheckSelection( maSelection, mpEditSource-GetTextForwarder() );
+CheckSelection( maSelection, mpEditSource );
 
 maSelection.nEndPara = maSelection.nStartPara;
 maSelection.nEndPos  = maSelection.nStartPos;
@@ -1316,7 +1320,7 @@ void SvxUnoTextRangeBase::CollapseToStart(void) throw()
 
 void SvxUnoTextRangeBase::CollapseToEnd(void) throw()
 {
-CheckSelection( maSelection, mpEditSource-GetTextForwarder() );
+CheckSelection( maSelection, mpEditSource );
 
 maSelection.nStartPara = maSelection.nEndPara;
 maSelection.nStartPos  = maSelection.nEndPos;
@@ -1324,7 +1328,7 @@ void SvxUnoTextRangeBase::CollapseToEnd(void) throw()
 
 sal_Bool SvxUnoTextRangeBase::IsCollapsed(void) throw()
 {
-CheckSelection( maSelection, mpEditSource-GetTextForwarder() );
+CheckSelection( maSelection, mpEditSource );
 
 return ( maSelection.nStartPara == maSelection.nEndPara 
  maSelection.nStartPos  == maSelection.nEndPos );
@@ -1332,7 +1336,7 @@ sal_Bool SvxUnoTextRangeBase::IsCollapsed(void) throw()
 
 sal_Bool SvxUnoTextRangeBase::GoLeft(sal_Int16 nCount, sal_Bool Expand) throw()
 {
-CheckSelection( maSelection, mpEditSource-GetTextForwarder() );
+CheckSelection( maSelection, mpEditSource );
 
 //  #75098# use end position, as in Writer (start is anchor, end is cursor)
 sal_uInt16 nNewPos = maSelection.nEndPos;
@@ -1375,7 +1379,6 @@ sal_Bool SvxUnoTextRangeBase::GoRight(sal_Int16 nCount, 
sal_Bool Expand)  throw(
 {
 CheckSelection( maSelection, pForwarder );
 
-
 sal_uInt16 nNewPos = maSelection.nEndPos + nCount; //! Overflow???
 sal_uInt16 nNewPar = maSelection.nEndPara;
 
@@ -1419,7 +1422,7 @@ void SvxUnoTextRangeBase::GotoStart(sal_Bool Expand) 
throw()
 
 void SvxUnoTextRangeBase::GotoEnd(sal_Bool Expand) throw()
 {
-CheckSelection( maSelection, mpEditSource-GetTextForwarder() );
+CheckSelection( maSelection, mpEditSource );
 
 SvxTextForwarder* pForwarder = mpEditSource ? 
mpEditSource-GetTextForwarder() : NULL;
 if( pForwarder )
@@ -1670,8 +1673,11 @@ void SAL_CALL SvxUnoTextBase::insertString( const 
uno::Reference text::XTextRan
 return;
 
 ESelection aSelection;
-::GetSelection( aSelection, GetEditSource()-GetTextForwarder() );
-SetSelection( aSelection );
+if (GetEditSource())
+{
+::GetSelection( aSelection, GetEditSource()-GetTextForwarder() );
+SetSelection( aSelection );
+}
 
 SvxUnoTextRangeBase* pRange = SvxUnoTextRange::getImplementation( xRange );
 if(pRange)
@@ -1771,10 +1777,10 @@ void SAL_CALL SvxUnoTextBase::insertControlCharacter( 
const uno::Reference text
 return;
 }
 

[Libreoffice-commits] .: editeng/source

2011-11-09 Thread Caolán McNamara
 editeng/source/editeng/impedit3.cxx |3 ---
 1 file changed, 3 deletions(-)

New commits:
commit 3f9df85defed04730fda2c862810b505579fb958
Author: Christina Rossmanith chrrossman...@web.de
Date:   Wed Nov 9 16:59:47 2011 +

_SVSTDARR_sal_uIt16S is a typo and doesn't do anything

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 83695cf..bdf1bbd 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -36,9 +36,6 @@
 #include vcl/metaact.hxx
 #include vcl/gdimtf.hxx
 
-#define _SVSTDARR_sal_uIt16S
-#include svl/svstdarr.hxx
-
 #include vcl/wrkwin.hxx
 #include editeng/adjitem.hxx
 #include editeng/tstpitem.hxx
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source formula/source sc/source sd/source svtools/source sw/source vcl/aqua vcl/inc vcl/unx vcl/win

2011-10-21 Thread Michael Meeks
 editeng/source/editeng/editeng.cxx   |2 +-
 editeng/source/editeng/impedit3.cxx  |2 +-
 formula/source/ui/dlg/formula.cxx|4 ++--
 sc/source/core/tool/chartlis.cxx |2 +-
 sc/source/ui/app/scmod.cxx   |2 +-
 sd/source/ui/tools/IdleDetection.cxx |2 +-
 svtools/source/control/inettbc.cxx   |2 +-
 svtools/source/edit/textview.cxx |2 +-
 sw/source/core/layout/paintfrm.cxx   |2 +-
 sw/source/core/view/viewsh.cxx   |2 +-
 sw/source/ui/docvw/edtwin.cxx|2 +-
 vcl/aqua/source/app/salinst.cxx  |4 ++--
 vcl/inc/vcl/apptypes.hxx |6 +++---
 vcl/unx/generic/app/salinst.cxx  |4 ++--
 vcl/win/source/app/salinst.cxx   |2 +-
 15 files changed, 20 insertions(+), 20 deletions(-)

New commits:
commit b0224f8050373cbdde58c3dee1bbda71976b50b6
Author: Olivier Hallot olivier.hal...@alta.org.br
Date:   Fri Oct 21 16:54:35 2011 -0200

Fix for bug fdo#41997, cleanup vcl enumeration

diff --git a/editeng/source/editeng/editeng.cxx 
b/editeng/source/editeng/editeng.cxx
index 5d745c0..573b30b 100644
--- a/editeng/source/editeng/editeng.cxx
+++ b/editeng/source/editeng/editeng.cxx
@@ -1195,7 +1195,7 @@ sal_Bool EditEngine::PostKeyEvent( const KeyEvent 
rKeyEvent, EditView* pEditVie
 DBG_ASSERT( !bReadOnly, ReadOnly but modified??? );
 // Idle-Formatter only when AnyInput.
 if ( bAllowIdle  pImpEditEngine-GetStatus().UseIdleFormatter()
- Application::AnyInput( INPUT_KEYBOARD) )
+ Application::AnyInput( VCL_INPUT_KEYBOARD) )
 pImpEditEngine-IdleFormatAndUpdate( pEditView );
 else
 pImpEditEngine-FormatAndUpdate( pEditView );
diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index b4efe14..8f059a3 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -327,7 +327,7 @@ void ImpEditEngine::UpdateViews( EditView* pCurView )
 
 IMPL_LINK( ImpEditEngine, OnlineSpellHdl, Timer *, EMPTYARG )
 {
-if ( !Application::AnyInput( INPUT_KEYBOARD )  GetUpdateMode()  
IsFormatted() )
+if ( !Application::AnyInput( VCL_INPUT_KEYBOARD )  GetUpdateMode()  
IsFormatted() )
 DoOnlineSpelling();
 else
 aOnlineSpellTimer.Start();
diff --git a/formula/source/ui/dlg/formula.cxx 
b/formula/source/ui/dlg/formula.cxx
index 97ef663..3b7c2ec 100644
--- a/formula/source/ui/dlg/formula.cxx
+++ b/formula/source/ui/dlg/formula.cxx
@@ -592,7 +592,7 @@ sal_Bool FormulaDlg_Impl::CalcValue( const String rStrExp, 
String rStrResult )
 {
 // Only calculate the value when there isn't any more keyboard input:
 
-if ( !Application::AnyInput( INPUT_KEYBOARD ) )
+if ( !Application::AnyInput( VCL_INPUT_KEYBOARD ) )
 {
 bResult = m_pHelper-calculateValue(rStrExp,rStrResult);
 }
@@ -630,7 +630,7 @@ sal_Bool FormulaDlg_Impl::CalcStruct( const String rStrExp)
 {
 // Only calculate the value when there isn't any more keyboard input:
 
-if ( !Application::AnyInput( INPUT_KEYBOARD ) )
+if ( !Application::AnyInput( VCL_INPUT_KEYBOARD ) )
 {
 pStructPage-ClearStruct();
 
diff --git a/sc/source/core/tool/chartlis.cxx b/sc/source/core/tool/chartlis.cxx
index 8c52fe4..6f9129d 100644
--- a/sc/source/core/tool/chartlis.cxx
+++ b/sc/source/core/tool/chartlis.cxx
@@ -567,7 +567,7 @@ void ScChartListenerCollection::StartTimer()
 
 IMPL_LINK( ScChartListenerCollection, TimerHdl, Timer*, EMPTYARG )
 {
-if ( Application::AnyInput( INPUT_KEYBOARD ) )
+if ( Application::AnyInput( VCL_INPUT_KEYBOARD ) )
 {
 aTimer.Start();
 return 0;
diff --git a/sc/source/ui/app/scmod.cxx b/sc/source/ui/app/scmod.cxx
index 7932fca..b4284e7 100644
--- a/sc/source/ui/app/scmod.cxx
+++ b/sc/source/ui/app/scmod.cxx
@@ -1877,7 +1877,7 @@ IMPL_LINK( ScModule, IdleHandler, Timer*, EMPTYARG )
 
 IMPL_LINK( ScModule, SpellTimerHdl, Timer*, EMPTYARG )
 {
-if ( Application::AnyInput( INPUT_KEYBOARD ) )
+if ( Application::AnyInput( VCL_INPUT_KEYBOARD ) )
 {
 aSpellTimer.Start();
 return 0;   // dann spaeter wieder...
diff --git a/sd/source/ui/tools/IdleDetection.cxx 
b/sd/source/ui/tools/IdleDetection.cxx
index 4ba6163..e0fba1a 100644
--- a/sd/source/ui/tools/IdleDetection.cxx
+++ b/sd/source/ui/tools/IdleDetection.cxx
@@ -59,7 +59,7 @@ sal_Int32 IdleDetection::GetIdleState (const ::Window* 
pWindow)
 
 sal_Int32 IdleDetection::CheckInputPending (void)
 {
-if (GetpApp()-AnyInput(INPUT_MOUSE | INPUT_KEYBOARD | INPUT_PAINT))
+if (GetpApp()-AnyInput(VCL_INPUT_MOUSE | VCL_INPUT_KEYBOARD | 
INPUT_PAINT))
 return IDET_SYSTEM_EVENT_PENDING;
 else
 return IDET_IDLE;
diff --git a/svtools/source/control/inettbc.cxx 
b/svtools/source/control/inettbc.cxx
index 50739d8..097a8ad 100644
--- a/svtools/source/control/inettbc.cxx
+++ 

[Libreoffice-commits] .: editeng/source

2011-09-28 Thread Tor Lillqvist
 editeng/source/uno/unotext.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 0a56437e9ce6a228643e2831fe4877e8597cfa6b
Author: Tor Lillqvist t...@iki.fi
Date:   Wed Sep 28 23:23:33 2011 +0300

WaE: base class should be explicitly initialized in the copy constructor

diff --git a/editeng/source/uno/unotext.cxx b/editeng/source/uno/unotext.cxx
index 627f94b..ac241d2 100644
--- a/editeng/source/uno/unotext.cxx
+++ b/editeng/source/uno/unotext.cxx
@@ -214,6 +214,7 @@ SvxUnoTextRangeBase::SvxUnoTextRangeBase( const 
SvxEditSource* pSource, const Sv
 
 SvxUnoTextRangeBase::SvxUnoTextRangeBase( const SvxUnoTextRangeBase rRange ) 
throw()
 :   SvxUnoTextRangeBase_Base()
+,   osl::DebugBaseSvxUnoTextRangeBase()
 ,   mpPropSet(rRange.getPropertySet())
 {
 SolarMutexGuard aGuard;
___
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits