[Bug 54157] LibreOffice 3.7/4.0 most annoying bugs

2012-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

Jean-Baptiste Faure jbf.fa...@sud-ouest.org changed:

   What|Removed |Added

 Depends on||51300

--- Comment #47 from Jean-Baptiste Faure jbf.fa...@sud-ouest.org ---
I nominate bug 51300 : comments in XLS files are shown in white on yellow, so
unreadable.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - cui/source vcl/inc vcl/source

2012-12-13 Thread Libreoffice Gerrit user
 cui/source/dialogs/about.cxx |   39 ++--
 vcl/inc/vcl/svapp.hxx|1 
 vcl/inc/vcl/svgdata.hxx  |1 
 vcl/source/app/brand.cxx |   46 ---
 vcl/source/gdi/svgdata.cxx   |   29 +++
 5 files changed, 55 insertions(+), 61 deletions(-)

New commits:
commit b739389dba055f300d2e738474f468a1c672dcfb
Author: Muthu Subramanian sumu...@suse.com
Date:   Thu Dec 13 12:37:18 2012 +0530

fdo#56402: SVG in About Dialog.

diff --git a/cui/source/dialogs/about.cxx b/cui/source/dialogs/about.cxx
index ddc3b46..cc62e89 100644
--- a/cui/source/dialogs/about.cxx
+++ b/cui/source/dialogs/about.cxx
@@ -207,22 +207,19 @@ void AboutDialog::LayoutControls()
 Size aLogoSize( aIdealTextWidth, aIdealTextWidth / 20 );
 Point aLogoPos( 0, 0 );
 
-#ifdef FIXME_REMOVE_WHEN_RE_BASE_COMPLETE
-vcl::RenderGraphicRasterizer aRasterizerLogo = 
Application::LoadBrandSVG(flat_logo);
-if ( !aRasterizerLogo.GetRenderGraphic().IsEmpty() 
- aRasterizerLogo.GetDefaultSizePixel().Width()  0  
aRasterizerLogo.GetDefaultSizePixel().Height()  0 )
+if( Application::LoadBrandSVG(flat_logo, aLogoBitmap) 
+!aLogoBitmap.IsEmpty() )
 {
-const float aLogoWidthHeightRatio = 
(float)aRasterizerLogo.GetDefaultSizePixel().Width() / 
(float)aRasterizerLogo.GetDefaultSizePixel().Height();
+const float aLogoWidthHeightRatio = 
(float)aLogoBitmap.GetSizePixel().Width() / 
(float)aLogoBitmap.GetSizePixel().Height();
 aLogoSize.Width() = aDialogSize.Width() ;
 aLogoSize.Height() = aLogoSize.Width() / aLogoWidthHeightRatio ;
+aLogoBitmap.Scale(aLogoSize);
 
-aLogoBitmap = aRasterizerLogo.Rasterize( aLogoSize );
 aLogoImage.SetImage( Image( aLogoBitmap ) );
 aLogoImage.SetPosSizePixel( aLogoPos, aLogoSize );
 aLogoImage.Show();
 }
 else
-#endif
 {
 aLogoPos.X() = aDialogBorder;
 aLogoPos.Y() = aDialogBorder;
@@ -277,27 +274,21 @@ void AboutDialog::LayoutControls()
 
 
 // Layout background image
-#ifdef FIXME_REMOVE_WHEN_RE_BASE_COMPLETE
-if ( 
!(Application::GetSettings().GetStyleSettings().GetHighContrastMode()) )   {
-vcl::RenderGraphicRasterizer aRasterizerBackground = 
Application::LoadBrandSVG(shell/about);
+if ( 
!(Application::GetSettings().GetStyleSettings().GetHighContrastMode()) 
+  Application::LoadBrandSVG(shell/about, aBackgroundBitmap) 
+  !aBackgroundBitmap.IsEmpty() )
+{
+const float aBackgroundWidthHeightRatio = 
(float)aBackgroundBitmap.GetSizePixel().Width() /
+(float)aBackgroundBitmap.GetSizePixel().Height();
+Size aBackgroundSize (aDialogSize.Width(), aDialogSize.Width() / 
aBackgroundWidthHeightRatio );
 
-if ( !aRasterizerBackground.GetRenderGraphic().IsEmpty() 
-aRasterizerBackground.GetDefaultSizePixel().Width()  0  
aRasterizerBackground.GetDefaultSizePixel().Height()  0 )
+if ( aBackgroundSize.Height()  aDialogSize.Height())
 {
-const float aBackgroundWidthHeightRatio = 
(float)aRasterizerBackground.GetDefaultSizePixel().Width() /
-   
(float)aRasterizerBackground.GetDefaultSizePixel().Height();
-Size aBackgroundSize (aDialogSize.Width(), aDialogSize.Width() / 
aBackgroundWidthHeightRatio );
-
-if ( aBackgroundSize.Height()  aDialogSize.Height())
-{
-aBackgroundSize.Width() = aDialogSize.Height() * 
aBackgroundWidthHeightRatio ;
-aBackgroundSize.Height() = aDialogSize.Height();
-}
-
-aBackgroundBitmap = aRasterizerBackground.Rasterize( 
aBackgroundSize );
+aBackgroundSize.Width() = aDialogSize.Height() * 
aBackgroundWidthHeightRatio ;
+aBackgroundSize.Height() = aDialogSize.Height();
 }
+aBackgroundBitmap.Scale(aBackgroundSize);
 }
-#endif
 
 SetOutputSizePixel( aDialogSize );
 
diff --git a/vcl/inc/vcl/svapp.hxx b/vcl/inc/vcl/svapp.hxx
index d95e44b..9715b1b 100644
--- a/vcl/inc/vcl/svapp.hxx
+++ b/vcl/inc/vcl/svapp.hxx
@@ -267,6 +267,7 @@ public:
 static void SetAppName( const String rUniqueName );
 static String   GetAppName();
 static bool LoadBrandBitmap (const char* pName, BitmapEx 
rBitmap);
+static bool LoadBrandSVG( const char *pName, BitmapEx 
rBitmap );
 
 // default name of the application for message dialogs and printing
 static void SetDisplayName( const UniString rDisplayName 
);
diff --git a/vcl/inc/vcl/svgdata.hxx b/vcl/inc/vcl/svgdata.hxx
index 488092c..dc4c9e5 100644
--- a/vcl/inc/vcl/svgdata.hxx
+++ b/vcl/inc/vcl/svgdata.hxx
@@ -57,6 +57,7 @@ private:
 
 public:
 SvgData(const SvgDataArray rSvgDataArray, sal_uInt32 nSvgDataArrayLength, 
const rtl::OUString rPath);
+SvgData(const 

[PATCH] Change in core[libreoffice-3-6]: Take extra cache/uno_packages/*.tmp_/*properties files into ...

2012-12-13 Thread Stephan Bergmann (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1313

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/13/1313/1

Take extra cache/uno_packages/*.tmp_/*properties files into account

...when migrating user extensions from an old user profile.

Change-Id: I4425eba1e840bfdb33782bb3ed0542ce48b2f75f
(cherry picked from commit bd9c4510ff13ab7e935ed12c5b6d1f132c6f2b79)
---
M desktop/source/migration/services/oo3extensionmigration.cxx
1 file changed, 14 insertions(+), 11 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4425eba1e840bfdb33782bb3ed0542ce48b2f75f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Help with updating types.rdb

2012-12-13 Thread Lionel Elie Mamane
On Wed, Dec 12, 2012 at 01:12:03PM -0500, Kohei Yoshida wrote:

 I'm trying to unpublish several UNO API as discussed in ESC meeting.
 I've done unpublishing but am struggling with updating the
 types.rdb.

 I'm running

 make cmd cmd='./types_reference/update-rdb.sh
 com/sun/star/sheet/DataPilotSource.idl (...)
 inside offapi directory, and I'm getting
 
 make: *** No rule to make target `cmd'.  Stop.

Running from the top-level directory seems to do *something* at least :)

$ make cmd cmd='./offapi/type_reference/update-rdb.sh
com/sun/star/sheet/DataPilotSource.idl'
make -r -f
/home/master/src/libreoffice/workdirs/libreoffice-4.0/Makefile.top cmd
make[1]: Entering directory
`/home/master/src/libreoffice/workdirs/libreoffice-4.0'
echo custom cmd  ( ./offapi/type_reference/update-rdb.sh 
com/sun/star/sheet/DataPilotSource.idl )
custom cmd
sanity check diff:
1c1
 Registry 
file:///home/master/src/libreoffice/workdirs/libreoffice-4.0/offapi/type_reference/types.rdb:
---
 Registry 
 file:///home/master/src/libreoffice/workdirs/libreoffice-4.0/tmp/out2.rdb:
make[1]: Leaving 
directory`/home/master/src/libreoffice/workdirs/libreoffice-4.0'


-- 
Lionel
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'distro/suse/suse-3.6' - 3 commits - sw/inc sw/source

2012-12-13 Thread Libreoffice Gerrit user
 sw/inc/iodetect.hxx  |3 +--
 sw/source/filter/basflt/iodetect.cxx |   15 ---
 sw/source/ui/uno/swdet2.cxx  |2 +-
 3 files changed, 10 insertions(+), 10 deletions(-)

New commits:
commit 4a60855a3bb42e6f7f9474314d496e8fe8188c90
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Nov 29 09:48:24 2012 +

.dot files sometimes rejected by writer, bnc#793422

because CWW8 appears twice, once for normal .doc and once for .dot. So a 
.dot
files is tested twice if it's supported by the WW8 filter. Depending on the
(effectively arbitrary) order they appear in the list .dots may fail the 
is a
normal non template .doc test after the is a template .dot test and get
rejected as a CWW8 candidate.

Change-Id: I88aec29fdd5f9ec4dd4ad2813ff3c6b8fa5c5461
Signed-off-by: Tor Lillqvist tlillqv...@suse.com

diff --git a/sw/source/filter/basflt/iodetect.cxx 
b/sw/source/filter/basflt/iodetect.cxx
index b99d57c..f32fa1c 100644
--- a/sw/source/filter/basflt/iodetect.cxx
+++ b/sw/source/filter/basflt/iodetect.cxx
@@ -266,6 +266,12 @@ sal_Bool SwIoSystem::IsFileFilter(SfxMedium rMedium, 
const String rFmtName)
 }
 }
 }
+//The same underlying filter can appear multiple times in the
+//filter list, e.g. CWW8 filter twice, once for .doc and once for
+//.dot.  We just care here if its either, not enforce that it's
+//both which would be a bit of an odd requirement
+if (bRet)
+break;
 }
 
 pFltr = aIter.Next();
commit 20938c1b7a1f3a536c045d8e4fad6b04e1371c27
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Nov 29 09:29:19 2012 +

optional ppFlt argument never used

Change-Id: Ib2c68091f14d04818ae5e7db0ad7b8a1ec151f4f
Signed-off-by: Tor Lillqvist tlillqv...@suse.com

diff --git a/sw/inc/iodetect.hxx b/sw/inc/iodetect.hxx
index cc58b29..f8ff622 100644
--- a/sw/inc/iodetect.hxx
+++ b/sw/inc/iodetect.hxx
@@ -111,8 +111,7 @@ public:
 
 // Detect whether the given file is in the given format.
 // For now, only our own filters are supported!
-static sal_Bool IsFileFilter( SfxMedium rMedium, const String rFmtName,
-const SfxFilter** ppFlt = 0 );
+static sal_Bool IsFileFilter(SfxMedium rMedium, const String rFmtName);
 
 static sal_Bool IsValidStgFilter( SotStorage , const SfxFilter );
 static sal_Bool IsValidStgFilter( const com::sun::star::uno::Reference  
com::sun::star::embed::XStorage  rStg, const SfxFilter rFilter);
diff --git a/sw/source/filter/basflt/iodetect.cxx 
b/sw/source/filter/basflt/iodetect.cxx
index 7a72a6f..b99d57c 100644
--- a/sw/source/filter/basflt/iodetect.cxx
+++ b/sw/source/filter/basflt/iodetect.cxx
@@ -211,8 +211,7 @@ void TerminateBuffer(sal_Char *pBuffer, sal_uLong 
nBytesRead, sal_uLong nBufferL
 
 /* Feststellen ob das File in dem entsprechenden Format vorliegt. */
 /* Z.z werden nur unsere eigene Filter unterstuetzt   */
-sal_Bool SwIoSystem::IsFileFilter( SfxMedium rMedium, const String rFmtName,
-const SfxFilter** ppFilter )
+sal_Bool SwIoSystem::IsFileFilter(SfxMedium rMedium, const String rFmtName)
 {
 sal_Bool bRet = sal_False;
 
@@ -267,9 +266,6 @@ sal_Bool SwIoSystem::IsFileFilter( SfxMedium rMedium, 
const String rFmtName,
 }
 }
 }
-
-if( bRet  ppFilter  )
-*ppFilter = pFltr;
 }
 
 pFltr = aIter.Next();
diff --git a/sw/source/ui/uno/swdet2.cxx b/sw/source/ui/uno/swdet2.cxx
index 40e4be5..5594438 100644
--- a/sw/source/ui/uno/swdet2.cxx
+++ b/sw/source/ui/uno/swdet2.cxx
@@ -65,7 +65,7 @@ sal_uLong SwFilterDetect::DetectFilter( SfxMedium rMedium, 
const SfxFilter** pp
 String aPrefFlt = (*ppFilter)-GetUserData();
 
 // detection for TextFilter needs an additional checking
-sal_Bool bDetected = SwIoSystem::IsFileFilter( rMedium, aPrefFlt );
+sal_Bool bDetected = SwIoSystem::IsFileFilter(rMedium, aPrefFlt);
 return bDetected ? nRet : ERRCODE_ABORT;
 }
 
commit 9e16b4766a76b48bcb38c7d816975a3aae89ce2c
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Nov 29 09:23:37 2012 +

Simplify SwIoSystem::IsFileFilter

bRet = bRet  (pFltr-GetUserData().equals(rFmtName)); is already inside 
an if
bRet = bRet  (pFltr-GetUserData().equals(rFmtName)); condition

bRet = xStg.Is()  IsValidStgFilter( *xStg, *pFltr ); is already inside an
if ( xStg.Is() ) condition

Change-Id: I2115041d99b6fcbae7a829557c37627d03b5ee24
Signed-off-by: Tor Lillqvist tlillqv...@suse.com

diff --git a/sw/source/filter/basflt/iodetect.cxx 
b/sw/source/filter/basflt/iodetect.cxx
index 41adf67..7a72a6f 100644
--- a/sw/source/filter/basflt/iodetect.cxx
+++ b/sw/source/filter/basflt/iodetect.cxx
@@ -244,8 +244,7 @@ sal_Bool SwIoSystem::IsFileFilter( SfxMedium 

Re: is 'make check' supposed to work on ~master ?

2012-12-13 Thread Stephan Bergmann

On 12/12/2012 06:36 PM, John Smith wrote:

Im trying to run 'make check' at toplevel on ~master. It fails for me
at this point (soffice.bin crashes) :

make 
/usr/local/src/libreoffice/workdir/unxlngi6.pro/JunitTest/comphelper_complex/done


Is that crash reproducible for you?  The backtrace doesn't give much of 
a clue to why it fails, so if it is reproducible running under Valgrind 
might help.


In general, the online update mechanism spawns threads that it doesn't 
properly join again before exit, which is a notorious problem with tests 
(that often run quickly enough to terminate soffice.bin while the online 
update check is still in progress), so I recommend to configure with 
--disable-online-update (which is the implicit default on Linux anyway, 
though).  But the backtrace here doesn't look like it is caused by that 
problem anyway.


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: is 'make check' supposed to work on ~master ?

2012-12-13 Thread Noel Grandin


On 2012-12-13 11:35, Stephan Bergmann wrote:


In general, the online update mechanism spawns threads that it doesn't 
properly join again before exit, which is a notorious problem with 
tests (that often run quickly enough to terminate soffice.bin while 
the online update check is still in progress),


Is it not possible to pass in a parameter disabling the 
online-update-check? That should make the tests happier.


Disclaimer: http://www.peralex.com/disclaimer.html


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


what's with Rectangle::GetHeight() and GetWidth() in gen.hxx

2012-12-13 Thread Noel Grandin

Hi

I seem to have triggered a problem on the windows tinderbox, but I'm 
struggling to understand why.

The error on the tinderbox looks like:

   [build LNK] Library/ires.lib
   Creating library 
C:/cygwin/home/tinderbox/master-build/workdir/wntmsci13.pro/LinkTarget/Library/iembobj.lib 

   and object 
C:/cygwin/home/tinderbox/master-build/workdir/wntmsci13.pro/LinkTarget/Library/iembobj.exp
   docholder.o : error LNK2019: unresolved external symbol 
__declspec(dllimport) public: long __thiscall 
Rectangle::GetHeight(void)const  (__imp_?GetHeight@Rectangle@@QBEJXZ)
   referenced in function 
__catch$?GetDocFrame@DocumentHolder@@AAE?AV?$Reference@VXFrame@frame@star@sun@com@@@uno@star@sun@com@@XZ$0


And the definition in gen.hxx looks like:

  class TOOLS_DLLPUBLIC SAL_WARN_UNUSED Rectangle
  {
  public:
 longGetWidth() const;
 longGetHeight() const;
 }
 inline long Rectangle::GetWidth() const
 {
   .
 }

Firstly, there seems to be a missing inline statement in the class 
header section.
Secondly, given that there __is__ an inline modifier, why would the 
windows compiler be trying to link this function externally?


Regards, Noel Grandin



Disclaimer: http://www.peralex.com/disclaimer.html


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: is 'make check' supposed to work on ~master ?

2012-12-13 Thread Stephan Bergmann

On 12/13/2012 10:37 AM, Noel Grandin wrote:

On 2012-12-13 11:35, Stephan Bergmann wrote:

In general, the online update mechanism spawns threads that it doesn't
properly join again before exit, which is a notorious problem with
tests (that often run quickly enough to terminate soffice.bin while
the online update check is still in progress),


Is it not possible to pass in a parameter disabling the
online-update-check? That should make the tests happier.


I have the faint memory that we indeed somehow disabled it at least for 
some of the tests in the past, but can't find any relevant code. 
(There's apparently no such command-line parameter for soffice, and the 
code at extensions/source/update/check/updatecheckjob.cxx also does not 
check any environment variable or similar.)


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: is 'make check' supposed to work on ~master ?

2012-12-13 Thread Noel Grandin


On 2012-12-13 11:53, Stephan Bergmann wrote:

On 12/13/2012 10:37 AM, Noel Grandin wrote:

On 2012-12-13 11:35, Stephan Bergmann wrote:

In general, the online update mechanism spawns threads that it doesn't
properly join again before exit, which is a notorious problem with
tests (that often run quickly enough to terminate soffice.bin while
the online update check is still in progress),


Is it not possible to pass in a parameter disabling the
online-update-check? That should make the tests happier.


I have the faint memory that we indeed somehow disabled it at least 
for some of the tests in the past, but can't find any relevant code. 
(There's apparently no such command-line parameter for soffice, and 
the code at extensions/source/update/check/updatecheckjob.cxx also 
does not check any environment variable or similar.)




Sounds like something I could log as a MediumHack?

Disclaimer: http://www.peralex.com/disclaimer.html


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'distro/suse/suse-3.6' - oox/source

2012-12-13 Thread Libreoffice Gerrit user
 oox/source/vml/vmltextboxcontext.cxx |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 2c9e1123658c1448b8fe0b0e4295578ad1b183b0
Author: Miklos Vajna vmik...@suse.cz
Date:   Mon Dec 10 12:51:01 2012 +0100

oox: v:textbox's inset attribute should be imported in Hmm, not in Emu

Trivial reproducer: 
http://people.freedesktop.org/~vmiklos/2012/simple-textbox.docx

Change-Id: I030f7153326affc5a01e7ede1ddf1164fa500071
(cherry picked from commit 5bc835d48f1e5373b3a31bd166fadfdf19b3588f)

diff --git a/oox/source/vml/vmltextboxcontext.cxx 
b/oox/source/vml/vmltextboxcontext.cxx
index 841c150..3f188e3 100644
--- a/oox/source/vml/vmltextboxcontext.cxx
+++ b/oox/source/vml/vmltextboxcontext.cxx
@@ -137,16 +137,16 @@ TextBoxContext::TextBoxContext( ContextHandler2Helper 
rParent, TextBox rTextBo
 OUString inset = rAttribs.getString( XML_inset ).get();
 OUString value;
 ConversionHelper::separatePair( value, inset, inset, ',' );
-rTextBox.borderDistanceLeft = ConversionHelper::decodeMeasureToEmu( 
graphicHelper,
+rTextBox.borderDistanceLeft = ConversionHelper::decodeMeasureToHmm( 
graphicHelper,
 value.isEmpty() ? 0.1in : value, 0, false, false );
 ConversionHelper::separatePair( value, inset, inset, ',' );
-rTextBox.borderDistanceTop = ConversionHelper::decodeMeasureToEmu( 
graphicHelper,
+rTextBox.borderDistanceTop = ConversionHelper::decodeMeasureToHmm( 
graphicHelper,
 value.isEmpty() ? 0.05in : value, 0, false, false );
 ConversionHelper::separatePair( value, inset, inset, ',' );
-rTextBox.borderDistanceRight = ConversionHelper::decodeMeasureToEmu( 
graphicHelper,
+rTextBox.borderDistanceRight = ConversionHelper::decodeMeasureToHmm( 
graphicHelper,
 value.isEmpty() ? 0.1in : value, 0, false, false );
 ConversionHelper::separatePair( value, inset, inset, ',' );
-rTextBox.borderDistanceBottom = ConversionHelper::decodeMeasureToEmu( 
graphicHelper,
+rTextBox.borderDistanceBottom = ConversionHelper::decodeMeasureToHmm( 
graphicHelper,
 value.isEmpty() ? 0.05in : value, 0, false, false );
 rTextBox.borderDistanceSet = true;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: what's with Rectangle::GetHeight() and GetWidth() in gen.hxx

2012-12-13 Thread Stephan Bergmann

On 12/13/2012 10:48 AM, Noel Grandin wrote:

I seem to have triggered a problem on the windows tinderbox, but I'm
struggling to understand why.
The error on the tinderbox looks like:

[build LNK] Library/ires.lib
Creating library
C:/cygwin/home/tinderbox/master-build/workdir/wntmsci13.pro/LinkTarget/Library/iembobj.lib

and object
C:/cygwin/home/tinderbox/master-build/workdir/wntmsci13.pro/LinkTarget/Library/iembobj.exp

docholder.o : error LNK2019: unresolved external symbol
__declspec(dllimport) public: long __thiscall
Rectangle::GetHeight(void)const  (__imp_?GetHeight@Rectangle@@QBEJXZ)
referenced in function
__catch$?GetDocFrame@DocumentHolder@@AAE?AV?$Reference@VXFrame@frame@star@sun@com@@@uno@star@sun@com@@XZ$0


And the definition in gen.hxx looks like:

   class TOOLS_DLLPUBLIC SAL_WARN_UNUSED Rectangle
   {
   public:
  longGetWidth() const;
  longGetHeight() const;
  }
  inline long Rectangle::GetWidth() const
  {
.
  }

Firstly, there seems to be a missing inline statement in the class
header section.


The inline in the declaration of Rectangle::GetHeight should not 
strictly be necessary (though it would be good style anyway), but maybe 
the TOOLS_DLLPUBLIC on the enclosing class definition does make it 
relevant for MSC.


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: is 'make check' supposed to work on ~master ?

2012-12-13 Thread John Smith
On Thu, Dec 13, 2012 at 10:35 AM, Stephan Bergmann sberg...@redhat.com wrote:
 On 12/12/2012 06:36 PM, John Smith wrote:

 Im trying to run 'make check' at toplevel on ~master. It fails for me
 at this point (soffice.bin crashes) :

 make
 /usr/local/src/libreoffice/workdir/unxlngi6.pro/JunitTest/comphelper_complex/done


 Is that crash reproducible for you?  The backtrace doesn't give much of a
 clue to why it fails, so if it is reproducible running under Valgrind might
 help.

 In general, the online update mechanism spawns threads that it doesn't
 properly join again before exit, which is a notorious problem with tests
 (that often run quickly enough to terminate soffice.bin while the online
 update check is still in progress), so I recommend to configure with
 --disable-online-update (which is the implicit default on Linux anyway,
 though).  But the backtrace here doesn't look like it is caused by that
 problem anyway.

It's pretty much reproducable for me: I re-tried make a few times. Ill
retry with ./configure --disable-online-update  make build  make
check, and see if it still fails.


- John Smith
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Remove save to old Microsoft office formats

2012-12-13 Thread Michael Meeks
Hi Bryan,

On Wed, 2012-12-12 at 18:05 -0500, Bryan Quigley wrote:
 This patch removes the ability to save to old office 6/95 formats.
 I've also opened a bug on it here:
 https://bugs.freedesktop.org/show_bug.cgi?id=53471

Wow - that's bold :-) indeed there are loads of problems with fidelity
in the old formats, and of course maintenance pain  testing uncertainty
around them. Disabling them by default in the UI (with some setting to
have them back on ?) might be better at least for a release - and lets
see if anyone screams (?).

I assume the root problem you want to fix is of users accidentally
picking the wrong format in the filter / drop-down and getting a
needlessly bad experience here ?

Added to the ESC agenda for today anyhow ;-)

 It's a very trivial patch but:
 This contribution to LibreOffice may be licensed under the MPL/LGPLv3+
 dual license.  This patch was also written before I joined Canonical. 

Great to have your contribution !

All the best,

Michael.

-- 
michael.me...@suse.com  , Pseudo Engineer, itinerant idiot

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: offapi/type_reference registry/source

2012-12-13 Thread Libreoffice Gerrit user
 offapi/type_reference/update-rdb.sh |4 ++--
 registry/source/registry.cxx|2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 53fbd30b3cd6465251b1b07f4728813d524fcd82
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Dec 13 11:03:03 2012 +0100

Make registry's mergeKey fail if it cannot process regFileName

...so that regmerge fails for bad regfile arguments.  Add -v to the regmerge
calls in update-rdb.sh so that an error message is printed in case of 
failure.

Change-Id: I6613b9756b4e5628772541580938f6c80fd9bc9e

diff --git a/offapi/type_reference/update-rdb.sh 
b/offapi/type_reference/update-rdb.sh
index 0f41721..79fd29c 100755
--- a/offapi/type_reference/update-rdb.sh
+++ b/offapi/type_reference/update-rdb.sh
@@ -24,10 +24,10 @@ fi
 
 mkdir tmp
 for i in $@; do
-  ${OUTDIR_FOR_BUILD?}/bin/regmerge tmp/out1.rdb /UCR \
+  ${OUTDIR_FOR_BUILD?}/bin/regmerge -v tmp/out1.rdb /UCR \
 ${WORKDIR_FOR_BUILD?}/UnoApiPartTarget/${i%.idl}.urd
 done
-${OUTDIR_FOR_BUILD?}/bin/regmerge tmp/out2.rdb / \
+${OUTDIR_FOR_BUILD?}/bin/regmerge -v tmp/out2.rdb / \
   ${SRC_ROOT?}/offapi/type_reference/types.rdb tmp/out1.rdb
 echo sanity check diff:
 diff (${OUTDIR_FOR_BUILD?}/bin/regview \
diff --git a/registry/source/registry.cxx b/registry/source/registry.cxx
index 5851199..345631a 100644
--- a/registry/source/registry.cxx
+++ b/registry/source/registry.cxx
@@ -361,7 +361,7 @@ static RegError REGISTRY_CALLTYPE mergeKey(RegHandle hReg,
 return _ret;
 
 _ret = pReg-loadKey(pNewKey, regFileName, bWarnings, bReport);
-if (_ret == REG_MERGE_ERROR || (_ret == REG_MERGE_CONFLICT  
bWarnings))
+if (_ret != REG_NO_ERROR  (_ret != REG_MERGE_CONFLICT || bWarnings))
 {
 if (pNewKey != pKey)
 (void) pKey-closeKey(pNewKey);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: when to backport

2012-12-13 Thread Luke Deller

On 13/12/12 08:17, Michael Meeks wrote:
...

Does that help ?


Yes thanks Michael :-)

Luke.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Help with updating types.rdb

2012-12-13 Thread Stephan Bergmann

On 12/13/2012 09:41 AM, Lionel Elie Mamane wrote:

On Wed, Dec 12, 2012 at 01:12:03PM -0500, Kohei Yoshida wrote:

I'm trying to unpublish several UNO API as discussed in ESC meeting.
I've done unpublishing but am struggling with updating the
types.rdb.



I'm running



make cmd cmd='./types_reference/update-rdb.sh
com/sun/star/sheet/DataPilotSource.idl (...)
inside offapi directory, and I'm getting

make: *** No rule to make target `cmd'.  Stop.


Running from the top-level directory seems to do *something* at least :)


What's still missing is to adapt the path to the .idl file (i.e., add an 
offapi/ prefix).  regmerge (even in verbose mode) unfortunately did 
not fail for input files that do not exist; that's changed now with 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=53fbd30b3cd6465251b1b07f4728813d524fcd82 
Make registry's mergeKey fail if it cannot process regFileName.


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Help with updating types.rdb

2012-12-13 Thread Stephan Bergmann

On 12/12/2012 07:12 PM, Kohei Yoshida wrote:

I'm trying to unpublish several UNO API as discussed in ESC meeting.
I've done unpublishing but am struggling with updating the types.rdb.

[...]

Help appreciated.


Seeing that 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=0315ef5bbc168896b1d7d8ac5c1657f861fa6a93 
API CHANGE: unpublish Data Pilot interfaces contains changes to 
offapi/type_reference/types.rdb that amount to removing the published 
flag from certain interfaces and services, I assume that you managed to 
get that done after all?


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: is 'make check' supposed to work on ~master ?

2012-12-13 Thread Stephan Bergmann

On 12/13/2012 11:00 AM, Noel Grandin wrote:

On 2012-12-13 11:53, Stephan Bergmann wrote:

On 12/13/2012 10:37 AM, Noel Grandin wrote:

On 2012-12-13 11:35, Stephan Bergmann wrote:

In general, the online update mechanism spawns threads that it doesn't
properly join again before exit, which is a notorious problem with
tests (that often run quickly enough to terminate soffice.bin while
the online update check is still in progress),


Is it not possible to pass in a parameter disabling the
online-update-check? That should make the tests happier.


I have the faint memory that we indeed somehow disabled it at least
for some of the tests in the past, but can't find any relevant code.
(There's apparently no such command-line parameter for soffice, and
the code at extensions/source/update/check/updatecheckjob.cxx also
does not check any environment variable or similar.)



Sounds like something I could log as a MediumHack?


Preferred hack would of course be to fix that online update code, but it 
looks like an opaque disaster at least to me...


Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: SDK examples

2012-12-13 Thread Stephan Bergmann

On 12/12/2012 07:28 PM, Thorsten Behrens wrote:

Stephan Bergmann wrote:

Indeed it is, odk/examples/DevelopersGuide/Accessibility/.  Looks
like 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=70626249cd247d9acdad417b8eaf252bae22c059
API CHANGE a11y unpublishing and add/removeListener rename. didn't
address that.


Sorry, after fiddling with this build system for a while - *how* are
you building that, if at all?!


Compiling the SDK examples?

* cd install/sdk

* ./setsdkenv_unix

answer all its questions (it creates a ~/libreoffice*_sdk/ directory 
where it stores the information for subsequent invocations), it leaves 
you in a sub-shell with the necessary environment variable settings, so


* cd examples/...  make

(Yes, all of that is ugly.)

Stephan
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Calc saving csv-files

2012-12-13 Thread Michael Meeks
Hi Hannu,

On Wed, 2012-12-12 at 10:36 +0200, Hannu Puhakka wrote:
 User get often shocking surprises saving csv-files. If the file is new, 
 the system asks some questions, but saving old files again get more 
 problems. As an attachment is an example.

This is the developers list; are you going to hack on this problem ? if
so this is the right place to discuss your patch / get some code
pointers and more. If not - please file a bug - that is the best place
to track your issue.

Thanks !

Michael.


-- 
michael.me...@suse.com  , Pseudo Engineer, itinerant idiot

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: desktop/source desktop/unx desktop/win32

2012-12-13 Thread Libreoffice Gerrit user
 desktop/source/app/officeipcthread.cxx |   40 +-
 desktop/source/app/officeipcthread.hxx |2 
 desktop/unx/source/start.c |   17 -
 desktop/win32/source/officeloader/officeloader.cxx |   60 -
 4 files changed, 88 insertions(+), 31 deletions(-)

New commits:
commit 0dce7eae55bf90d2a7171a1fb8663d66ba4ac6d3
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Tue Nov 20 11:03:03 2012 +0100

startup: more reliable startup of multiple instances

Until now, when a new soffice instance (S2) started and tried to
connect to an existing soffice process (S1), S2 may have failed to
boostrap due to race condition in communication over the shared pipe.

S1 can be shutdown after S2 connected to it but _before_ S1 handled its
arguments (code run after 'accept' method in OfficeIPCThread).
This patch introduces a new message, sent by the main soffice after it
has called accept if and only if it's not shutting down (see mbDowning
 member).
The other soffice waits for this message before enabling going in
 PIPE_CONNECTED mode. If soffice fails to receive this message, pipe mode is
left unchanged and after a quick pause, it will try again.

Change-Id: I2e099a5804e1e8dd535cfd31ef454cffa44efa62
Signed-off-by: Stephan Bergmann sberg...@redhat.com

diff --git a/desktop/source/app/officeipcthread.cxx 
b/desktop/source/app/officeipcthread.cxx
index 7047d69..7d477a7 100644
--- a/desktop/source/app/officeipcthread.cxx
+++ b/desktop/source/app/officeipcthread.cxx
@@ -57,6 +57,8 @@ const char  *OfficeIPCThread::sc_aShowSequence = -tofront;
 const int OfficeIPCThread::sc_nShSeqLength = 5;
 const char  *OfficeIPCThread::sc_aConfirmationSequence = 
InternalIPC::ProcessingDone;
 const int OfficeIPCThread::sc_nCSeqLength = 27;
+const char  *OfficeIPCThread::sc_aSendArgumentsSequence = 
InternalIPC::SendArguments;
+const int OfficeIPCThread::sc_nCSASeqLength = 26;
 
 namespace { static char const ARGUMENT_PREFIX[] = InternalIPC::Arguments; }
 
@@ -497,8 +499,31 @@ OfficeIPCThread::Status 
OfficeIPCThread::EnableOfficeIPCThread()
 }
 else if( pThread-maPipe.create( pThread-maPipeIdent.getStr(), 
osl_Pipe_OPEN, rSecurity )) // Creation not successfull, now we try to connect
 {
-// Pipe connected to first office
-nPipeMode = PIPEMODE_CONNECTED;
+osl::StreamPipe aStreamPipe(pThread-maPipe.getHandle());
+char pReceiveBuffer[sc_nCSASeqLength + 1];
+int nResult = 0;
+int nBytes = 0;
+int nBufSz = sc_nCSASeqLength + 1;
+// read byte per byte
+while ((nResult=aStreamPipe.recv( pReceiveBuffer+nBytes, 
nBufSz-nBytes))0) {
+nBytes += nResult;
+if (pReceiveBuffer[nBytes-1]=='\0') {
+break;
+}
+}
+if (rtl::OString(sc_aSendArgumentsSequence).equals(pReceiveBuffer))
+{
+// Pipe connected to first office
+nPipeMode = PIPEMODE_CONNECTED;
+}
+else
+{
+// Pipe connection failed (other office exited or crashed)
+TimeValue tval;
+tval.Seconds = 0;
+tval.Nanosec = 5;
+salhelper::Thread::wait( tval );
+}
 }
 else
 {
@@ -655,6 +680,17 @@ void OfficeIPCThread::execute()
 // down during wait
 osl::ClearableMutexGuard aGuard( GetMutex() );
 
+if (!mbDowning)
+{
+// notify client we're ready to process its args
+int nBytes = 0;
+int nResult = 0;
+while (
+(nResult = 
maStreamPipe.send(sc_aSendArgumentsSequence+nBytes, sc_nCSASeqLength-nBytes))0 

+((nBytes += nResult)  sc_nCSASeqLength) ) ;
+}
+maStreamPipe.write(\0, 1);
+
 // test byte by byte
 const int nBufSz = 2048;
 char pBuf[nBufSz];
diff --git a/desktop/source/app/officeipcthread.hxx 
b/desktop/source/app/officeipcthread.hxx
index 734119d..3dd7eac 100644
--- a/desktop/source/app/officeipcthread.hxx
+++ b/desktop/source/app/officeipcthread.hxx
@@ -94,6 +94,8 @@ class OfficeIPCThread : public salhelper::Thread
 static const int sc_nShSeqLength;
 static const char *sc_aConfirmationSequence;
 static const int sc_nCSeqLength;
+static const char *sc_aSendArgumentsSequence;
+static const int sc_nCSASeqLength;
 
 OfficeIPCThread();
 
diff --git a/desktop/unx/source/start.c b/desktop/unx/source/start.c
index 2d12d89..0e32c59 100644
--- a/desktop/unx/source/start.c
+++ b/desktop/unx/source/start.c
@@ -839,10 +839,19 @@ SAL_IMPLEMENT_MAIN_WITH_ARGS( argc, argv )
 
 if ( ( fd = connect_pipe( 

[PUSHED] startup: more reliable startup of multiple instances

2012-12-13 Thread Stephan Bergmann (via Code Review)
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1089


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2e099a5804e1e8dd535cfd31ef454cffa44efa62
Gerrit-PatchSet: 5
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Gerrit-Reviewer: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Gerrit-Reviewer: Stephan Bergmann sberg...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'feature/rb1' - 0 commits -

2012-12-13 Thread Libreoffice Gerrit user
Rebased ref, commits from common ancestor:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] Changes to 'feature/rb1'

2012-12-13 Thread Libreoffice Gerrit user
New branch 'feature/rb1' available with the following commits:
commit f9edb3d7802ad38dff2cea2a90070548ceeb875e
Author: Michael Meeks michael.me...@suse.com
Date:   Wed Dec 12 19:20:46 2012 +

re-base on ALv2 code. Includes:

Patch contributed by Oliver Rainer-Wittmann
i#118572 - remove ui string and help content regarding
usage of Java Mail in Writer's Mail Merge as Java
Mail is not used.
http://svn.apache.org/viewvc?view=revisionrevision=1197035

Patches contributed by Herbert Duerr
updated help text for regexp word boundaries
http://svn.apache.org/viewvc?view=revisionrevision=1228026
improved help text for word boundary regexp expression \b
http://svn.apache.org/viewvc?view=revisionrevision=1234738

Patches contributed by Jurgen Schmidt
remove onlineregistration with dependencies
http://svn.apache.org/viewvc?view=revisionrevision=1240245

Patch contributed by Regina Henschel
Extended tips for corner and cap style
http://svn.apache.org/viewvc?view=revisionrevision=1242287

* re-enable on-line update help, remove obsolete on-line purchasing.

commit 2700032f791ae80431ec7dc3429a963afa709fde
Author: Michael Meeks michael.me...@suse.com
Date:   Thu Dec 13 10:12:36 2012 +

Revert fdo#36057 time format using [] is needed to show hours non-modulo 
24

This reverts commit 05e4e6345e7503b162356cfcff36a8787a34e05f.

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - desktop/source desktop/unx desktop/win32

2012-12-13 Thread Libreoffice Gerrit user
 desktop/source/app/officeipcthread.cxx |   40 +-
 desktop/source/app/officeipcthread.hxx |2 
 desktop/unx/source/start.c |   17 -
 desktop/win32/source/officeloader/officeloader.cxx |   60 -
 4 files changed, 88 insertions(+), 31 deletions(-)

New commits:
commit 8376da60d5d272cf6b3ebee91934bbcd970c7658
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Tue Nov 20 11:03:03 2012 +0100

startup: more reliable startup of multiple instances

Until now, when a new soffice instance (S2) started and tried to
connect to an existing soffice process (S1), S2 may have failed to
boostrap due to race condition in communication over the shared pipe.

S1 can be shutdown after S2 connected to it but _before_ S1 handled its
arguments (code run after 'accept' method in OfficeIPCThread).
This patch introduces a new message, sent by the main soffice after it
has called accept if and only if it's not shutting down (see mbDowning
 member).
The other soffice waits for this message before enabling going in
 PIPE_CONNECTED mode. If soffice fails to receive this message, pipe mode is
left unchanged and after a quick pause, it will try again.

Change-Id: I2e099a5804e1e8dd535cfd31ef454cffa44efa62
Signed-off-by: Stephan Bergmann sberg...@redhat.com
(cherry picked from commit 0dce7eae55bf90d2a7171a1fb8663d66ba4ac6d3)

diff --git a/desktop/source/app/officeipcthread.cxx 
b/desktop/source/app/officeipcthread.cxx
index 7047d69..7d477a7 100644
--- a/desktop/source/app/officeipcthread.cxx
+++ b/desktop/source/app/officeipcthread.cxx
@@ -57,6 +57,8 @@ const char  *OfficeIPCThread::sc_aShowSequence = -tofront;
 const int OfficeIPCThread::sc_nShSeqLength = 5;
 const char  *OfficeIPCThread::sc_aConfirmationSequence = 
InternalIPC::ProcessingDone;
 const int OfficeIPCThread::sc_nCSeqLength = 27;
+const char  *OfficeIPCThread::sc_aSendArgumentsSequence = 
InternalIPC::SendArguments;
+const int OfficeIPCThread::sc_nCSASeqLength = 26;
 
 namespace { static char const ARGUMENT_PREFIX[] = InternalIPC::Arguments; }
 
@@ -497,8 +499,31 @@ OfficeIPCThread::Status 
OfficeIPCThread::EnableOfficeIPCThread()
 }
 else if( pThread-maPipe.create( pThread-maPipeIdent.getStr(), 
osl_Pipe_OPEN, rSecurity )) // Creation not successfull, now we try to connect
 {
-// Pipe connected to first office
-nPipeMode = PIPEMODE_CONNECTED;
+osl::StreamPipe aStreamPipe(pThread-maPipe.getHandle());
+char pReceiveBuffer[sc_nCSASeqLength + 1];
+int nResult = 0;
+int nBytes = 0;
+int nBufSz = sc_nCSASeqLength + 1;
+// read byte per byte
+while ((nResult=aStreamPipe.recv( pReceiveBuffer+nBytes, 
nBufSz-nBytes))0) {
+nBytes += nResult;
+if (pReceiveBuffer[nBytes-1]=='\0') {
+break;
+}
+}
+if (rtl::OString(sc_aSendArgumentsSequence).equals(pReceiveBuffer))
+{
+// Pipe connected to first office
+nPipeMode = PIPEMODE_CONNECTED;
+}
+else
+{
+// Pipe connection failed (other office exited or crashed)
+TimeValue tval;
+tval.Seconds = 0;
+tval.Nanosec = 5;
+salhelper::Thread::wait( tval );
+}
 }
 else
 {
@@ -655,6 +680,17 @@ void OfficeIPCThread::execute()
 // down during wait
 osl::ClearableMutexGuard aGuard( GetMutex() );
 
+if (!mbDowning)
+{
+// notify client we're ready to process its args
+int nBytes = 0;
+int nResult = 0;
+while (
+(nResult = 
maStreamPipe.send(sc_aSendArgumentsSequence+nBytes, sc_nCSASeqLength-nBytes))0 

+((nBytes += nResult)  sc_nCSASeqLength) ) ;
+}
+maStreamPipe.write(\0, 1);
+
 // test byte by byte
 const int nBufSz = 2048;
 char pBuf[nBufSz];
diff --git a/desktop/source/app/officeipcthread.hxx 
b/desktop/source/app/officeipcthread.hxx
index 734119d..3dd7eac 100644
--- a/desktop/source/app/officeipcthread.hxx
+++ b/desktop/source/app/officeipcthread.hxx
@@ -94,6 +94,8 @@ class OfficeIPCThread : public salhelper::Thread
 static const int sc_nShSeqLength;
 static const char *sc_aConfirmationSequence;
 static const int sc_nCSeqLength;
+static const char *sc_aSendArgumentsSequence;
+static const int sc_nCSASeqLength;
 
 OfficeIPCThread();
 
diff --git a/desktop/unx/source/start.c b/desktop/unx/source/start.c
index 2d12d89..0e32c59 100644
--- a/desktop/unx/source/start.c
+++ b/desktop/unx/source/start.c
@@ -839,10 +839,19 @@ 

[Libreoffice-commits] .: postprocess/packregistry

2012-12-13 Thread Libreoffice Gerrit user
 postprocess/packregistry/makefile.mk |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 82c53d537a05dadf4d7fd7ea41292897bf2d47c7
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Dec 13 12:03:54 2012 +0100

Missing dependency

(WriterWindowState.xcs is in writer.xcd)

Change-Id: I4e6d7cea64afae21cfb00ca9dc7873c5ce90d2bc

diff --git a/postprocess/packregistry/makefile.mk 
b/postprocess/packregistry/makefile.mk
index 3150607..f576a10 100644
--- a/postprocess/packregistry/makefile.mk
+++ b/postprocess/packregistry/makefile.mk
@@ -114,7 +114,7 @@ MY_FILES_impress = \
 $(MY_MOD)/org/openoffice/Office/ProtocolHandler-impress.xcu \
 $(MY_MOD)/org/openoffice/Setup-impress.xcu
 
-MY_DEPS_librelogo = main
+MY_DEPS_librelogo = main writer
 MY_FILES_librelogo = \
 $(MY_MOD)/org/openoffice/Office/Addons-librelogo.xcu \
 $(MY_MOD)/org/openoffice/Office/UI/WriterWindowState-librelogo.xcu
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Addin exceptions in SvStream

2012-12-13 Thread Marc-André Laverdière
That's the problem... what we don't know can be hurting us big time.
And since the tests are not very thorough, I can't have a good
confidence that things are not broken.

So, either we finish this patch and hope people will pick up those
cases along the way - a risky proposal but it has the advantage of
immunizing pretty much every filter for 'cheap'.

Otherwise, I suggest that we go for a SvStream wrapper that throws
exceptions - no changes to SvStream. It can be used wherever wanted,
so that's a lot of engineering effort to bolt that in place - but less
work than just putting stream-good() checks everywhere.
Marc-André LAVERDIÈRE
Perseverance must finish its work so that you may be mature and complete,
not lacking anything. -James 1:4
http://asimplediscipleslife.blogspot.com/
mlaverd.theunixplace.com



On Thu, Dec 13, 2012 at 4:40 AM, Michael Meeks michael.me...@suse.com wrote:
 Hi Marc,

 On Thu, 2012-12-13 at 02:09 -0500, Marc-André Laverdière wrote:
 I feel that this behavior of SvStream not to change the parameter in
 case of a broken stream should be changed. I am not sure why that is
 there.

 Well - it is -thought- (but hard to prove) that this behaviour is
 relied on by the code in a number of places:

 sal_uInt16 nFallbackStyleFoo = 0x;
 nStream  nFallbackStyleFoo;

 And that nil'ing it - though attractive -might- then break things.
 Clever ideas to find and fix those few cases appreciated - but in
 general combining that with re-working to use explicit read functions
 seems like a good (albeit time-consuming) idea.

 Of course, it'd be possible to imagine a boolean on the stream to
 return defined data in these cases - but that's a bit sick IMHO ;-) if
 we have to audit the code and test it, why not clean it to use (much
 safer) readInt16(); style methods :-)

 HTH,

 Michael.

 --
 michael.me...@suse.com  , Pseudo Engineer, itinerant idiot

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Bug 54157] LibreOffice 3.7/4.0 most annoying bugs

2012-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

Bug 54157 depends on bug 58142, which changed state.

Bug 58142 Summary: EDITING: CRASH when copy paste multiple pictures from file 
explorer
https://bugs.freedesktop.org/show_bug.cgi?id=58142

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - 2 commits - sw/source

2012-12-13 Thread Libreoffice Gerrit user
 sw/source/core/layout/atrfrm.cxx  |5 +
 sw/source/core/layout/trvlfrm.cxx |   14 --
 2 files changed, 13 insertions(+), 6 deletions(-)

New commits:
commit efa1a63d940c942c805f18696ce304aa47e04ef3
Author: Michael Stahl mst...@redhat.com
Date:   Thu Dec 13 01:36:33 2012 +0100

SwFmtAnchor::SetAnchor(): assert that position is on SwTxtNode

Change-Id: I1c2a1cc2de4984cdcf1e5b8d0eafbd1471538e4a
(cherry picked from commit 90a0116ccb48388d91b21128fcac2e4609838581)

diff --git a/sw/source/core/layout/atrfrm.cxx b/sw/source/core/layout/atrfrm.cxx
index a59d2d2..f5d7798 100644
--- a/sw/source/core/layout/atrfrm.cxx
+++ b/sw/source/core/layout/atrfrm.cxx
@@ -98,6 +98,9 @@
 #include svl/cjkoptions.hxx
 #include switerator.hxx
 #include pagedeschint.hxx
+#ifndef NDEBUG
+#include ndtxt.hxx
+#endif
 
 using namespace ::com::sun::star;
 using ::rtl::OUString;
@@ -1511,6 +1514,8 @@ SwFmtAnchor::~SwFmtAnchor()
 
 void SwFmtAnchor::SetAnchor( const SwPosition *pPos )
 {
+// anchor only to paragraphs
+assert(!pPos || dynamic_castSwTxtNode*(pPos-nNode.GetNode()));
 m_pCntntAnchor .reset( (pPos) ? new SwPosition( *pPos ) : 0 );
 // Flys anchored AT paragraph should not point into the paragraph content
 if (m_pCntntAnchor 
commit 2ab2046fe7d939be02649b1c69f571cb19f9aeaa
Author: Michael Stahl mst...@redhat.com
Date:   Thu Dec 13 01:28:59 2012 +0100

fdo#58142: fix frame selection in SwPageFrm::GetCrsrOfst:

The function does not handle the case where a paragraph was hit but no
background frame was hit properly, which may or may not (depending on
the platform) cause it to return a position of a frame when that is
undesirable, such as when called from SwWrtShell::UnSelectFrm().

When UnselectFrm() leaves a frame selected pasting multiple pictures
leads to a crash.

(regression from e8fbe97900f13305b17015d9044993bde4adab36)

Change-Id: I3604825f654c523a37f47a9fc660647d8b3e7077
(cherry picked from commit 2fc4af311656e7d019ca802d23444e5161f820ee)

diff --git a/sw/source/core/layout/trvlfrm.cxx 
b/sw/source/core/layout/trvlfrm.cxx
index a70b08d..443af12 100644
--- a/sw/source/core/layout/trvlfrm.cxx
+++ b/sw/source/core/layout/trvlfrm.cxx
@@ -264,8 +264,12 @@ sal_Bool SwPageFrm::GetCrsrOfst( SwPosition *pPos, Point 
rPoint,
 if ( ( bTestBackground  bBackRet ) || !bTextRet )
 {
 bRet = bBackRet;
-pPos-nNode = aBackPos.nNode;
-pPos-nContent = aBackPos.nContent;
+(*pPos) = aBackPos;
+}
+else if (bTextRet  !bBackRet)
+{
+bRet = bTextRet;
+(*pPos) = aTextPos;
 }
 else
 {
@@ -306,14 +310,12 @@ sal_Bool SwPageFrm::GetCrsrOfst( SwPosition *pPos, Point 
rPoint,
 if ( bValidTextDistance  bValidBackDistance  
basegfx::fTools::more( nTextDistance, nBackDistance ) )
 {
 bRet = bBackRet;
-pPos-nNode = aBackPos.nNode;
-pPos-nContent = aBackPos.nContent;
+(*pPos) = aBackPos;
 }
 else
 {
 bRet = bTextRet;
-pPos-nNode = aTextPos.nNode;
-pPos-nContent = aTextPos.nContent;
+(*pPos) = aTextPos;
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 2 commits - tools/inc

2012-12-13 Thread Libreoffice Gerrit user
 tools/inc/tools/gen.hxx |   80 
 1 file changed, 40 insertions(+), 40 deletions(-)

New commits:
commit 805ec83f188af51906a8d3a25ba9f5bc2e7f61c8
Author: Noel Grandin n...@peralex.com
Date:   Thu Dec 13 13:39:18 2012 +0200

whitespace cleanup

Since I'm hacking in this file anyhow, make the class definition
a little prettier.

Change-Id: Id47f1084fe103e85cf1dacc84a9e536774767e61

diff --git a/tools/inc/tools/gen.hxx b/tools/inc/tools/gen.hxx
index 568d873..1819a90 100644
--- a/tools/inc/tools/gen.hxx
+++ b/tools/inc/tools/gen.hxx
@@ -41,8 +41,8 @@ public:
 long   A() { return nA; }
 long   B() { return nB; }
 
-sal_Booloperator == ( const Pair rPair ) const;
-sal_Booloperator != ( const Pair rPair ) const;
+sal_Booloperator == ( const Pair rPair ) const;
+sal_Booloperator != ( const Pair rPair ) const;
 
 TOOLS_DLLPUBLIC friend SvStreamoperator( SvStream rIStream, Pair 
rPair );
 TOOLS_DLLPUBLIC friend SvStreamoperator( SvStream rOStream, const 
Pair rPair );
@@ -88,10 +88,10 @@ public:
 long   Y() { return nB; }
 
 voidMove( long nHorzMove, long nVertMove );
-sal_BoolIsAbove( const Point rPoint ) const;
-sal_BoolIsBelow( const Point rPoint ) const;
-sal_BoolIsLeft( const Point rPoint ) const;
-sal_BoolIsRight( const Point rPoint ) const;
+sal_BoolIsAbove( const Point rPoint ) const;
+sal_BoolIsBelow( const Point rPoint ) const;
+sal_BoolIsLeft( const Point rPoint ) const;
+sal_BoolIsRight( const Point rPoint ) const;
 
 Point  operator += ( const Point rPoint );
 Point  operator -= ( const Point rPoint );
@@ -212,10 +212,10 @@ public:
 long   Width()  { return nA; }
 long   Height() { return nB; }
 
-longgetWidth() const { return Width(); }
-longgetHeight() const { return Height(); }
-voidsetWidth(long nWidth)  { Width() = nWidth; }
-voidsetHeight(long nHeight)  { Height() = nHeight; }
+longgetWidth() const { return Width(); }
+longgetHeight() const { return Height(); }
+voidsetWidth(long nWidth)  { Width() = nWidth; }
+voidsetHeight(long nHeight)  { Height() = nHeight; }
 };
 
 inline Size::Size()
@@ -251,7 +251,7 @@ public:
 long   Min() { return nA; }
 long   Max() { return nB; }
 
-sal_BoolIsInside( long nIs ) const;
+sal_BoolIsInside( long nIs ) const;
 
 voidJustify();
 };
@@ -305,16 +305,16 @@ public:
 long   Min() { return nA; }
 long   Max() { return nB; }
 
-sal_BoolIsInside( long nIs ) const;
+sal_BoolIsInside( long nIs ) const;
 
 voidJustify();
 
-sal_Booloperator !() const { return !Len(); }
+sal_Booloperator !() const { return !Len(); }
 
-longgetMin() const { return Min(); }
-longgetMax() const { return Max(); }
-voidsetMin(long nMin)  { Min() = nMin; }
-voidsetMax(long nMax)  { Max() = nMax; }
+longgetMin() const { return Min(); }
+longgetMax() const { return Max(); }
+voidsetMin(long nMin)  { Min() = nMin; }
+voidsetMax(long nMax)  { Max() = nMax; }
 };
 
 inline Selection::Selection()
commit c317b7f963f6d3fb2f325bc212c148d61b3dfa31
Author: Noel Grandin n...@peralex.com
Date:   Thu Dec 13 13:36:57 2012 +0200

make the inline method attributes match with the method definition.

And make the whitespace layout a little cleaner.

Change-Id: I12b042600f061e74825de92ce836ee34ebdfabdb

diff --git a/tools/inc/tools/gen.hxx b/tools/inc/tools/gen.hxx
index 2ddb21f..568d873 100644
--- a/tools/inc/tools/gen.hxx
+++ b/tools/inc/tools/gen.hxx
@@ -374,44 +374,44 @@ public:
 long   Top()   { return nTop;}
 long   Bottom(){ return nBottom; }
 
-inline PointTopLeft() const;
-Point   TopRight() const;
-Point   TopCenter() const;
-Point   BottomLeft() const;
-Point   BottomRight() const;
-Point   BottomCenter() const;
-Point   LeftCenter() const;
-Point   RightCenter() const;
-Point   Center() const;
-
-voidMove( long nHorzMove, long nVertMove );
+inline PointTopLeft() const;
+inline PointTopRight() const;
+inline PointTopCenter() const;
+inline Point

[PATCH] Change in core[libreoffice-3-6]: fdo#58142: fix frame selection in SwPageFrm::GetCrsrOfst:

2012-12-13 Thread Michael Stahl (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1314

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/14/1314/1

fdo#58142: fix frame selection in SwPageFrm::GetCrsrOfst:

The function does not handle the case where a paragraph was hit but no
background frame was hit properly, which may or may not (depending on
the platform) cause it to return a position of a frame when that is
undesirable, such as when called from SwWrtShell::UnSelectFrm().

When UnselectFrm() leaves a frame selected pasting multiple pictures
leads to a crash.

(regression from e8fbe97900f13305b17015d9044993bde4adab36)

Change-Id: I3604825f654c523a37f47a9fc660647d8b3e7077
(cherry picked from commit 2fc4af311656e7d019ca802d23444e5161f820ee)
---
M sw/source/core/layout/trvlfrm.cxx
1 file changed, 8 insertions(+), 6 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3604825f654c523a37f47a9fc660647d8b3e7077
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Michael Stahl mst...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Addin exceptions in SvStream

2012-12-13 Thread Michael Meeks
Hi Marc,

On Thu, 2012-12-13 at 06:10 -0500, Marc-André Laverdière wrote:
 That's the problem... what we don't know can be hurting us big time.
 And since the tests are not very thorough, I can't have a good
 confidence that things are not broken.

Of course.

 So, either we finish this patch and hope people will pick up those
 cases along the way - a risky proposal but it has the advantage of
 immunizing pretty much every filter for 'cheap'.

and/or breaking it ? :-)

 Otherwise, I suggest that we go for a SvStream wrapper that throws
 exceptions - no changes to SvStream. It can be used wherever wanted,
 so that's a lot of engineering effort to bolt that in place - but less
 work than just putting stream-good() checks everywhere.

AFAICS throwing an exception without going through and auditing each
call-site will create even more pathalogical problems than just
returning 0 - people who were careless (like they are) and routinely
read over the end of some stream and then later checked will now throw
an exception and fail to read the whole document - right ? :-)

Personally, I don't see a clean solution beyond reading each of the
call sites to those operators. If we want to take the risk of busting
things - personally I'd prefer zeroing the data: it has a much lower
risk/down-side as far as I can see. Then again - the ideal is clearly to
do the work of code cleanup  audit - that's fairly mindless busy-work
that just needs to be done carefully.

ATB,

Michael.

-- 
michael.me...@suse.com  , Pseudo Engineer, itinerant idiot

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 2 commits - src/lib

2012-12-13 Thread Libreoffice Gerrit user
 src/lib/CDRDocument.cpp |   12 +++-
 src/lib/CMXDocument.cpp |5 +
 2 files changed, 16 insertions(+), 1 deletion(-)

New commits:
commit 625677ba4ec9f0b0bb930fcefab5b7fc91b9bae8
Author: David Tardon dtar...@redhat.com
Date:   Thu Dec 13 12:49:19 2012 +0100

coverity: do not let CDRDocument::parse throw

diff --git a/src/lib/CDRDocument.cpp b/src/lib/CDRDocument.cpp
index 4a75389..2633954 100644
--- a/src/lib/CDRDocument.cpp
+++ b/src/lib/CDRDocument.cpp
@@ -129,7 +129,17 @@ bool libcdr::CDRDocument::parse(::WPXInputStream *input, 
libwpg::WPGPaintInterfa
 {
   input-seek(0, WPX_SEEK_SET);
   bool retVal = false;
-  unsigned version = getCDRVersion(input);
+  unsigned version = 0;
+  try
+  {
+version = getCDRVersion(input);
+  }
+  catch (libcdr::EndOfStreamException const)
+  {
+// This can only happen if isSupported() has not been called before
+return false;
+  }
+
   if (version)
   {
 input-seek(0, WPX_SEEK_SET);
commit c21b18b668b9efc0dc14257e1e43a85c1ccea807
Author: David Tardon dtar...@redhat.com
Date:   Thu Dec 13 12:13:14 2012 +0100

coverity: do not let CMXDocument::isSupported throw

diff --git a/src/lib/CMXDocument.cpp b/src/lib/CMXDocument.cpp
index 35d361d..9845552 100644
--- a/src/lib/CMXDocument.cpp
+++ b/src/lib/CMXDocument.cpp
@@ -45,6 +45,7 @@ Analyzes the content of an input stream to see if it can be 
parsed
 stream is a Corel Draw Document that libcdr is able to parse
 */
 bool libcdr::CMXDocument::isSupported(WPXInputStream *input)
+try
 {
   input-seek(0, WPX_SEEK_SET);
   unsigned riff = readU32(input);
@@ -62,6 +63,10 @@ bool libcdr::CMXDocument::isSupported(WPXInputStream *input)
 return false;
   return true;
 }
+catch (...)
+{
+  return false;
+}
 
 /**
 Parses the input stream content. It will make callbacks to the functions 
provided by a
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Remove save to old Microsoft office formats

2012-12-13 Thread Caolán McNamara
On Thu, 2012-12-13 at 10:05 +, Michael Meeks wrote:
   I assume the root problem you want to fix is of users accidentally
 picking the wrong format in the filter / drop-down and getting a
 needlessly bad experience here ?

I seem to recall a bug (maybe fixed ?) where in the kde file dialog
typing foo.doc as the save name would auto pick the word 6 format
instead of the word 8 format.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Addin exceptions in SvStream

2012-12-13 Thread Caolán McNamara
On Thu, 2012-12-13 at 09:40 +, Michael Meeks wrote:
   Well - it is -thought- (but hard to prove) that this behaviour is
 relied on by the code in a number of places:

I know of one (it could be the only one) place in
WW8ScannerBase::OpenPieceTable in sw/source/filter/ww8/ww8scan.cxx 

while( 1 ) // Zaehle Zahl der Grpprls
{
sal_uInt8 clxt(2);
*pStr  clxt;
if( 2 == clxt ) // PLCFfpcd ?
break;  // PLCFfpcd gefunden
...
}

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: embeddedobj/Library_embobj.mk

2012-12-13 Thread Libreoffice Gerrit user
 embeddedobj/Library_embobj.mk |1 +
 1 file changed, 1 insertion(+)

New commits:
commit ec89369f9f1c522c2c15fe1eaf6808f54233a4e3
Author: Luboš Luňák l.lu...@suse.cz
Date:   Thu Dec 13 13:30:07 2012 +0100

link embedobj to tools library

Usage introduced in dde234b6955a421d51d2b37e4fc3972c660146f0 and MSVC
wants an external symbol even if the function is inline, for some reason.

diff --git a/embeddedobj/Library_embobj.mk b/embeddedobj/Library_embobj.mk
index 1fa1c5a..f3e1d51 100644
--- a/embeddedobj/Library_embobj.mk
+++ b/embeddedobj/Library_embobj.mk
@@ -46,6 +46,7 @@ $(eval $(call gb_Library_use_libraries,embobj,\
cppuhelper \
sal \
vcl \
+   tl \
$(gb_UWINAPI) \
 ))
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: what's with Rectangle::GetHeight() and GetWidth() in gen.hxx

2012-12-13 Thread Lubos Lunak
On Thursday 13 of December 2012, Noel Grandin wrote:
 Hi

 I seem to have triggered a problem on the windows tinderbox, but I'm
 struggling to understand why.
 The error on the tinderbox looks like:

 C:/cygwin/home/tinderbox/master-build/workdir/wntmsci13.pro/LinkTarget/Libr
ary/iembobj.exp docholder.o : error LNK2019: unresolved external symbol
 __declspec(dllimport) public: long __thiscall
 Rectangle::GetHeight(void)const  (__imp_?GetHeight@Rectangle@@QBEJXZ)
 referenced in function
 __catch$?GetDocFrame@DocumentHolder@@AAE?AV?$Reference@VXFrame@frame@star@s
un@com@@@uno@star@sun@com@@XZ$0

 Firstly, there seems to be a missing inline statement in the class
 header section.

 That's not needed actually.

 Secondly, given that there __is__ an inline modifier, why would the
 windows compiler be trying to link this function externally?

 The compiler is not required to inline, the keyword is just a hint.

 I'm not a Windows expert, but it looks like it is required on Windows to pass 
every used library to the link command, and this problem exists because 
embedobj does not link the tools library, and you've just introduced usage of 
it. I've pushed a fix for this.

-- 
 Lubos Lunak
 l.lu...@suse.cz
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 4 commits - oox/inc sal/qa solenv/gbuild sw/inc sw/Library_msword.mk sw/source writerfilter/inc xmloff/inc

2012-12-13 Thread Libreoffice Gerrit user
 oox/inc/oox/export/chartexport.hxx|1 
 sal/qa/osl/process/osl_Thread.cxx |3 
 solenv/gbuild/Helper.mk   |   10 
 sw/Library_msword.mk  |2 
 sw/inc/pch/precompiled_msword.cxx |   12 +
 sw/inc/pch/precompiled_msword.hxx |  257 ++
 sw/inc/pch/precompiled_sw.hxx |   27 --
 sw/source/filter/ww8/wrtw8esh.cxx |6 
 writerfilter/inc/pch/precompiled_writerfilter.hxx |1 
 xmloff/inc/pch/precompiled_xmloff.hxx |1 
 10 files changed, 285 insertions(+), 35 deletions(-)

New commits:
commit 481bc6cf3d8ab8b7f37c709bbe4e7f83b60ef8ec
Author: Luboš Luňák l.lu...@suse.cz
Date:   Wed Dec 12 23:14:01 2012 +0100

substitute BUILDDIR in gb_Helper_abbreviate_dirs too

diff --git a/solenv/gbuild/Helper.mk b/solenv/gbuild/Helper.mk
index f81a6b4..25cf47f 100644
--- a/solenv/gbuild/Helper.mk
+++ b/solenv/gbuild/Helper.mk
@@ -38,12 +38,22 @@ gb_Helper_MISCDUMMY := $(gb_Helper_MISC)/DUMMY
 $(gb_Helper_MISCDUMMY) :
@mkdir -p $(dir $@)  touch $@
 
+ifeq ($(SRCDIR),$(BUILDDIR))
 define gb_Helper_abbreviate_dirs
 S=$(SRCDIR)  \
 $(subst $(SRCDIR)/,$$S/,O=$(OUTDIR))  \
 $(subst $(SRCDIR)/,$$S/,W=$(WORKDIR))  \
 $(subst $(SRCDIR)/,$$S/,$(subst $(OUTDIR)/,$$O/,$(subst 
$(WORKDIR)/,$$W/,$(1
 endef
+else
+define gb_Helper_abbreviate_dirs
+S=$(SRCDIR)  \
+$(subst $(SRCDIR)/,$$S/,B=$(BUILDDIR))  \
+$(subst $(SRCDIR)/,$$S/,$(subst $(BUILDDIR)/,$$B/,O=$(OUTDIR)))  \
+$(subst $(SRCDIR)/,$$S/,$(subst $(BUILDDIR)/,$$B/,W=$(WORKDIR)))  \
+$(subst $(SRCDIR)/,$$S/,$(subst $(BUILDDIR)/,$$B/,$(subst 
$(OUTDIR)/,$$O/,$(subst $(WORKDIR)/,$$W/,$(1)
+endef
+endif
 
 define gb_Helper_abbreviate_dirs_native
 $(call gb_Output_error,gb_Helper_abbreviate_dirs_native: use 
gb_Helper_abbreviate_dirs instead.)
commit 40a0f91d63530959e04f7851e3390561780f9ad5
Author: Luboš Luňák l.lu...@suse.cz
Date:   Wed Dec 12 23:12:33 2012 +0100

NOMINMAX on Windows is defined globally

diff --git a/sal/qa/osl/process/osl_Thread.cxx 
b/sal/qa/osl/process/osl_Thread.cxx
index 11214bd..3e394aa 100644
--- a/sal/qa/osl/process/osl_Thread.cxx
+++ b/sal/qa/osl/process/osl_Thread.cxx
@@ -18,9 +18,6 @@
  */
 
 #ifdef WNT
-#if defined _MSC_VER
-#define NOMINMAX
-#endif
 #include windows.h
 #else
 #include unistd.h
commit 4a4d3e08f6abb2bb20f8937e4d704d372bd0bd25
Author: Luboš Luňák l.lu...@suse.cz
Date:   Wed Dec 12 18:12:42 2012 +0100

update PCH headers

diff --git a/sw/inc/pch/precompiled_sw.hxx b/sw/inc/pch/precompiled_sw.hxx
index 777b491..991c412 100644
--- a/sw/inc/pch/precompiled_sw.hxx
+++ b/sw/inc/pch/precompiled_sw.hxx
@@ -34,7 +34,6 @@
 #include boost/bind.hpp
 #include boost/foreach.hpp
 #include boost/ptr_container/ptr_set.hpp
-#include boost/ptr_container/ptr_vector.hpp
 #include boost/scoped_array.hpp
 #include boost/scoped_ptr.hpp
 #include boost/shared_ptr.hpp
@@ -58,7 +57,6 @@
 #include comphelper/MasterPropertySetInfo.hxx
 #include comphelper/mediadescriptor.hxx
 #include comphelper/processfactory.hxx
-#include comphelper/property.hxx
 #include comphelper/scoped_disposing_ptr.hxx
 #include comphelper/sequenceashashmap.hxx
 #include comphelper/sequenceasvector.hxx
@@ -113,7 +111,6 @@
 #include com/sun/star/beans/XPropertyState.hpp
 #include com/sun/star/beans/XPropertyStateChangeListener.hpp
 #include com/sun/star/container/XContainer.hpp
-#include com/sun/star/container/XContainerListener.hpp
 #include com/sun/star/container/XChild.hpp
 #include com/sun/star/container/XIndexAccess.hpp
 #include com/sun/star/container/XIndexContainer.hpp
@@ -244,10 +241,6 @@
 #include com/sun/star/linguistic2/XProofreadingIterator.hpp
 #include com/sun/star/linguistic2/XSpellChecker1.hpp
 #include com/sun/star/linguistic2/XThesaurus.hpp
-#include com/sun/star/mail/MailAttachment.hpp
-#include com/sun/star/mail/MailServiceProvider.hpp
-#include com/sun/star/mail/MailServiceType.hpp
-#include com/sun/star/mail/XMailService.hpp
 #include com/sun/star/mail/XSmtpService.hpp
 #include com/sun/star/packages/WrongPasswordException.hpp
 #include com/sun/star/packages/zip/ZipIOException.hpp
@@ -272,7 +265,6 @@
 #include com/sun/star/script/XLibraryContainer.hpp
 #include com/sun/star/sdbc/DataType.hpp
 #include com/sun/star/sdb/CommandType.hpp
-#include com/sun/star/sdbc/ResultSetType.hpp
 #include com/sun/star/sdbc/XConnection.hpp
 #include com/sun/star/sdbc/XDataSource.hpp
 #include com/sun/star/sdbc/XResultSet.hpp
@@ -282,12 +274,7 @@
 #include com/sun/star/sdbcx/XRowLocate.hpp
 #include com/sun/star/sdbcx/XTablesSupplier.hpp
 #include com/sun/star/sdb/DatabaseContext.hpp
-#include com/sun/star/sdb/SQLContext.hpp
-#include com/sun/star/sdb/XColumn.hpp
-#include com/sun/star/sdb/XCompletedConnection.hpp
-#include com/sun/star/sdb/XCompletedExecution.hpp
 #include com/sun/star/sdb/XDatabaseAccess.hpp
-#include com/sun/star/sdb/XDocumentDataSource.hpp
 

[Libreoffice-commits] .: solenv/gbuild

2012-12-13 Thread Libreoffice Gerrit user
 solenv/gbuild/platform/com_MSC_defs.mk |7 +++
 1 file changed, 3 insertions(+), 4 deletions(-)

New commits:
commit b988f64d37f1e46fb5a9449758b379fc01661fa2
Author: Luboš Luňák l.lu...@suse.cz
Date:   Wed Dec 12 17:48:52 2012 +0100

do not disable msvc warning 4005

And in fact make it an error with PCH, as there's it's very likely
a PCH-specific problem.
History refers to i76713 when introducing this, and that was something
about .NET 2005, so this quite likely no longer applies.

diff --git a/solenv/gbuild/platform/com_MSC_defs.mk 
b/solenv/gbuild/platform/com_MSC_defs.mk
index e8d2f29..bea9b91 100644
--- a/solenv/gbuild/platform/com_MSC_defs.mk
+++ b/solenv/gbuild/platform/com_MSC_defs.mk
@@ -74,8 +74,6 @@ gb_AFLAGS := /c /Cp
 # cleaning away from the code, to avoid warnings when building with
 # gcc or Clang and -Wall -Werror.
 
-# C4005: 'identifier' : macro redefinition
-
 # C4127: conditional expression is constant
 
 # C4180: qualifier applied to function type has no meaning; ignored
@@ -149,7 +147,6 @@ gb_CFLAGS := \
-MD \
-nologo \
-W4 \
-   -wd4005 \
-wd4127 \
-wd4180 \
-wd4189 \
@@ -183,7 +180,6 @@ gb_CXXFLAGS := \
-MD \
-nologo \
-W4 \
-   -wd4005 \
-wd4127 \
-wd4180 \
-wd4189 \
@@ -220,11 +216,14 @@ ifeq ($(VCVER),100)
 
 endif
 
+# C4005: 'identifier' : macro redefinition
+
 gb_PCHWARNINGS = \
-we4650 \
-we4651 \
-we4652 \
-we4653 \
+   -we4005 \
 
 gb_STDLIBS := \
advapi32.lib \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: what's with Rectangle::GetHeight() and GetWidth() in gen.hxx

2012-12-13 Thread Noel Grandin


On 2012-12-13 14:31, Lubos Lunak wrote:
it. I've pushed a fix for this. 

Thanks.

Disclaimer: http://www.peralex.com/disclaimer.html


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: i18npool/inc i18npool/source

2012-12-13 Thread Libreoffice Gerrit user
 i18npool/inc/i18npool/lang.h|2 ++
 i18npool/source/isolang/isolang.cxx |2 ++
 2 files changed, 4 insertions(+)

New commits:
commit 09bc1464ec9dde61e69cca393e65e72143bdd383
Author: Eike Rathke er...@redhat.com
Date:   Thu Dec 13 13:45:14 2012 +0100

introduced [mul] multiple and [und] undetermined language codes

Various places use an empty locale or string to indicate different
meanings and/or variants of this is not a real locale but I didn't know
what to use else or abuse LANGUAGE_DONTKNOW in one or the other way. In
preparation of changing that awkward situation now offer the ISO 639
codes for multiple languages [mul] and language undetermined [und],
mapping to LANGUAGE_MULTIPLE and LANGUAGE_UNDETERMINED.

Change-Id: I687de23ffc00a9a056f2837b024e0a62658e3df2

diff --git a/i18npool/inc/i18npool/lang.h b/i18npool/inc/i18npool/lang.h
index cf4ec6b..932f92b 100644
--- a/i18npool/inc/i18npool/lang.h
+++ b/i18npool/inc/i18npool/lang.h
@@ -566,6 +566,8 @@ typedef unsigned short LanguageType;
 #define LANGUAGE_USER_YOMBE 0x0684
 #define LANGUAGE_USER_YOMBE_CONGO   0x8284  /* makeLangID( 0x20, 
getPrimaryLanguage( LANGUAGE_USER_YOMBE)) */
 #define LANGUAGE_USER_SIDAMA0x0685
+#define LANGUAGE_MULTIPLE   0xFFEF  /* multiple languages, 
primary 0x3ef, sub 0x3f */
+#define LANGUAGE_UNDETERMINED   0xFFF0  /* undetermined language, 
primary 0x3f0, sub 0x3f */
 #define LANGUAGE_USER_SYSTEM_CONFIG 0xFFFE  /* not a locale, to be 
used only in configuration context to obtain system default, primary 0x3fe, sub 
0x3f */
 
 #endif /* INCLUDED_I18NPOOL_LANG_H */
diff --git a/i18npool/source/isolang/isolang.cxx 
b/i18npool/source/isolang/isolang.cxx
index 439590a..bbe3e0f 100644
--- a/i18npool/source/isolang/isolang.cxx
+++ b/i18npool/source/isolang/isolang.cxx
@@ -511,6 +511,8 @@ static MsLangId::IsoLangEntry const aImplIsoLangEntries[] =
 { LANGUAGE_USER_YOMBE, yom, CD },
 { LANGUAGE_USER_YOMBE_CONGO,   yom, CG },
 { LANGUAGE_USER_SIDAMA,sid, ET },
+{ LANGUAGE_MULTIPLE,   mul,},   // multiple 
languages, many languages are used
+{ LANGUAGE_UNDETERMINED,   und,},   // undetermined 
language, language cannot be identified
 { LANGUAGE_NONE,   zxx,},   // added to ISO 
639-2 on 2006-01-11: Used to declare the absence of linguistic information
 { LANGUAGE_DONTKNOW,,  }// marks end of 
table
 };
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2012-12-13 Thread Libreoffice Gerrit user
 sw/inc/unochart.hxx|2 -
 sw/source/core/undo/rolbck.cxx |   14 +++
 sw/source/core/undo/undel.cxx  |   10 ++---
 sw/source/core/undo/undobj.cxx |2 -
 sw/source/core/undo/unmove.cxx |2 -
 sw/source/core/undo/unovwr.cxx |2 -
 sw/source/core/undo/untbl.cxx  |   36 +-
 sw/source/core/undo/untblk.cxx |2 -
 sw/source/core/unocore/swunohelper.cxx |8 ++--
 sw/source/core/unocore/unochart.cxx|   63 -
 10 files changed, 71 insertions(+), 70 deletions(-)

New commits:
commit 20f2aaced1dabbd62ea516368b85e0e881d42165
Author: Takeshi Abe t...@fixedpoint.jp
Date:   Thu Dec 13 21:52:13 2012 +0900

sal_Bool to bool

Change-Id: I56e3889774b13fc040f72d576b8ea530bf31f9a5

diff --git a/sw/inc/unochart.hxx b/sw/inc/unochart.hxx
index fce97d2..e40ac8e 100644
--- a/sw/inc/unochart.hxx
+++ b/sw/inc/unochart.hxx
@@ -66,7 +66,7 @@ class SwFrmFmt;
 
 //
 
-sal_Bool FillRangeDescriptor( SwRangeDescriptor rDesc, const String 
rCellRangeName );
+bool FillRangeDescriptor( SwRangeDescriptor rDesc, const String 
rCellRangeName );
 
 //
 
diff --git a/sw/source/core/undo/rolbck.cxx b/sw/source/core/undo/rolbck.cxx
index a991a5e..10664f4 100644
--- a/sw/source/core/undo/rolbck.cxx
+++ b/sw/source/core/undo/rolbck.cxx
@@ -738,7 +738,7 @@ SwHistorySetAttrSet::SwHistorySetAttrSet( const SfxItemSet 
rSet,
 break;
 pItem = aIter.NextItem();
 pOrigItem = aOrigIter.NextItem();
-} while( sal_True );
+} while( true );
 }
 
 void SwHistorySetAttrSet::SetInDoc( SwDoc* pDoc, bool )
@@ -775,7 +775,7 @@ SwHistoryResetAttrSet::SwHistoryResetAttrSet( const 
SfxItemSet rSet,
 SfxItemIter aIter( rSet );
 bool bAutoStyle = false;
 
-while( sal_True )
+while( true )
 {
 const sal_uInt16 nWhich = aIter.GetCurItem()-Which();
 
@@ -1195,7 +1195,7 @@ void SwHistory::CopyFmtAttr( const SfxItemSet rSet, 
sal_uLong nNodeIdx )
 if( aIter.IsAtEnd() )
 break;
 aIter.NextItem();
-} while( sal_True );
+} while( true );
 }
 }
 
@@ -1218,17 +1218,17 @@ void SwHistory::CopyAttr( SwpHints* pHts, sal_uLong 
nNodeIdx,
 break;
 
 // never copy Flys and Ftn !!
-sal_Bool bNextAttr = sal_False;
+bool bNextAttr = false;
 switch( pHt-Which() )
 {
 case RES_TXTATR_FIELD:
 // no fields, ... copy ??
 if( !bFields )
-bNextAttr = sal_True;
+bNextAttr = true;
 break;
 case RES_TXTATR_FLYCNT:
 case RES_TXTATR_FTN:
-bNextAttr = sal_True;
+bNextAttr = true;
 break;
 }
 
@@ -1400,7 +1400,7 @@ void SwRegHistory::_MakeSetWhichIds()
 {
 SfxItemIter aIter( *pSet );
 sal_uInt16 nW = aIter.FirstItem()-Which();
-while( sal_True )
+while( true )
 {
 m_WhichIdSet.insert( nW );
 if( aIter.IsAtEnd() )
diff --git a/sw/source/core/undo/undel.cxx b/sw/source/core/undo/undel.cxx
index dc92233..9a53e8c 100644
--- a/sw/source/core/undo/undel.cxx
+++ b/sw/source/core/undo/undel.cxx
@@ -154,8 +154,8 @@ SwUndoDelete::SwUndoDelete( SwPaM rPam, sal_Bool 
bFullPara, sal_Bool bCalledByT
 : pEnd-nNode.GetNode().GetTxtNode();
 }
 
-sal_Bool bMoveNds = *pStt == *pEnd  // any area still existent?
-? sal_False
+bool bMoveNds = *pStt == *pEnd  // any area still existent?
+? false
 : ( SaveCntnt( pStt, pEnd, pSttTxtNd, pEndTxtNd ) || 
bFromTableCopy );
 
 if( pSttTxtNd  pEndTxtNd  pSttTxtNd != pEndTxtNd )
@@ -342,7 +342,7 @@ sal_Bool SwUndoDelete::SaveCntnt( const SwPosition* pStt, 
const SwPosition* pEnd
 // 1 - copy start in Start-String
 if( pSttTxtNd )
 {
-sal_Bool bOneNode = nSttNode == nEndNode;
+bool bOneNode = nSttNode == nEndNode;
 xub_StrLen nLen = bOneNode ? nEndCntnt - nSttCntnt
 : pSttTxtNd-GetTxt().Len() - nSttCntnt;
 SwRegHistory aRHst( *pSttTxtNd, pHistory );
@@ -467,7 +467,7 @@ sal_Bool SwUndoDelete::CanGrouping( SwDoc* pDoc, const 
SwPaM rDelPam )
 if( !FillSaveData( rDelPam, *pTmpSav, sal_False ))
 delete pTmpSav, pTmpSav = 0;
 
-sal_Bool bOk = ( !pRedlSaveData  !pTmpSav ) ||
+bool bOk = ( !pRedlSaveData  !pTmpSav ) ||
( pRedlSaveData  pTmpSav 
 SwUndo::CanRedlineGroup( *pRedlSaveData, *pTmpSav, bBackSp ));
 delete pTmpSav;
@@ -762,7 +762,7 @@ void SwUndoDelete::UndoImpl(::sw::UndoRedoContext  
rContext)
 else
 pInsNd = 0; // do 

[Libreoffice-commits] .: Branch 'libreoffice-4-0' - i18npool/inc i18npool/source

2012-12-13 Thread Libreoffice Gerrit user
 i18npool/inc/i18npool/lang.h|2 ++
 i18npool/source/isolang/isolang.cxx |2 ++
 2 files changed, 4 insertions(+)

New commits:
commit ebcc02052d6fdfad59cdbcb4f18df0c007033999
Author: Eike Rathke er...@redhat.com
Date:   Thu Dec 13 13:45:14 2012 +0100

introduced [mul] multiple and [und] undetermined language codes

Various places use an empty locale or string to indicate different
meanings and/or variants of this is not a real locale but I didn't know
what to use else or abuse LANGUAGE_DONTKNOW in one or the other way. In
preparation of changing that awkward situation now offer the ISO 639
codes for multiple languages [mul] and language undetermined [und],
mapping to LANGUAGE_MULTIPLE and LANGUAGE_UNDETERMINED.

Change-Id: I687de23ffc00a9a056f2837b024e0a62658e3df2
(cherry picked from commit 09bc1464ec9dde61e69cca393e65e72143bdd383)

diff --git a/i18npool/inc/i18npool/lang.h b/i18npool/inc/i18npool/lang.h
index cf4ec6b..932f92b 100644
--- a/i18npool/inc/i18npool/lang.h
+++ b/i18npool/inc/i18npool/lang.h
@@ -566,6 +566,8 @@ typedef unsigned short LanguageType;
 #define LANGUAGE_USER_YOMBE 0x0684
 #define LANGUAGE_USER_YOMBE_CONGO   0x8284  /* makeLangID( 0x20, 
getPrimaryLanguage( LANGUAGE_USER_YOMBE)) */
 #define LANGUAGE_USER_SIDAMA0x0685
+#define LANGUAGE_MULTIPLE   0xFFEF  /* multiple languages, 
primary 0x3ef, sub 0x3f */
+#define LANGUAGE_UNDETERMINED   0xFFF0  /* undetermined language, 
primary 0x3f0, sub 0x3f */
 #define LANGUAGE_USER_SYSTEM_CONFIG 0xFFFE  /* not a locale, to be 
used only in configuration context to obtain system default, primary 0x3fe, sub 
0x3f */
 
 #endif /* INCLUDED_I18NPOOL_LANG_H */
diff --git a/i18npool/source/isolang/isolang.cxx 
b/i18npool/source/isolang/isolang.cxx
index 439590a..bbe3e0f 100644
--- a/i18npool/source/isolang/isolang.cxx
+++ b/i18npool/source/isolang/isolang.cxx
@@ -511,6 +511,8 @@ static MsLangId::IsoLangEntry const aImplIsoLangEntries[] =
 { LANGUAGE_USER_YOMBE, yom, CD },
 { LANGUAGE_USER_YOMBE_CONGO,   yom, CG },
 { LANGUAGE_USER_SIDAMA,sid, ET },
+{ LANGUAGE_MULTIPLE,   mul,},   // multiple 
languages, many languages are used
+{ LANGUAGE_UNDETERMINED,   und,},   // undetermined 
language, language cannot be identified
 { LANGUAGE_NONE,   zxx,},   // added to ISO 
639-2 on 2006-01-11: Used to declare the absence of linguistic information
 { LANGUAGE_DONTKNOW,,  }// marks end of 
table
 };
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Addin exceptions in SvStream

2012-12-13 Thread Lubos Lunak
On Thursday 13 of December 2012, Michael Meeks wrote:
   Personally, I don't see a clean solution beyond reading each of the
 call sites to those operators. If we want to take the risk of busting
 things - personally I'd prefer zeroing the data: it has a much lower
 risk/down-side as far as I can see. Then again - the ideal is clearly to
 do the work of code cleanup  audit - that's fairly mindless busy-work
 that just needs to be done carefully.

 I think it shouldn't be that difficult to write a Clang plugin that finds all 
uses of SvStream::operator/ , checks whether the variable read to is 
initalized [*] , and possibly it shouldn't be even that big deal to have it 
rewrite them to a series of SvStream::readInt32() etc. calls (i.e. EasyHack 
fdo#56110). I have not written yet any howto on plugins besides the examples 
under compilerplugins/, but if there would be interest, I could do something.

[*] Or, since I assume all those variables have been needlessly initialized 
just in attempt to silence the broken warning with the wretched gcc thing on 
Apple, the check will need to be more complicated and find if the initializer 
is not the default value for the type.

-- 
 Lubos Lunak
 l.lu...@suse.cz
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: helpcontent2

2012-12-13 Thread Libreoffice Gerrit user
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 3f390ae06bc960b85ee52d0df1ebe6e84358aafa
Author: Michael Meeks michael.me...@suse.com
Date:   Wed Dec 12 19:20:46 2012 +

Updated core
Project: help  df4d4db7408ed3fc135eded6787bad80853619fd
re-base on ALv2 code. Includes:

diff --git a/helpcontent2 b/helpcontent2
index 56562e9..df4d4db 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 56562e9b04dcbfd63012eecddb53b76dc4cee84b
+Subproject commit df4d4db7408ed3fc135eded6787bad80853619fd
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: filter/source

2012-12-13 Thread Libreoffice Gerrit user
 filter/source/xsltdialog/xmlfiltersettingsdialog.cxx |  111 +--
 filter/source/xsltdialog/xmlfiltersettingsdialog.hxx |2 
 2 files changed, 56 insertions(+), 57 deletions(-)

New commits:
commit 3d5789d48d6602832abd33268e31afa4080cb87a
Author: Ricardo Montania rica...@linuxafundo.com.br
Date:   Wed Dec 12 13:26:57 2012 -0200

String removal in filter

Change-Id: I170d132e9329dd58ed6dc9bbf4094c234d122592
Reviewed-on: https://gerrit.libreoffice.org/1309
Reviewed-by: Luboš Luňák l.lu...@suse.cz
Tested-by: Luboš Luňák l.lu...@suse.cz

diff --git a/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx 
b/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx
index edf23e1..29c5d55 100644
--- a/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx
+++ b/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx
@@ -52,7 +52,6 @@ using namespace com::sun::star::container;
 using namespace com::sun::star::beans;
 using namespace com::sun::star::util;
 
-using ::rtl::OUString;
 using ::rtl::Uri;
 
 XMLFilterSettingsDialog::XMLFilterSettingsDialog(Window* pParent,
@@ -90,11 +89,11 @@ XMLFilterSettingsDialog::XMLFilterSettingsDialog(Window* 
pParent,
 
 try
 {
-mxFilterContainer = Reference XNameContainer ::query( 
rxMSF-createInstance( OUString( RTL_CONSTASCII_USTRINGPARAM( 
com.sun.star.document.FilterFactory )) ) );
-mxTypeDetection = Reference XNameContainer ::query( 
rxMSF-createInstance( OUString( RTL_CONSTASCII_USTRINGPARAM( 
com.sun.star.document.TypeDetection )) ));
-mxExtendedTypeDetection = Reference XNameContainer ::query( 
rxMSF-createInstance( OUString( RTL_CONSTASCII_USTRINGPARAM( 
com.sun.star.document.ExtendedTypeDetectionFactory )) ) );
+mxFilterContainer = Reference XNameContainer ::query( 
rxMSF-createInstance( com.sun.star.document.FilterFactory ) );
+mxTypeDetection = Reference XNameContainer ::query( 
rxMSF-createInstance( com.sun.star.document.TypeDetection ) );
+mxExtendedTypeDetection = Reference XNameContainer ::query( 
rxMSF-createInstance( com.sun.star.document.ExtendedTypeDetectionFactory ) );
 
-Reference XConfigManager  xCfgMgr( mxMSF-createInstance(OUString( 
RTL_CONSTASCII_USTRINGPARAM( com.sun.star.config.SpecialConfigManager )) ), 
UNO_QUERY );
+Reference XConfigManager  xCfgMgr( mxMSF-createInstance( 
com.sun.star.config.SpecialConfigManager ), UNO_QUERY );
 if( xCfgMgr.is() )
 {
 m_sTemplatePath = xCfgMgr-substituteVariables( m_sTemplatePath );
@@ -198,7 +197,7 @@ void XMLFilterSettingsDialog::updateStates()
 sal_Int32 nFact = SvtModuleOptions::E_WRITER;
 while(nFact = SvtModuleOptions::E_BASIC)
 {
-::rtl::OUString sDefault = 
maModuleOpt.GetFactoryDefaultFilter((SvtModuleOptions::EFactory)nFact);
+OUString sDefault = 
maModuleOpt.GetFactoryDefaultFilter((SvtModuleOptions::EFactory)nFact);
 if( sDefault == pInfo-maFilterName )
 {
 bIsDefault = true;
@@ -224,14 +223,14 @@ void XMLFilterSettingsDialog::onNew()
 aTempInfo.maFilterName = 
createUniqueFilterName(RESIDSTR(STR_DEFAULT_FILTER_NAME));
 
 // init default extension
-String aDefaultExtension(RESIDSTR(STR_DEFAULT_EXTENSION));
+OUString aDefaultExtension(RESIDSTR(STR_DEFAULT_EXTENSION));
 aTempInfo.maExtension = aDefaultExtension;
 
 // set default ui name
 aTempInfo.maInterfaceName = 
createUniqueInterfaceName(RESIDSTR(STR_DEFAULT_UI_NAME));
 
 // set default application
-aTempInfo.maDocumentService = OUString( RTL_CONSTASCII_USTRINGPARAM( 
com.sun.star.text.TextDocument ));
+aTempInfo.maDocumentService = com.sun.star.text.TextDocument;
 
 // execute XML Filter Dialog
 XMLFilterTabDialog aDlg( this, *getXSLTDialogResMgr(), mxMSF, aTempInfo );
@@ -273,7 +272,7 @@ void XMLFilterSettingsDialog::onEdit()
 
 /** helper to create a sequence of strings from an extensions strings
 ext1;ext2;ext3 will become { ext1, ext2, ext3 } */
-static Sequence OUString  createExtensionsSequence( const rtl::OUString 
rExtensions )
+static Sequence OUString  createExtensionsSequence( const OUString 
rExtensions )
 {
 // first count how many extensions we have inside the string
 int nExtensions = 0;
@@ -419,7 +418,7 @@ OUString 
XMLFilterSettingsDialog::createUniqueInterfaceName( const OUString rIn
 if( nDefaultNumber )
 {
 aInterfaceName += OUString( sal_Unicode( ' ' ) );
-aInterfaceName += String::CreateFromInt32( nDefaultNumber );
+aInterfaceName += OUString::valueOf( 
static_castsal_Int32(nDefaultNumber) );
 }
 
 return aInterfaceName;
@@ -540,28 +539,28 @@ bool XMLFilterSettingsDialog::insertOrEdit( 
filter_info_impl* pNewInfo, const fi
 // 3. create property values for filter entry
 Sequence PropertyValue  aFilterData( 8 );
 
-aFilterData[0].Name = OUString( RTL_CONSTASCII_USTRINGPARAM( Type ) 

[PUSHED] String removal in filter

2012-12-13 Thread via Code Review
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1309

Approvals:
  Luboš Luňák: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I170d132e9329dd58ed6dc9bbf4094c234d122592
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ricardo Montania rica...@linuxafundo.com.br
Gerrit-Reviewer: Luboš Luňák l.lu...@suse.cz

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Remove save to old Microsoft office formats

2012-12-13 Thread Lubos Lunak
On Thursday 13 of December 2012, Caolán McNamara wrote:
 On Thu, 2012-12-13 at 10:05 +, Michael Meeks wrote:
  I assume the root problem you want to fix is of users accidentally
  picking the wrong format in the filter / drop-down and getting a
  needlessly bad experience here ?

 I seem to recall a bug (maybe fixed ?) where in the kde file dialog
 typing foo.doc as the save name would auto pick the word 6 format
 instead of the word 8 format.

 Definitely fixed.

-- 
 Lubos Lunak
 l.lu...@suse.cz
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'distro/suse/suse-3.6' - sw/source

2012-12-13 Thread Libreoffice Gerrit user
 sw/source/core/doc/notxtfrm.cxx   |2 
 sw/source/core/inc/cellfrm.hxx|2 
 sw/source/core/inc/flyfrm.hxx |2 
 sw/source/core/inc/frame.hxx  |2 
 sw/source/core/inc/layfrm.hxx |2 
 sw/source/core/inc/notxtfrm.hxx   |2 
 sw/source/core/inc/pagefrm.hxx|2 
 sw/source/core/inc/rootfrm.hxx|2 
 sw/source/core/inc/txtfrm.hxx |2 
 sw/source/core/layout/trvlfrm.cxx |   96 ++
 sw/source/core/layout/unusedf.cxx |2 
 sw/source/core/text/frmcrsr.cxx   |2 
 12 files changed, 77 insertions(+), 41 deletions(-)

New commits:
commit b3dadcc1f542d406f8cb8c59c57ae34b2de9fe22
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Fri Nov 23 17:51:16 2012 +0100

fdo#52182: Fixed click in frames located in header/footer

Using a distance to click to select the best object to select between
normal text and background object.

(cherry picked from commit e8fbe97900f13305b17015d9044993bde4adab36)

Conflicts:
sw/source/ui/docvw/edtwin.cxx
sw/source/ui/inc/edtwin.hxx

Change-Id: Ib5b53161c7af2c16f4df379382f2e53fc6d8092b
Signed-off-by: Michael Stahl mst...@redhat.com

diff --git a/sw/source/core/doc/notxtfrm.cxx b/sw/source/core/doc/notxtfrm.cxx
index d3ca641..f116e11 100644
--- a/sw/source/core/doc/notxtfrm.cxx
+++ b/sw/source/core/doc/notxtfrm.cxx
@@ -609,7 +609,7 @@ sal_Bool SwNoTxtFrm::GetCharRect( SwRect rRect, const 
SwPosition rPos,
 
 
 sal_Bool SwNoTxtFrm::GetCrsrOfst(SwPosition* pPos, Point ,
- SwCrsrMoveState* ) const
+ SwCrsrMoveState*, bool ) const
 {
 SwCntntNode* pCNd = (SwCntntNode*)GetNode();
 pPos-nNode = *pCNd;
diff --git a/sw/source/core/inc/cellfrm.hxx b/sw/source/core/inc/cellfrm.hxx
index 0bc62ae..958806e 100644
--- a/sw/source/core/inc/cellfrm.hxx
+++ b/sw/source/core/inc/cellfrm.hxx
@@ -48,7 +48,7 @@ public:
 SwCellFrm( const SwTableBox , SwFrm*, bool bInsertContent = true );
 ~SwCellFrm();
 
-virtual sal_Bool GetCrsrOfst( SwPosition *, Point, SwCrsrMoveState* = 0 ) 
const;
+virtual sal_Bool GetCrsrOfst( SwPosition *, Point, SwCrsrMoveState* = 0, 
bool bTestBackground = false ) const;
 virtual void Paint( SwRect const,
 SwPrintData const*const pPrintData = NULL ) const;
 virtual void CheckDirection( sal_Bool bVert );
diff --git a/sw/source/core/inc/flyfrm.hxx b/sw/source/core/inc/flyfrm.hxx
index 170064e..635451b 100644
--- a/sw/source/core/inc/flyfrm.hxx
+++ b/sw/source/core/inc/flyfrm.hxx
@@ -163,7 +163,7 @@ public:
 SwPrintData const*const pPrintData = NULL ) const;
 virtual Size ChgSize( const Size aNewSize );
 virtual sal_Bool GetCrsrOfst( SwPosition *, Point,
-  SwCrsrMoveState* = 0 ) const;
+  SwCrsrMoveState* = 0, bool bTestBackground = 
false ) const;
 
 virtual void  CheckDirection( sal_Bool bVert );
 virtual void Cut();
diff --git a/sw/source/core/inc/frame.hxx b/sw/source/core/inc/frame.hxx
index e2213cf..729d358 100644
--- a/sw/source/core/inc/frame.hxx
+++ b/sw/source/core/inc/frame.hxx
@@ -786,7 +786,7 @@ public:
 virtual boolFillSelection( SwSelectionList rList, const SwRect rRect 
) const;
 
 virtual sal_BoolGetCrsrOfst( SwPosition *, Point,
- SwCrsrMoveState* = 0 ) const;
+ SwCrsrMoveState* = 0, bool bTestBackground = 
false ) const;
 virtual sal_BoolGetCharRect( SwRect , const SwPosition,
  SwCrsrMoveState* = 0 ) const;
 virtual void Paint( SwRect const,
diff --git a/sw/source/core/inc/layfrm.hxx b/sw/source/core/inc/layfrm.hxx
index 4c4e4e6..a8d4bc4 100644
--- a/sw/source/core/inc/layfrm.hxx
+++ b/sw/source/core/inc/layfrm.hxx
@@ -92,7 +92,7 @@ public:
 virtual boolFillSelection( SwSelectionList rList, const SwRect rRect 
) const;
 
 virtual sal_Bool  GetCrsrOfst( SwPosition *, Point,
-   SwCrsrMoveState* = 0 ) const;
+   SwCrsrMoveState* = 0, bool bTestBackground = 
false ) const;
 
 virtual void Cut();
 virtual void Paste( SwFrm* pParent, SwFrm* pSibling = 0 );
diff --git a/sw/source/core/inc/notxtfrm.hxx b/sw/source/core/inc/notxtfrm.hxx
index 17494b0..f8c1033 100644
--- a/sw/source/core/inc/notxtfrm.hxx
+++ b/sw/source/core/inc/notxtfrm.hxx
@@ -61,7 +61,7 @@ public:
 virtual sal_Bool GetCharRect( SwRect , const SwPosition,
   SwCrsrMoveState* = 0) const;
 sal_Bool GetCrsrOfst(SwPosition* pPos, Point aPoint,
- SwCrsrMoveState* = 0) const;
+ SwCrsrMoveState* = 0, bool bTestBackground = false) const;
 
 const Size GetGrfSize() const  { return GetSize(); }
 void GetGrfArea( SwRect rRect, SwRect * = 0, sal_Bool bMirror = sal_True 
) const;

[Libreoffice-commits] .: sw/AllLangResTarget_sw.mk sw/inc sw/source sw/uiconfig sw/UI_swriter.mk

2012-12-13 Thread Libreoffice Gerrit user
 sw/AllLangResTarget_sw.mk|1 
 sw/UI_swriter.mk |1 
 sw/inc/globals.hrc   |2 
 sw/inc/helpid.h  |1 
 sw/source/ui/app/app.src |2 
 sw/source/ui/frmdlg/cption.cxx   |   83 ++-
 sw/source/ui/frmdlg/cption.hrc   |   48 
 sw/source/ui/frmdlg/cption.src   |  152 -
 sw/uiconfig/swriter/ui/captionoptions.ui |  360 +++
 9 files changed, 395 insertions(+), 255 deletions(-)

New commits:
commit f82b7a2aea7bcbc6dca13fc251b11685d62b8380
Author: Jack Leigh leigh...@gmx.se
Date:   Wed Dec 12 21:58:17 2012 +

migrate 'Caption options' dialog to .ui file

Change-Id: I58dbb70faf22c5958b65e5192664b7d12d62f04c
Reviewed-on: https://gerrit.libreoffice.org/1312
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sw/AllLangResTarget_sw.mk b/sw/AllLangResTarget_sw.mk
index ea1196b..d80bfbc 100644
--- a/sw/AllLangResTarget_sw.mk
+++ b/sw/AllLangResTarget_sw.mk
@@ -131,7 +131,6 @@ $(eval $(call gb_SrsTarget_add_files,sw/res,\
 sw/source/ui/fldui/inpdlg.src \
 sw/source/ui/fmtui/tmpdlg.src \
 sw/source/ui/frmdlg/column.src \
-sw/source/ui/frmdlg/cption.src \
 sw/source/ui/frmdlg/frmpage.src \
 sw/source/ui/frmdlg/frmui.src \
 sw/source/ui/frmdlg/wrap.src \
diff --git a/sw/UI_swriter.mk b/sw/UI_swriter.mk
index f39bb2f..9da10bf 100644
--- a/sw/UI_swriter.mk
+++ b/sw/UI_swriter.mk
@@ -12,6 +12,7 @@ $(eval $(call gb_UI_UI,modules/swriter))
 $(eval $(call gb_UI_add_uifiles,modules/swriter,\
sw/uiconfig/swriter/ui/autoformattable \
sw/uiconfig/swriter/ui/bibliographyentry \
+   sw/uiconfig/swriter/ui/captionoptions \
sw/uiconfig/swriter/ui/charurlpage \
sw/uiconfig/swriter/ui/columnwidth \
sw/uiconfig/swriter/ui/converttexttable \
diff --git a/sw/inc/globals.hrc b/sw/inc/globals.hrc
index 5362aee..1b79466 100644
--- a/sw/inc/globals.hrc
+++ b/sw/inc/globals.hrc
@@ -73,7 +73,7 @@
 #define STR_CAPTION_END (RC_GLOBALS_BEGIN + 43)
 #define STR_CAPTION_ABOVE   (RC_GLOBALS_BEGIN + 44)
 #define STR_CAPTION_BELOW   (RC_GLOBALS_BEGIN + 45)
-#define STR_CAPTION_CATEGORY_NONE   (RC_GLOBALS_BEGIN + 46)
+#define STR_CAPTION_NONE(RC_GLOBALS_BEGIN + 46)
 
 // DIALOGS ---
 
diff --git a/sw/inc/helpid.h b/sw/inc/helpid.h
index 917d9de..d443b6d 100644
--- a/sw/inc/helpid.h
+++ b/sw/inc/helpid.h
@@ -132,7 +132,6 @@
 #define HID_SW_SORT_POSITION
SW_HID_SW_SORT_POSITION
 #define HID_SYNC_BTN
SW_HID_SYNC_BTN
 #define HID_EDIT_COMMENT
SW_HID_EDIT_COMMENT
-#define HID_DLG_SEQUENCE_OPTION 
SW_HID_DLG_SEQUENCE_OPTION
 #define HID_DLG_FLDEDT_NEXT 
SW_HID_DLG_FLDEDT_NEXT
 #define HID_DLG_FLDEDT_PREV 
SW_HID_DLG_FLDEDT_PREV
 #define HID_DLG_FLDEDT_ADDRESS  
SW_HID_DLG_FLDEDT_ADDRESS
diff --git a/sw/source/ui/app/app.src b/sw/source/ui/app/app.src
index 9111d94..0239ddf 100644
--- a/sw/source/ui/app/app.src
+++ b/sw/source/ui/app/app.src
@@ -256,7 +256,7 @@ String STR_CAPTION_BELOW
 {
 Text [ en-US ] = Below ;
 };
-String STR_CAPTION_CATEGORY_NONE
+String STR_CAPTION_NONE
 {
 Text [ en-US ] = None ;
 };
diff --git a/sw/source/ui/frmdlg/cption.cxx b/sw/source/ui/frmdlg/cption.cxx
index 404b36d..3585ded 100644
--- a/sw/source/ui/frmdlg/cption.cxx
+++ b/sw/source/ui/frmdlg/cption.cxx
@@ -44,7 +44,6 @@
 #include comphelper/string.hxx
 #include frmui.hrc
 #include globals.hrc
-#include cption.hrc
 #include SwStyleNameMapper.hxx
 
 using namespace ::com::sun::star;
@@ -56,25 +55,14 @@ extern String* GetOldDrwCat();
 
 class SwSequenceOptionDialog : public SvxStandardDialog
 {
-FixedLine   aFlHeader;
-FixedText   aFtLevel;
-ListBox aLbLevel;
-FixedText   aFtDelim;
-EditaEdDelim;
+ListBox*m_pLbLevel;
+Edit*   m_pEdDelim;
 
-FixedLine   aFlCatAndFrame;
-FixedText   aFtCharStyle;
-ListBox aLbCharStyle;
-CheckBoxaApplyBorderAndShadowCB;
+ListBox*m_pLbCharStyle;
+CheckBox*   m_pApplyBorderAndShadowCB;
 
 //#i61007# order of captions
-FixedLine   aFlCaptionOrder;
-FixedText   aFtCaptionOrder;
-ListBox aLbCaptionOrder;
-
-OKButtonaOKButton;
-CancelButtonaCancelButton;
-HelpButton  aHelpButton;
+ListBox*m_pLbCaptionOrder;
 
 SwView rView;
 String  aFldTypeName;
@@ -85,12 +73,12 @@ public:
 virtual ~SwSequenceOptionDialog();
 virtual void Apply();
 

[PUSHED] migrate 'Caption options' dialog to .ui file

2012-12-13 Thread via Code Review
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1312

Approvals:
  Caolán McNamara: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I58dbb70faf22c5958b65e5192664b7d12d62f04c
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Jack Leigh leigh...@gmx.se
Gerrit-Reviewer: Caolán McNamara caol...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - helpcontent2

2012-12-13 Thread Libreoffice Gerrit user
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 62cde8eded955de1164c6c9b347cab21c35963af
Author: Michael Meeks michael.me...@suse.com
Date:   Wed Dec 12 19:20:46 2012 +

Updated core
Project: help  32c4964e1415547af5dd956b590dc0edb3d10778
re-base on ALv2 code. Includes:

diff --git a/helpcontent2 b/helpcontent2
index d9d28eb..32c4964 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit d9d28ebb575e96e742d6590a28dd59c62b89dc4e
+Subproject commit 32c4964e1415547af5dd956b590dc0edb3d10778
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 5 commits - sfx2/inc sfx2/source svl/source

2012-12-13 Thread Libreoffice Gerrit user
 sfx2/inc/sfx2/templateabstractview.hxx|   13 +
 sfx2/inc/sfx2/templatelocalview.hxx   |2 
 sfx2/inc/sfx2/templateremoteview.hxx  |7 -
 sfx2/inc/sfx2/templateview.hxx|   22 ---
 sfx2/inc/sfx2/thumbnailview.hxx   |   12 -
 sfx2/inc/sfx2/thumbnailviewitem.hxx   |1 
 sfx2/inc/templatedlg.hxx  |4 
 sfx2/source/control/templateabstractview.cxx  |   19 ++
 sfx2/source/control/templatelocalview.cxx |   27 
 sfx2/source/control/templatelocalviewitem.cxx |6 
 sfx2/source/control/templateremoteview.cxx|   27 
 sfx2/source/control/templateview.cxx  |  175 ++
 sfx2/source/control/templateview.hrc  |4 
 sfx2/source/control/templateview.src  |   22 ++-
 sfx2/source/control/templateviewitem.cxx  |6 
 sfx2/source/control/thumbnailview.cxx |   37 -
 sfx2/source/doc/templatedlg.cxx   |  119 -
 sfx2/source/doc/templatedlg.src   |1 
 svl/source/misc/inettype.cxx  |   19 +-
 19 files changed, 140 insertions(+), 383 deletions(-)

New commits:
commit cc9329c52c8327f0d8b52162755959bac9364265
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Thu Dec 13 14:53:01 2012 +0100

Bad compareToIgnoreAsciiCaseAscii result comparison caused endless loop

Change-Id: I6ef5aa87623a0d61942183b7eb888fc0f9cf6af0

diff --git a/svl/source/misc/inettype.cxx b/svl/source/misc/inettype.cxx
index 143a11a..d5f01f6 100644
--- a/svl/source/misc/inettype.cxx
+++ b/svl/source/misc/inettype.cxx
@@ -643,19 +643,14 @@ MediaTypeEntry const * seekEntry(OUString const  
rTypeName,
 {
 sal_Size nMiddle = (nLow + nHigh) / 2;
 MediaTypeEntry const * pEntry = pMap + nMiddle;
-switch (rTypeName.compareToIgnoreAsciiCaseAscii(pEntry-m_pTypeName))
-{
-case COMPARE_LESS:
-nHigh = nMiddle;
-break;
-
-case COMPARE_EQUAL:
-return pEntry;
+sal_Int32 nCmp = 
rTypeName.compareToIgnoreAsciiCaseAscii(pEntry-m_pTypeName);
+if (nCmp  0)
+nHigh = nMiddle;
+else if (nCmp == 0)
+return pEntry;
 
-case COMPARE_GREATER:
-nLow = nMiddle + 1;
-break;
-}
+else
+nLow = nMiddle + 1;
 }
 return 0;
 }
commit 8f54315f716ab37331caf80d4cf3d5aeacc78644
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Thu Dec 13 14:29:52 2012 +0100

Template Manager: make it sizeable

Change-Id: I14d3c73b714b9cbfefad7adcb02efb790c8a1e5b

diff --git a/sfx2/inc/sfx2/templateabstractview.hxx 
b/sfx2/inc/sfx2/templateabstractview.hxx
index d1e4b14..a86ced0 100644
--- a/sfx2/inc/sfx2/templateabstractview.hxx
+++ b/sfx2/inc/sfx2/templateabstractview.hxx
@@ -117,10 +117,10 @@ public:
 
 static BitmapEx fetchThumbnail (const OUString msURL, long width, long 
height);
 
-protected:
-
 virtual void Resize();
 
+protected:
+
 virtual void Paint( const Rectangle rRect );
 
 virtual void DrawItem (ThumbnailViewItem *pItem);
diff --git a/sfx2/inc/sfx2/thumbnailview.hxx b/sfx2/inc/sfx2/thumbnailview.hxx
index 6d3a4c2..9548bd1 100644
--- a/sfx2/inc/sfx2/thumbnailview.hxx
+++ b/sfx2/inc/sfx2/thumbnailview.hxx
@@ -236,6 +236,8 @@ public:
 
 void setItemStateHdl (const Link aLink) { maItemStateHdl = aLink; }
 
+virtual void Resize();
+
 protected:
 
 virtual void MouseButtonDown( const MouseEvent rMEvt );
@@ -250,8 +252,6 @@ protected:
 
 virtual void LoseFocus();
 
-virtual void Resize();
-
 virtual void StateChanged( StateChangedType nStateChange );
 
 virtual void DataChanged( const DataChangedEvent rDCEvt );
diff --git a/sfx2/inc/templatedlg.hxx b/sfx2/inc/templatedlg.hxx
index f24c8e4..3f7ba13 100644
--- a/sfx2/inc/templatedlg.hxx
+++ b/sfx2/inc/templatedlg.hxx
@@ -52,6 +52,8 @@ private:
 
 virtual void MouseButtonDown( const MouseEvent rMEvt );
 
+virtual void Resize ();
+
 DECL_LINK(CloseOverlayHdl, void*);
 
 DECL_LINK(TBXViewHdl, void*);
diff --git a/sfx2/source/control/templateabstractview.cxx 
b/sfx2/source/control/templateabstractview.cxx
index 02c61b0..2caeba3 100644
--- a/sfx2/source/control/templateabstractview.cxx
+++ b/sfx2/source/control/templateabstractview.cxx
@@ -275,6 +275,7 @@ BitmapEx TemplateAbstractView::fetchThumbnail (const 
OUString msURL, long width
 void TemplateAbstractView::Resize()
 {
 mpItemView-SetSizePixel(GetSizePixel());
+ThumbnailView::Resize();
 }
 
 void TemplateAbstractView::Paint(const Rectangle rRect)
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 1d0512e..3546826 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -122,32 +122,6 @@ SfxTemplateManagerDlg::SfxTemplateManagerDlg (Window 
*parent)
 

Re: Help with updating types.rdb

2012-12-13 Thread Kohei Yoshida

On 12/13/2012 05:15 AM, Stephan Bergmann wrote:

On 12/12/2012 07:12 PM, Kohei Yoshida wrote:

I'm trying to unpublish several UNO API as discussed in ESC meeting.
I've done unpublishing but am struggling with updating the types.rdb.

[...]

Help appreciated.


Seeing that 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=0315ef5bbc168896b1d7d8ac5c1657f861fa6a93
API CHANGE: unpublish Data Pilot interfaces contains changes to
offapi/type_reference/types.rdb that amount to removing the published
flag from certain interfaces and services, I assume that you managed to
get that done after all?


Well, in the end Michael Stahl picked it up for me (the types.rdb part). 
 I remained unsuccessful until the end.


Kohei

--
Kohei Yoshida, LibreOffice hacker, Calc
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - 5 commits - sfx2/inc sfx2/source svl/source

2012-12-13 Thread Libreoffice Gerrit user
 sfx2/inc/sfx2/templateabstractview.hxx|   13 +
 sfx2/inc/sfx2/templatelocalview.hxx   |2 
 sfx2/inc/sfx2/templateremoteview.hxx  |7 -
 sfx2/inc/sfx2/templateview.hxx|   22 ---
 sfx2/inc/sfx2/thumbnailview.hxx   |   12 -
 sfx2/inc/sfx2/thumbnailviewitem.hxx   |1 
 sfx2/inc/templatedlg.hxx  |4 
 sfx2/source/control/templateabstractview.cxx  |   19 ++
 sfx2/source/control/templatelocalview.cxx |   27 
 sfx2/source/control/templatelocalviewitem.cxx |6 
 sfx2/source/control/templateremoteview.cxx|   27 
 sfx2/source/control/templateview.cxx  |  175 ++
 sfx2/source/control/templateview.hrc  |4 
 sfx2/source/control/templateview.src  |   22 ++-
 sfx2/source/control/templateviewitem.cxx  |6 
 sfx2/source/control/thumbnailview.cxx |   37 -
 sfx2/source/doc/templatedlg.cxx   |  119 -
 sfx2/source/doc/templatedlg.src   |1 
 svl/source/misc/inettype.cxx  |   19 +-
 19 files changed, 140 insertions(+), 383 deletions(-)

New commits:
commit c69399e5a12cd4e63efc7280c8470b514ab2f453
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Thu Dec 13 14:53:01 2012 +0100

Bad compareToIgnoreAsciiCaseAscii result comparison caused endless loop

Change-Id: I6ef5aa87623a0d61942183b7eb888fc0f9cf6af0

diff --git a/svl/source/misc/inettype.cxx b/svl/source/misc/inettype.cxx
index 143a11a..d5f01f6 100644
--- a/svl/source/misc/inettype.cxx
+++ b/svl/source/misc/inettype.cxx
@@ -643,19 +643,14 @@ MediaTypeEntry const * seekEntry(OUString const  
rTypeName,
 {
 sal_Size nMiddle = (nLow + nHigh) / 2;
 MediaTypeEntry const * pEntry = pMap + nMiddle;
-switch (rTypeName.compareToIgnoreAsciiCaseAscii(pEntry-m_pTypeName))
-{
-case COMPARE_LESS:
-nHigh = nMiddle;
-break;
-
-case COMPARE_EQUAL:
-return pEntry;
+sal_Int32 nCmp = 
rTypeName.compareToIgnoreAsciiCaseAscii(pEntry-m_pTypeName);
+if (nCmp  0)
+nHigh = nMiddle;
+else if (nCmp == 0)
+return pEntry;
 
-case COMPARE_GREATER:
-nLow = nMiddle + 1;
-break;
-}
+else
+nLow = nMiddle + 1;
 }
 return 0;
 }
commit bd6b9414e7914a50d24db5b37ebefb831eabec79
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Thu Dec 13 14:29:52 2012 +0100

Template Manager: make it sizeable

Change-Id: I14d3c73b714b9cbfefad7adcb02efb790c8a1e5b

diff --git a/sfx2/inc/sfx2/templateabstractview.hxx 
b/sfx2/inc/sfx2/templateabstractview.hxx
index d1e4b14..a86ced0 100644
--- a/sfx2/inc/sfx2/templateabstractview.hxx
+++ b/sfx2/inc/sfx2/templateabstractview.hxx
@@ -117,10 +117,10 @@ public:
 
 static BitmapEx fetchThumbnail (const OUString msURL, long width, long 
height);
 
-protected:
-
 virtual void Resize();
 
+protected:
+
 virtual void Paint( const Rectangle rRect );
 
 virtual void DrawItem (ThumbnailViewItem *pItem);
diff --git a/sfx2/inc/sfx2/thumbnailview.hxx b/sfx2/inc/sfx2/thumbnailview.hxx
index 6d3a4c2..9548bd1 100644
--- a/sfx2/inc/sfx2/thumbnailview.hxx
+++ b/sfx2/inc/sfx2/thumbnailview.hxx
@@ -236,6 +236,8 @@ public:
 
 void setItemStateHdl (const Link aLink) { maItemStateHdl = aLink; }
 
+virtual void Resize();
+
 protected:
 
 virtual void MouseButtonDown( const MouseEvent rMEvt );
@@ -250,8 +252,6 @@ protected:
 
 virtual void LoseFocus();
 
-virtual void Resize();
-
 virtual void StateChanged( StateChangedType nStateChange );
 
 virtual void DataChanged( const DataChangedEvent rDCEvt );
diff --git a/sfx2/inc/templatedlg.hxx b/sfx2/inc/templatedlg.hxx
index f24c8e4..3f7ba13 100644
--- a/sfx2/inc/templatedlg.hxx
+++ b/sfx2/inc/templatedlg.hxx
@@ -52,6 +52,8 @@ private:
 
 virtual void MouseButtonDown( const MouseEvent rMEvt );
 
+virtual void Resize ();
+
 DECL_LINK(CloseOverlayHdl, void*);
 
 DECL_LINK(TBXViewHdl, void*);
diff --git a/sfx2/source/control/templateabstractview.cxx 
b/sfx2/source/control/templateabstractview.cxx
index ae9e958..b66c3c9 100644
--- a/sfx2/source/control/templateabstractview.cxx
+++ b/sfx2/source/control/templateabstractview.cxx
@@ -282,6 +282,7 @@ BitmapEx TemplateAbstractView::fetchThumbnail (const 
OUString msURL, long width
 void TemplateAbstractView::Resize()
 {
 mpItemView-SetSizePixel(GetSizePixel());
+ThumbnailView::Resize();
 }
 
 void TemplateAbstractView::Paint(const Rectangle rRect)
diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 1d0512e..3546826 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -122,32 +122,6 @@ SfxTemplateManagerDlg::SfxTemplateManagerDlg (Window 
*parent)
 

[Libreoffice-commits] .: 2 commits - desktop/source

2012-12-13 Thread Libreoffice Gerrit user
 desktop/source/app/officeipcthread.cxx |   64 ++---
 desktop/source/app/officeipcthread.hxx |2 -
 2 files changed, 28 insertions(+), 38 deletions(-)

New commits:
commit 4ce2602befd59e69264d8e4ced8730b40c2b947c
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Dec 13 15:41:10 2012 +0100

Related fdo#33484: Terminate OfficeIPCThread by closing the accepting pipe

... (and setting mbDowning to indicate an error returned from accept() is 
due to
termination) instead of setting up an extra pipe connection to send an
InternalIPC::TerminateThread message (which allegedly caused deadlocks, 
see
https://gerrit.libreoffice.org/#/c/1311/ Change Idf933915: office ipc: 
use
timeout pipe feature when connecting to self).

Change-Id: Id302ca13112fc409685e7665b38f1030704a0ccf

diff --git a/desktop/source/app/officeipcthread.cxx 
b/desktop/source/app/officeipcthread.cxx
index e287d2b..8828311 100644
--- a/desktop/source/app/officeipcthread.cxx
+++ b/desktop/source/app/officeipcthread.cxx
@@ -51,8 +51,6 @@ using ::rtl::OString;
 using ::rtl::OUString;
 using ::rtl::OUStringBuffer;
 
-const char  *OfficeIPCThread::sc_aTerminationSequence = 
InternalIPC::TerminateThread;
-const int OfficeIPCThread::sc_nTSeqLength = 28;
 const char  *OfficeIPCThread::sc_aShowSequence = -tofront;
 const int OfficeIPCThread::sc_nShSeqLength = 5;
 const char  *OfficeIPCThread::sc_aConfirmationSequence = 
InternalIPC::ProcessingDone;
@@ -427,8 +425,6 @@ OfficeIPCThread::Status 
OfficeIPCThread::EnableOfficeIPCThread()
 
 rtl::Reference OfficeIPCThread  pThread(new OfficeIPCThread);
 
-pThread-maPipeIdent = OUString( SingleOfficeIPC_  );
-
 // The name of the named pipe is created with the hashcode of the user 
installation directory (without /user). We have to retrieve
 // this information from a unotools implementation.
 ::utl::Bootstrap::PathStatus aLocateResult = 
::utl::Bootstrap::locateUserInstallation( aUserInstallPath );
@@ -485,19 +481,19 @@ OfficeIPCThread::Status 
OfficeIPCThread::EnableOfficeIPCThread()
 if ( aUserInstallPathHashCode.isEmpty() )
 return IPC_STATUS_BOOTSTRAP_ERROR; // Something completely broken, we 
cannot create a valid hash code!
 
-pThread-maPipeIdent = pThread-maPipeIdent + aUserInstallPathHashCode;
+OUString aPipeIdent( SingleOfficeIPC_ + aUserInstallPathHashCode );
 
 PipeMode nPipeMode = PIPEMODE_DONTKNOW;
 do
 {
 osl::Security rSecurity = Security::get();
 // Try to create pipe
-if ( pThread-maPipe.create( pThread-maPipeIdent.getStr(), 
osl_Pipe_CREATE, rSecurity ))
+if ( pThread-maPipe.create( aPipeIdent.getStr(), osl_Pipe_CREATE, 
rSecurity ))
 {
 // Pipe created
 nPipeMode = PIPEMODE_CREATED;
 }
-else if( pThread-maPipe.create( pThread-maPipeIdent.getStr(), 
osl_Pipe_OPEN, rSecurity )) // Creation not successfull, now we try to connect
+else if( pThread-maPipe.create( aPipeIdent.getStr(), osl_Pipe_OPEN, 
rSecurity )) // Creation not successfull, now we try to connect
 {
 osl::StreamPipe aStreamPipe(pThread-maPipe.getHandle());
 char pReceiveBuffer[sc_nCSASeqLength + 1];
@@ -601,18 +597,8 @@ void OfficeIPCThread::DisableOfficeIPCThread(bool join)
 pGlobalOfficeIPCThread);
 pGlobalOfficeIPCThread.clear();
 
-// send thread a termination message
-// this is done so the subsequent join will not hang
-// because the thread hangs in accept of pipe
-osl::StreamPipe aPipe ( pOfficeIPCThread-maPipeIdent, osl_Pipe_OPEN, 
Security::get() );
-if (aPipe.is())
-{
-aPipe.send( sc_aTerminationSequence, sc_nTSeqLength+1 ); // also 
send 0-byte
-
-// close the pipe so that the streampipe on the other
-// side produces EOF
-aPipe.close();
-}
+pOfficeIPCThread-mbDowning = true;
+pOfficeIPCThread-maPipe.close();
 
 // release mutex to avoid deadlocks
 aMutex.clear();
@@ -680,22 +666,23 @@ void OfficeIPCThread::execute()
 // down during wait
 osl::ClearableMutexGuard aGuard( GetMutex() );
 
-if (!mbDowning)
+if ( mbDowning )
 {
-// notify client we're ready to process its args
-int nBytes = 0;
-int nResult = 0;
-while (
-(nResult = 
aStreamPipe.send(sc_aSendArgumentsSequence+nBytes, sc_nCSASeqLength-nBytes))0 

-((nBytes += nResult)  sc_nCSASeqLength) ) ;
+break;
 }
+
+// notify client we're ready to process its args
+int nBytes = 0;
+int nResult;
+while (
+(nResult = aStreamPipe.send(sc_aSendArgumentsSequence+nBytes, 
sc_nCSASeqLength-nBytes))0 
+((nBytes += 

[Libreoffice-commits] .: 3 commits - sfx2/inc sfx2/source sw/inc sw/source sw/uiconfig vcl/inc vcl/source

2012-12-13 Thread Libreoffice Gerrit user
 sfx2/inc/sfx2/sfxdlg.hxx |   20 +--
 sfx2/inc/sfx2/tabdlg.hxx |   12 +
 sfx2/source/dialog/tabdlg.cxx|   20 +--
 sfx2/source/view/printer.cxx |2 
 sw/inc/swabstdlg.hxx |2 
 sw/source/ui/app/docst.cxx   |  202 ++-
 sw/source/ui/dialog/swdlgfact.cxx|   30 ++--
 sw/source/ui/dialog/swdlgfact.hxx|   19 ++
 sw/source/ui/fmtui/tmpdlg.cxx|   12 -
 sw/source/ui/inc/tmpdlg.hxx  |1 
 sw/uiconfig/swriter/ui/captionoptions.ui |   14 +-
 sw/uiconfig/swriter/ui/insertcaption.ui  |   12 +
 vcl/inc/vcl/dialog.hxx   |4 
 vcl/source/window/dialog.cxx |   10 +
 14 files changed, 226 insertions(+), 134 deletions(-)

New commits:
commit 0157f0b1ba364f7f9af2aacd1be9fbb5ddec2b4d
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Dec 13 10:12:13 2012 +

Related: fdo#38606 keep style dialog open when apply is clicked

i.e. don't close and reopen, just stay open

Change-Id: I11cdc492e3c4adfb75c9a0dfa52e5ba351914858

diff --git a/sfx2/inc/sfx2/sfxdlg.hxx b/sfx2/inc/sfx2/sfxdlg.hxx
index 3585645..0465dbb 100644
--- a/sfx2/inc/sfx2/sfxdlg.hxx
+++ b/sfx2/inc/sfx2/sfxdlg.hxx
@@ -55,7 +55,7 @@ namespace com { namespace sun { namespace star { namespace 
frame {
 class XModel;
 } } } }
 
-class SfxAbstractDialog : public VclAbstractDialog
+class SfxAbstractDialog : virtual public VclAbstractDialog
 {
 public:
 virtual const SfxItemSet*   GetOutputItemSet() const = 0;
@@ -63,15 +63,21 @@ public:
 virtual String  GetText() const = 0;
 };
 
-class SfxAbstractTabDialog : public SfxAbstractDialog
+class SfxAbstractTabDialog : virtual public SfxAbstractDialog
 {
 public:
 virtual voidSetCurPageId( sal_uInt16 nId ) = 0;
-virtual const sal_uInt16*   GetInputRanges( const SfxItemPool ) = 0;
+virtual const sal_uInt16*   GetInputRanges( const SfxItemPool ) = 0;
 virtual voidSetInputSet( const SfxItemSet* pInSet ) = 0;
 };
 
-class SfxAbstractInsertObjectDialog : public VclAbstractDialog
+class SfxAbstractApplyTabDialog : virtual public SfxAbstractTabDialog
+{
+public:
+virtual voidSetApplyHdl( const Link rLink ) = 0;
+};
+
+class SfxAbstractInsertObjectDialog : virtual public VclAbstractDialog
 {
 public:
 virtual com::sun::star::uno::Reference  
com::sun::star::embed::XEmbeddedObject  GetObject()=0;
@@ -79,7 +85,7 @@ public:
 virtual sal_Bool IsCreateNew()=0;
 };
 
-class SfxAbstractPasteDialog : public VclAbstractDialog
+class SfxAbstractPasteDialog : virtual public VclAbstractDialog
 {
 public:
 virtual void Insert( SotFormatStringId nFormat, const String  rFormatName 
) = 0;
@@ -89,11 +95,11 @@ public:
 const TransferableObjectDescriptor* pDesc=0 ) = 0;
 };
 
-class SfxAbstractLinksDialog : public VclAbstractDialog
+class SfxAbstractLinksDialog : virtual public VclAbstractDialog
 {
 };
 
-class AbstractScriptSelectorDialog : public VclAbstractDialog
+class AbstractScriptSelectorDialog : virtual public VclAbstractDialog
 {
 public:
 virtual String  GetScriptURL() const = 0;
diff --git a/sfx2/inc/sfx2/tabdlg.hxx b/sfx2/inc/sfx2/tabdlg.hxx
index 6b1e985..c07c012 100644
--- a/sfx2/inc/sfx2/tabdlg.hxx
+++ b/sfx2/inc/sfx2/tabdlg.hxx
@@ -201,7 +201,13 @@ public:
 voidSetApplyHandler(const Link _rHdl);
 
 SAL_DLLPRIVATE void Start_Impl();
-SAL_DLLPRIVATE sal_Bool OK_Impl() { return PrepareLeaveCurrentPage(); }
+bool OK_Impl()
+{
+bool bRet = PrepareLeaveCurrentPage();
+if (bRet)
+Ok();
+return bRet;
+}
 };
 
 namespace sfx { class ItemConnectionBase; }
diff --git a/sw/inc/swabstdlg.hxx b/sw/inc/swabstdlg.hxx
index 555680f..a6f8ab5 100644
--- a/sw/inc/swabstdlg.hxx
+++ b/sw/inc/swabstdlg.hxx
@@ -416,7 +416,7 @@ public:
 sal_BoolbFmt = 
sal_False,
 sal_uInt16  nDefPage = 
0,
 const String*   pFmtStr  = 0) 
= 0;  //add for SwFrmDlg
-virtual SfxAbstractTabDialog*   CreateTemplateDialog( int nResId,
+virtual SfxAbstractApplyTabDialog*  CreateTemplateDialog(
 Window* pParent,
 SfxStyleSheetBase  rBase,
 sal_uInt16  
nRegion,
diff --git a/sw/source/ui/app/docst.cxx b/sw/source/ui/app/docst.cxx
index 3685eb1..ad8c702 100644
--- a/sw/source/ui/app/docst.cxx
+++ b/sw/source/ui/app/docst.cxx
@@ -497,6 +497,118 @@ void SwDocShell::ExecStyleSheet( SfxRequest rReq )
 
 }
 
+class ApplyStyle
+{
+public:
+ApplyStyle(SwDocShell rDocSh, bool bNew, SfxStyleSheetBase* pStyle,
+  

[PATCH] fix alphebetization

2012-12-13 Thread Jack Leigh (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1323

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/23/1323/1

fix alphebetization

Change-Id: Iabf4d1e4d02c81cab580c07b65bb17da1877c6b1
---
M cui/UI_cui.mk
1 file changed, 2 insertions(+), 2 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iabf4d1e4d02c81cab580c07b65bb17da1877c6b1
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Jack Leigh leigh...@gmx.se

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: sfx2/source

2012-12-13 Thread Libreoffice Gerrit user
 sfx2/source/doc/templatedlg.src |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit af0a4a7a4efe19c88cf7cf567f49abe21a69df3d
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Thu Dec 13 16:07:29 2012 +0100

Template Manager: fixed typo in .src file

Change-Id: I27195076b6a0a8f48c676d884d671fc9a746ce6b

diff --git a/sfx2/source/doc/templatedlg.src b/sfx2/source/doc/templatedlg.src
index a9a844c..2480862 100644
--- a/sfx2/source/doc/templatedlg.src
+++ b/sfx2/source/doc/templatedlg.src
@@ -107,7 +107,7 @@ ModelessDialog DLG_TEMPLATE_MANAGER
 SVLook = TRUE;
 Moveable = TRUE;
 Closeable = TRUE;
-Resizeable = TRUE;
+Sizeable = TRUE;
 Hide = TRUE;
 Size = MAP_APPFONT ( 290 , 250 );
 Text [en-US] = Template Manager;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - sfx2/source

2012-12-13 Thread Libreoffice Gerrit user
 sfx2/source/doc/templatedlg.src |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 83c36b07fc6d33577f2e624110e500b47842f910
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Thu Dec 13 16:07:29 2012 +0100

Template Manager: fixed typo in .src file

Change-Id: I27195076b6a0a8f48c676d884d671fc9a746ce6b

diff --git a/sfx2/source/doc/templatedlg.src b/sfx2/source/doc/templatedlg.src
index a9a844c..2480862 100644
--- a/sfx2/source/doc/templatedlg.src
+++ b/sfx2/source/doc/templatedlg.src
@@ -107,7 +107,7 @@ ModelessDialog DLG_TEMPLATE_MANAGER
 SVLook = TRUE;
 Moveable = TRUE;
 Closeable = TRUE;
-Resizeable = TRUE;
+Sizeable = TRUE;
 Hide = TRUE;
 Size = MAP_APPFONT ( 290 , 250 );
 Text [en-US] = Template Manager;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


git-new-workdir

2012-12-13 Thread Lionel Elie Mamane
On Wed, Dec 12, 2012 at 02:21:18PM -0500, Kohei Yoshida wrote:

 Not following the thread fully I'm not sure what this is about.  I'm
 guessing this is about git-new-workdir vs submodules.

 To put it short, I got it to work, and so far I haven't seen any
 major issues but some occasional oddities which I just dismiss for
 now.

What version of git do you use? I haven't gotten it to work on the
submodules, actually. So I duplicate the submodules currently.

I'd appreciate a description of how did it, actually.

 To me git-new-workdir is not about diskspace, but about avoiding the
 hassle of managing multiple local repos.  Keeping it in one local
 repo make it easier to manage especially when I tend to cherry-pick
 between branches quite a lot.

Exactly. In its absence, you have to constantly push/pull from one
local repo to the other, and that's a hassle.

-- 
Lionel
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: l10ntools/source

2012-12-13 Thread Libreoffice Gerrit user
 l10ntools/source/po.cxx |   17 +
 1 file changed, 9 insertions(+), 8 deletions(-)

New commits:
commit 3ddd4502c4d4e234017e01ead26813caf81593e8
Author: Zolnai Tamás zolnaitamas2...@gmail.com
Date:   Thu Dec 13 15:32:02 2012 +0100

Fix adding keyid to po when read from file

Change-Id: I29fdd23da97f1102974a6b5821c224264a37efc3

diff --git a/l10ntools/source/po.cxx b/l10ntools/source/po.cxx
index 8d51671..cdc2c98 100644
--- a/l10ntools/source/po.cxx
+++ b/l10ntools/source/po.cxx
@@ -793,14 +793,7 @@ void PoIfstream::readEntry( PoEntry rPoEntry )
 (sType == text || sType == quickhelptext || sType == title)
 !aGenPo.getMsgId().isEmpty() )
 {
-if( rPoEntry.m_pGenPo )
-{
-*(rPoEntry.m_pGenPo) = aGenPo;
-}
-else
-{
-rPoEntry.m_pGenPo = new GenPoEntry( aGenPo );
-}
+//Generate keyid if po file not includes it
 const OString sExtractCom = aGenPo.getExtractCom();
 if( sExtractCom.isEmpty() ||
 ( sExtractCom.getLength() != 4 
@@ -812,6 +805,14 @@ void PoIfstream::readEntry( PoEntry rPoEntry )
 aGenPo.getReference() + sMsgCtxt +
 aGenPo.getMsgId() ) );
 }
+if( rPoEntry.m_pGenPo )
+{
+*(rPoEntry.m_pGenPo) = aGenPo;
+}
+else
+{
+rPoEntry.m_pGenPo = new GenPoEntry( aGenPo );
+}
 rPoEntry.m_bIsInitialized = true;
 }
 else
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [PATCH] Printer Properties widget and File-Properties widget

2012-12-13 Thread Caolán McNamara
On Tue, 2012-12-11 at 18:52 +, Gokul Swaminathan wrote:
 Dear Mr. Caolan
Manual conversion of Printer property dialog and 
 file - Properties dialog.
 I have an doubt Regarding Custom Properties tab. we need to add 4 items on 
 clicking the
  ADD button, how to handle it using Scroll bar.
 
 As of now i have added Scrolled window

I've pushed these now to master, I haven't gotten around to adapting the
code yet to use them. For the custom properties tab its probably a
custom widget and we'll just have a placeholder in the .ui that get
replaced by the custom widget, so a scrolled window (or a
GtkDrawingArea) is fine as a placeholder for those cases.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PATCH][PUSHED] Printer Properties widget and File-Properties widget

2012-12-13 Thread Caolán McNamara
On Thu, 2012-12-13 at 15:32 +, Caolán McNamara wrote:
 I've pushed these now to master.

grr, wanted to mark thread as pushed.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: cui/UI_cui.mk

2012-12-13 Thread Libreoffice Gerrit user
 cui/UI_cui.mk |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit b040da8d0ea66a783635760ba6494a0f09cd2deb
Author: Jack Leigh leigh...@gmx.se
Date:   Thu Dec 13 14:53:00 2012 +

fix alphebetization

Change-Id: Iabf4d1e4d02c81cab580c07b65bb17da1877c6b1
Reviewed-on: https://gerrit.libreoffice.org/1323
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/cui/UI_cui.mk b/cui/UI_cui.mk
index a9ecc1e..32b64d4 100644
--- a/cui/UI_cui.mk
+++ b/cui/UI_cui.mk
@@ -17,11 +17,11 @@ $(eval $(call gb_UI_add_uifiles,cui,\
cui/uiconfig/ui/insertoleobject \
cui/uiconfig/ui/insertplugin \
cui/uiconfig/ui/insertrowcolumn \
-   cui/uiconfig/ui/numberformatpage \
-   cui/uiconfig/ui/scriptorganizer \
cui/uiconfig/ui/macroselectordialog \
+   cui/uiconfig/ui/numberformatpage \
cui/uiconfig/ui/personalization_tab \
cui/uiconfig/ui/positionpage \
+   cui/uiconfig/ui/scriptorganizer \
cui/uiconfig/ui/select_persona_dialog \
cui/uiconfig/ui/specialcharacters \
cui/uiconfig/ui/thesaurus \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] fix alphebetization

2012-12-13 Thread via Code Review
Hi,

Thank you for your patch!  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/1323

Approvals:
  Caolán McNamara: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iabf4d1e4d02c81cab580c07b65bb17da1877c6b1
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Jack Leigh leigh...@gmx.se
Gerrit-Reviewer: Caolán McNamara caol...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: filter/source

2012-12-13 Thread Libreoffice Gerrit user
 filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu  |  
  2 +-
 filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu |  
  2 +-
 filter/source/config/fragments/filters/MS_Excel_95.xcu  |  
  2 +-
 filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu |  
  2 +-
 filter/source/config/fragments/filters/MS_WinWord_6_0.xcu   |  
  2 +-
 filter/source/config/fragments/filters/MS_Word_95.xcu   |  
  2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

New commits:
commit 664d465a4b0f02200748067c0693eb95444174d4
Author: Bryan Quigley b...@bryanquigley.com
Date:   Wed Dec 12 15:34:12 2012 -0500

Removed save to Office 6.0/95 formats

Change-Id: Ifeeb85afbd2531be772e27e10c6283158f0dac09

diff --git a/filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu 
b/filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu
index d7fc944..bd8423b 100644
--- a/filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu
+++ b/filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Excel 5.0/95 oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN PREFERRED/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN PREFERRED/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserData/
diff --git 
a/filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu 
b/filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu
index edfb473..4542d27 100644
--- 
a/filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu
+++ 
b/filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Excel 5.0/95 Vorlage/Template oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN TEMPLATE 
TEMPLATEPATH/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN TEMPLATE 
TEMPLATEPATH/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserData/
diff --git a/filter/source/config/fragments/filters/MS_Excel_95.xcu 
b/filter/source/config/fragments/filters/MS_Excel_95.xcu
index 79d259d..4e227ca 100644
--- a/filter/source/config/fragments/filters/MS_Excel_95.xcu
+++ b/filter/source/config/fragments/filters/MS_Excel_95.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Excel 95 oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN PREFERRED/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN PREFERRED/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserData/
diff --git 
a/filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu 
b/filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu
index 1ef6756..a5eab88 100644
--- a/filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu
+++ b/filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Excel 95 Vorlage/Template oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN TEMPLATE 
TEMPLATEPATH/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN TEMPLATE 
TEMPLATEPATH/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserData/
diff --git a/filter/source/config/fragments/filters/MS_WinWord_6_0.xcu 
b/filter/source/config/fragments/filters/MS_WinWord_6_0.xcu
index 00a3aa4..e784b52 100644
--- a/filter/source/config/fragments/filters/MS_WinWord_6_0.xcu
+++ b/filter/source/config/fragments/filters/MS_WinWord_6_0.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS WinWord 6.0 oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserDatavalueCWW6/value/prop
diff --git a/filter/source/config/fragments/filters/MS_Word_95.xcu 
b/filter/source/config/fragments/filters/MS_Word_95.xcu
index ffea860..356b8fa 100644
--- a/filter/source/config/fragments/filters/MS_Word_95.xcu
+++ b/filter/source/config/fragments/filters/MS_Word_95.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Word 95 oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserDatavalueCWW6/value/prop

[Libreoffice-commits] .: Branch 'libreoffice-4-0' - filter/source

2012-12-13 Thread Libreoffice Gerrit user
 filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu  |  
  2 +-
 filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu |  
  2 +-
 filter/source/config/fragments/filters/MS_Excel_95.xcu  |  
  2 +-
 filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu |  
  2 +-
 filter/source/config/fragments/filters/MS_WinWord_6_0.xcu   |  
  2 +-
 filter/source/config/fragments/filters/MS_Word_95.xcu   |  
  2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

New commits:
commit 279ed1d1ea3e692072b19a510f0cba3d892ca046
Author: Bryan Quigley b...@bryanquigley.com
Date:   Wed Dec 12 15:34:12 2012 -0500

Resolves: fdo#53471 Remove save to Office 6.0/95 formats

(cherry picked from commit 664d465a4b0f02200748067c0693eb95444174d4)

Change-Id: Ifeeb85afbd2531be772e27e10c6283158f0dac09
Signed-off-by: Caolán McNamara caol...@redhat.com

diff --git a/filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu 
b/filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu
index d7fc944..bd8423b 100644
--- a/filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu
+++ b/filter/source/config/fragments/filters/MS_Excel_5_0_95.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Excel 5.0/95 oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN PREFERRED/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN PREFERRED/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserData/
diff --git 
a/filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu 
b/filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu
index edfb473..4542d27 100644
--- 
a/filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu
+++ 
b/filter/source/config/fragments/filters/MS_Excel_5_0_95_Vorlage_Template.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Excel 5.0/95 Vorlage/Template oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN TEMPLATE 
TEMPLATEPATH/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN TEMPLATE 
TEMPLATEPATH/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserData/
diff --git a/filter/source/config/fragments/filters/MS_Excel_95.xcu 
b/filter/source/config/fragments/filters/MS_Excel_95.xcu
index 79d259d..4e227ca 100644
--- a/filter/source/config/fragments/filters/MS_Excel_95.xcu
+++ b/filter/source/config/fragments/filters/MS_Excel_95.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Excel 95 oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN PREFERRED/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN PREFERRED/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserData/
diff --git 
a/filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu 
b/filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu
index 1ef6756..a5eab88 100644
--- a/filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu
+++ b/filter/source/config/fragments/filters/MS_Excel_95_Vorlage_Template.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Excel 95 Vorlage/Template oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN TEMPLATE 
TEMPLATEPATH/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN TEMPLATE 
TEMPLATEPATH/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserData/
diff --git a/filter/source/config/fragments/filters/MS_WinWord_6_0.xcu 
b/filter/source/config/fragments/filters/MS_WinWord_6_0.xcu
index 00a3aa4..e784b52 100644
--- a/filter/source/config/fragments/filters/MS_WinWord_6_0.xcu
+++ b/filter/source/config/fragments/filters/MS_WinWord_6_0.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS WinWord 6.0 oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN/value/prop
 prop oor:name=UIComponent/
 prop oor:name=FilterService/
 prop oor:name=UserDatavalueCWW6/value/prop
diff --git a/filter/source/config/fragments/filters/MS_Word_95.xcu 
b/filter/source/config/fragments/filters/MS_Word_95.xcu
index ffea860..356b8fa 100644
--- a/filter/source/config/fragments/filters/MS_Word_95.xcu
+++ b/filter/source/config/fragments/filters/MS_Word_95.xcu
@@ -16,7 +16,7 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 --
 node oor:name=MS Word 95 oor:op=replace
-prop oor:name=FlagsvalueIMPORT EXPORT ALIEN/value/prop
+prop oor:name=FlagsvalueIMPORT ALIEN/value/prop
  

[PUSHED] Re: Remove save to old Microsoft office formats

2012-12-13 Thread Caolán McNamara
On Wed, 2012-12-12 at 18:05 -0500, Bryan Quigley wrote:
 Hi all,
 
 This patch removes the ability to save to old office 6/95 formats.
 I've also opened a bug on it here:
 https://bugs.freedesktop.org/show_bug.cgi?id=53471
 
 It's a very trivial patch but:
 This contribution to LibreOffice may be licensed under the MPL/LGPLv3+
 dual license.  This patch was also written before I joined Canonical. 

Was discussed in ESC call, so pushed to master and 4.0 now. Could you
update the features page with this info, there should be some existing
entry about the binfilter removal.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Bug 44446] LibreOffice 3.6 most annoying bugs

2012-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=6

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

 Depends on|56068   |

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: helpcontent2

2012-12-13 Thread Libreoffice Gerrit user
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 20379fca091aa36bbd49a1ddc312c4210d383e78
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Dec 13 16:05:59 2012 +

Updated core
Project: help  ca261267f5f67d4932e0046e0358e8d0fb8b4d1b
the .tree files are xml, so need xml comments not c++ ones

diff --git a/helpcontent2 b/helpcontent2
index df4d4db..ca26126 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit df4d4db7408ed3fc135eded6787bad80853619fd
+Subproject commit ca261267f5f67d4932e0046e0358e8d0fb8b4d1b
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: source/auxiliary

2012-12-13 Thread Libreoffice Gerrit user
 source/auxiliary/sbasic.tree   |7 +++
 source/auxiliary/scalc.tree|7 +++
 source/auxiliary/schart.tree   |7 +++
 source/auxiliary/shared.tree   |7 +++
 source/auxiliary/simpress.tree |7 +++
 source/auxiliary/smath.tree|7 +++
 source/auxiliary/swriter.tree  |7 +++
 7 files changed, 21 insertions(+), 28 deletions(-)

New commits:
commit ca261267f5f67d4932e0046e0358e8d0fb8b4d1b
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Dec 13 16:05:59 2012 +

the .tree files are xml, so need xml comments not c++ ones

diff --git a/source/auxiliary/sbasic.tree b/source/auxiliary/sbasic.tree
index b228483..054daad 100644
--- a/source/auxiliary/sbasic.tree
+++ b/source/auxiliary/sbasic.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=sbasic id=07 title=Macros and 
Programming
node id=0701 title=General Information and User Interface 
Usage
diff --git a/source/auxiliary/scalc.tree b/source/auxiliary/scalc.tree
index 0da6d2a..fbaf76d 100644
--- a/source/auxiliary/scalc.tree
+++ b/source/auxiliary/scalc.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=scalc id=08 title=Spreadsheets
node id=0801 title=General Information and User Interface 
Usage
diff --git a/source/auxiliary/schart.tree b/source/auxiliary/schart.tree
index 4efafac..a470b99 100644
--- a/source/auxiliary/schart.tree
+++ b/source/auxiliary/schart.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=scalc id=05 title=Charts and Diagrams
node id=0501 title=General Information
diff --git a/source/auxiliary/shared.tree b/source/auxiliary/shared.tree
index ba7c734..5fc0ca1 100644
--- a/source/auxiliary/shared.tree
+++ b/source/auxiliary/shared.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=swriter id=01 title=Installation
 topic id=shared/text/shared/guide/ms_doctypes.xhpChanging the Association 
of Microsoft Office Document Types/topic
diff --git a/source/auxiliary/simpress.tree b/source/auxiliary/simpress.tree
index 520639c..b27417c 100644
--- a/source/auxiliary/simpress.tree
+++ b/source/auxiliary/simpress.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=simpress id=04 title=Presentations and 
Drawings
node id=0401 title=General Information and User Interface 
Usage
diff --git a/source/auxiliary/smath.tree b/source/auxiliary/smath.tree
index 6694bd8..07e61a0 100644
--- a/source/auxiliary/smath.tree
+++ b/source/auxiliary/smath.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla 

[Libreoffice-commits] .: liborcus/ExternalProject_liborcus.mk liborcus/UnpackedTarball_orcus.mk liborcus/vsprojects

2012-12-13 Thread Libreoffice Gerrit user
 liborcus/ExternalProject_liborcus.mk|3 
 liborcus/UnpackedTarball_orcus.mk   |4 
 liborcus/vsprojects/liborcus-static-nozip/liborcus-static-nozip.vcxproj |  147 
++
 liborcus/vsprojects/liborcus/DefaultConfig.props|   14 
 liborcus/vsprojects/liborcus/zLib.props |   14 
 5 files changed, 180 insertions(+), 2 deletions(-)

New commits:
commit 355cbe5ca531abca784b9df7c002cf4b4feba74c
Author: Tor Lillqvist tlillqv...@suse.com
Date:   Thu Dec 13 18:10:35 2012 +0200

Add VS2010 project, avoids unreliable vcupgrade

diff --git a/liborcus/ExternalProject_liborcus.mk 
b/liborcus/ExternalProject_liborcus.mk
index 0c64d86..9de89da 100644
--- a/liborcus/ExternalProject_liborcus.mk
+++ b/liborcus/ExternalProject_liborcus.mk
@@ -33,11 +33,10 @@ $(call gb_ExternalProject_get_state_target,liborcus,build) :
 export BOOST_INCLUDE_DIR=$(OUTDIR)/inc/external \
 export ZLIB_INCLUDE_DIR=$(OUTDIR)/inc/external/zlib \
 export BOOST_LIB_DIR=$(OUTDIR)/lib \
-$(COMPATH)/../Common7/Tools/vcupgrade.exe 
liborcus-static-nozip.vcproj \
 MSBuild.exe liborcus-static-nozip.vcxproj /p:Configuration=Release 
/p:OutDir=Release/ /p:TargetName=orcus /p:WholeProgramOptimization=no \
 touch $@
 endif
-   
+
 else
 
 # must be built with debug STL if --enable-dbgutil
diff --git a/liborcus/UnpackedTarball_orcus.mk 
b/liborcus/UnpackedTarball_orcus.mk
index 3375c95..ca5232d 100644
--- a/liborcus/UnpackedTarball_orcus.mk
+++ b/liborcus/UnpackedTarball_orcus.mk
@@ -27,4 +27,8 @@ $(eval $(call gb_UnpackedTarball_add_patches,orcus,\
$(foreach patch,$(orcus_patches),liborcus/$(patch)) \
 ))
 
+$(eval $(call 
gb_UnpackedTarball_add_file,orcus,vsprojects/liborcus-static-nozip/liborcus-static-nozip.vcxproj,liborcus/vsprojects/liborcus-static-nozip/liborcus-static-nozip.vcxproj))
+$(eval $(call 
gb_UnpackedTarball_add_file,orcus,vsprojects/liborcus/DefaultConfig.props,liborcus/vsprojects/liborcus/DefaultConfig.props))
+$(eval $(call 
gb_UnpackedTarball_add_file,orcus,vsprojects/liborcus/zLib.props,liborcus/vsprojects/liborcus/zLib.props))
+
 # vim: set noet sw=4 ts=4:
diff --git 
a/liborcus/vsprojects/liborcus-static-nozip/liborcus-static-nozip.vcxproj 
b/liborcus/vsprojects/liborcus-static-nozip/liborcus-static-nozip.vcxproj
new file mode 100644
index 000..c63af42
--- /dev/null
+++ b/liborcus/vsprojects/liborcus-static-nozip/liborcus-static-nozip.vcxproj
@@ -0,0 +1,147 @@
+?xml version=1.0 encoding=utf-8?
+Project DefaultTargets=Build ToolsVersion=4.0 
xmlns=http://schemas.microsoft.com/developer/msbuild/2003;
+  ItemGroup Label=ProjectConfigurations
+ProjectConfiguration Include=Debug|Win32
+  ConfigurationDebug/Configuration
+  PlatformWin32/Platform
+/ProjectConfiguration
+ProjectConfiguration Include=Release|Win32
+  ConfigurationRelease/Configuration
+  PlatformWin32/Platform
+/ProjectConfiguration
+  /ItemGroup
+  PropertyGroup Label=Globals
+ProjectGuid{50DD1B15-9FEB-4037-9FBF-D52A711977E7}/ProjectGuid
+RootNamespaceliborcusnozip/RootNamespace
+  /PropertyGroup
+  Import Project=$(VCTargetsPath)\Microsoft.Cpp.Default.props /
+  PropertyGroup Condition='$(Configuration)|$(Platform)'=='Release|Win32' 
Label=Configuration
+ConfigurationTypeStaticLibrary/ConfigurationType
+CharacterSetMultiByte/CharacterSet
+WholeProgramOptimizationtrue/WholeProgramOptimization
+  /PropertyGroup
+  PropertyGroup Condition='$(Configuration)|$(Platform)'=='Debug|Win32' 
Label=Configuration
+ConfigurationTypeStaticLibrary/ConfigurationType
+CharacterSetMultiByte/CharacterSet
+  /PropertyGroup
+  Import Project=$(VCTargetsPath)\Microsoft.Cpp.props /
+  ImportGroup Label=ExtensionSettings
+  /ImportGroup
+  ImportGroup Condition='$(Configuration)|$(Platform)'=='Release|Win32' 
Label=PropertySheets
+Import Project=$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props 
Condition=exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props') 
Label=LocalAppDataPlatform /
+Import Project=..\liborcus\DefaultConfig.props /
+Import Project=..\liborcus\zLib.props /
+  /ImportGroup
+  ImportGroup Condition='$(Configuration)|$(Platform)'=='Debug|Win32' 
Label=PropertySheets
+Import Project=$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props 
Condition=exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props') 
Label=LocalAppDataPlatform /
+Import Project=..\liborcus\DefaultConfig.props /
+Import Project=..\liborcus\zLib.props /
+  /ImportGroup
+  PropertyGroup Label=UserMacros /
+  PropertyGroup
+_ProjectFileVersion10.0.40219.1/_ProjectFileVersion
+OutDir 
Condition='$(Configuration)|$(Platform)'=='Debug|Win32'$(SolutionDir)$(Configuration)\/OutDir
+IntDir 
Condition='$(Configuration)|$(Platform)'=='Debug|Win32'$(Configuration)\/IntDir
+OutDir 

Regression Key word by BSA

2012-12-13 Thread Rainer Bielefeld

Hi all,

can you please leave some brief statements in Bug 58251 - 
BUGZILLAASSISTANT: Inappropriate regression keyword

 https://www.libreoffice.org/bugzilla/show_bug.cgi?id=58251
as base for a decision?

Discussion if necessary please on libreoffice...@lists.freedesktop.org!

Thank you and best regards

Rainer Bielefeld
(QA-Volunteer)
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: sw/source

2012-12-13 Thread Libreoffice Gerrit user
 sw/source/core/text/portxt.cxx |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit d3a9e97164c0071d8b18f8dcf4197ec7c5c5c2f9
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Dec 13 16:35:06 2012 +

Resolves: fdo#32181 #i68503# don't emit pesky space unless in tagged pdf 
mode

which means that they are not there to be underlined

Change-Id: Ic02372c5284384e18198ebea44c36e83ccb6bf2a

diff --git a/sw/source/core/text/portxt.cxx b/sw/source/core/text/portxt.cxx
index 617cf39..d2140b6 100644
--- a/sw/source/core/text/portxt.cxx
+++ b/sw/source/core/text/portxt.cxx
@@ -22,6 +22,7 @@
 #include com/sun/star/i18n/ScriptType.hpp
 #include i18npool/mslangid.hxx
 #include hintids.hxx // CH_TXTATR
+#include EnhancedPDFExportHelper.hxx
 #include SwPortionHandler.hxx
 #include porlay.hxx
 #include inftxt.hxx
@@ -717,7 +718,8 @@ SwLinePortion *SwHolePortion::Compress() { return this; }
 void SwHolePortion::Paint( const SwTxtPaintInfo rInf ) const
 {
 // #i16816# tagged pdf support
-if( rInf.GetVsh()  rInf.GetVsh()-GetViewOptions()-IsPDFExport() )
+if( rInf.GetVsh()  rInf.GetVsh()-GetViewOptions()-IsPDFExport() 
+SwTaggedPDFHelper::IsExportTaggedPDF( *rInf.GetOut()) )
 {
 const OUString aTxt( ' ' );
 rInf.DrawText( aTxt, *this, 0, 1, false );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 37361] LibreOffice 3.5 most annoying bugs

2012-12-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=37361

Bug 37361 depends on bug 32181, which changed state.

Bug 32181 Summary: PRINTING: underlining in Justified aligned paragraph bleed 
over margin, also PDFEXPORT
https://bugs.freedesktop.org/show_bug.cgi?id=32181

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] Change in core[libreoffice-4-0]: Resolves: fdo#32181 #i68503# don't emit pesky space unless i...

2012-12-13 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1327

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/27/1327/1

Resolves: fdo#32181 #i68503# don't emit pesky space unless in tagged pdf mode

which means that they are not there to be underlined

Change-Id: Ic02372c5284384e18198ebea44c36e83ccb6bf2a
(cherry picked from commit d3a9e97164c0071d8b18f8dcf4197ec7c5c5c2f9)
---
M sw/source/core/text/portxt.cxx
1 file changed, 3 insertions(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic02372c5284384e18198ebea44c36e83ccb6bf2a
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Caolán McNamara caol...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] Change in core[libreoffice-3-6]: Resolves: fdo#32181 #i68503# don't emit pesky space unless i...

2012-12-13 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1328

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/28/1328/1

Resolves: fdo#32181 #i68503# don't emit pesky space unless in tagged pdf mode

which means that they are not there to be underlined

Change-Id: Ic02372c5284384e18198ebea44c36e83ccb6bf2a
(cherry picked from commit d3a9e97164c0071d8b18f8dcf4197ec7c5c5c2f9)
---
M sw/source/core/text/portxt.cxx
1 file changed, 3 insertions(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic02372c5284384e18198ebea44c36e83ccb6bf2a
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Caolán McNamara caol...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] Change in core[libreoffice-3-5]: Resolves: fdo#32181 #i68503# don't emit pesky space unless i...

2012-12-13 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1329

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/29/1329/1

Resolves: fdo#32181 #i68503# don't emit pesky space unless in tagged pdf mode

which means that they are not there to be underlined

Change-Id: Ic02372c5284384e18198ebea44c36e83ccb6bf2a
(cherry picked from commit d3a9e97164c0071d8b18f8dcf4197ec7c5c5c2f9)
---
M sw/source/core/text/portxt.cxx
1 file changed, 3 insertions(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic02372c5284384e18198ebea44c36e83ccb6bf2a
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-5
Gerrit-Owner: Caolán McNamara caol...@redhat.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Build Problem

2012-12-13 Thread Akshay Ratan
Hi,
I am a newbie in this LibreOffice Community and wanted to start
contributing to it. However, when I am trying to build LibreOffice
initially, I am facing one small error which is hindering my build. I know
it might be too small for you experienced developers. Suggestions are
welcome for this..

*On running ./autogen.sh on my ubuntu 11.10 oneiric terminal,following
error is reported,
.
.
.

BEGIN failed--compilation aborted at -e line 1.
configure: error: Failed to find some modules
Error running configure at ./autogen.sh line 201.

*
Cheers,
Akshay
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: 2 commits - oox/inc oox/source sc/source

2012-12-13 Thread Libreoffice Gerrit user
 oox/inc/oox/core/relations.hxx|2 ++
 oox/source/core/relations.cxx |6 ++
 oox/source/drawingml/hyperlinkcontext.cxx |4 +++-
 sc/source/filter/inc/worksheetbuffer.hxx  |4 
 sc/source/filter/oox/drawingfragment.cxx  |   22 --
 sc/source/filter/oox/workbookhelper.cxx   |4 ++--
 sc/source/filter/oox/worksheetbuffer.cxx  |   19 +++
 7 files changed, 56 insertions(+), 5 deletions(-)

New commits:
commit 70b960b966d36b1cb12bfb0c71d093fa0c5a3288
Author: Noel Power noel.po...@suse.com
Date:   Thu Dec 13 16:52:50 2012 +

fix sometimes data corruption with xlsx import ( with scenario sheets )

formula import is buffered 'till the end of import. The processing of the
formula data need to happen before the scenario import happens. This is
necessary because sheet numbers stored in the formula addresses can become
invalid as scenario import insertes new hidden sheets upsetting the previous
table order

Change-Id: I9357f028f31bec1b1504ca991f5534f80d79c9bc

diff --git a/sc/source/filter/oox/workbookhelper.cxx 
b/sc/source/filter/oox/workbookhelper.cxx
index a9201ee..e5521e7 100644
--- a/sc/source/filter/oox/workbookhelper.cxx
+++ b/sc/source/filter/oox/workbookhelper.cxx
@@ -597,8 +597,6 @@ void WorkbookGlobals::finalize()
 // #i79826# enable updating automatic row height after loading the 
document
 aPropSet.setProperty( PROP_IsAdjustHeightEnabled, true );
 
-getFormulaBuffer().finalizeImport();
-
 // Insert all pivot tables. Must be done after loading all sheets and
 // formulas, because data pilots expect existing source data on
 // creation.
@@ -674,6 +672,8 @@ void WorkbookHelper::finalizeWorkbookImport()
 mrBookGlob.getWorkbookSettings().finalizeImport();
 mrBookGlob.getViewSettings().finalizeImport();
 
+// need to import formulas before scenarios
+mrBookGlob.getFormulaBuffer().finalizeImport();
 /*  Insert scenarios after all sheet processing is done, because new hidden
 sheets are created for scenarios which would confuse code that relies
 on certain sheet indexes. Must be done after pivot tables too. */
commit 2551fec6d2a73d10cfa2aec85ae7d00e237e0294
Author: Noel Power noel.po...@suse.com
Date:   Thu Dec 13 16:26:58 2012 +

fix fdo#58237 import hyperlinks for shapes in xlsx documents

Change-Id: Ib0c661dbb3ce9a2f8c8d29707a1cf0c65aadc81f

diff --git a/oox/inc/oox/core/relations.hxx b/oox/inc/oox/core/relations.hxx
index f840448..94b9908 100644
--- a/oox/inc/oox/core/relations.hxx
+++ b/oox/inc/oox/core/relations.hxx
@@ -79,6 +79,8 @@ public:
 
 /** Returns the external target of the relation with the passed relation 
identifier. */
 ::rtl::OUString getExternalTargetFromRelId( const ::rtl::OUString 
rRelId ) const;
+/** Returns the internal target of the relation with the passed relation 
identifier. */
+::rtl::OUString getInternalTargetFromRelId( const ::rtl::OUString 
rRelId ) const;
 
 /** Returns the full fragment path for the target of the passed relation. 
*/
 ::rtl::OUString getFragmentPathFromRelation( const Relation rRelation 
) const;
diff --git a/oox/source/core/relations.cxx b/oox/source/core/relations.cxx
index 501f515..c49fe63 100644
--- a/oox/source/core/relations.cxx
+++ b/oox/source/core/relations.cxx
@@ -83,6 +83,12 @@ OUString Relations::getExternalTargetFromRelId( const 
OUString rRelId ) const
 return (pRelation  pRelation-mbExternal) ? pRelation-maTarget : 
OUString();
 }
 
+OUString Relations::getInternalTargetFromRelId( const OUString rRelId ) const
+{
+const Relation* pRelation = getRelationFromRelId( rRelId );
+return (pRelation  !pRelation-mbExternal) ? pRelation-maTarget : 
OUString();
+}
+
 OUString Relations::getFragmentPathFromRelation( const Relation rRelation ) 
const
 {
 // no target, no fragment path
diff --git a/oox/source/drawingml/hyperlinkcontext.cxx 
b/oox/source/drawingml/hyperlinkcontext.cxx
index f2d715c..231b7f9 100644
--- a/oox/source/drawingml/hyperlinkcontext.cxx
+++ b/oox/source/drawingml/hyperlinkcontext.cxx
@@ -44,10 +44,12 @@ HyperLinkContext::HyperLinkContext( ContextHandler rParent,
 {
 OSL_TRACE(OOX: URI rId %s, OUStringToOString (aRelId, 
RTL_TEXTENCODING_UTF8).pData-buffer);
 sHref = getRelations().getExternalTargetFromRelId( aRelId );
+OUString sExtHref = getRelations().getExternalTargetFromRelId( aRelId 
);
+sURL = getRelations().getInternalTargetFromRelId( aRelId );
 if( !sHref.isEmpty() )
 {
 OSL_TRACE(OOX: URI href %s, OUStringToOString (sHref, 
RTL_TEXTENCODING_UTF8).pData-buffer);
-sURL = getFilter().getAbsoluteUrl( sHref );
+sURL = getFilter().getAbsoluteUrl( sExtHref );
 }
 }
 OUString sTooltip = xAttributes-getOptionalValue( R_TOKEN( tooltip ) );
diff --git 

Re: git-new-workdir

2012-12-13 Thread Kohei Yoshida

On 12/13/2012 10:21 AM, Lionel Elie Mamane wrote:

On Wed, Dec 12, 2012 at 02:21:18PM -0500, Kohei Yoshida wrote:


Not following the thread fully I'm not sure what this is about.  I'm
guessing this is about git-new-workdir vs submodules.



To put it short, I got it to work, and so far I haven't seen any
major issues but some occasional oddities which I just dismiss for
now.


What version of git do you use? I haven't gotten it to work on the
submodules, actually. So I duplicate the submodules currently.


I use git 1.8.0.  I did upgrade my git specifically for submodules.



I'd appreciate a description of how did it, actually.


Well, there is not much to it, actually.  Here is what I have/did.

I have this primary master branch

~/libo/master

and in it, submodules work more or less as expected.  It automatically 
checks out required submodules which for me is just helpcontent2.


Now, to create a new workdir for, say, libreoffice-4-0 branch, I run

cd ~/libo
git-new-workdir master libreoffice-4-0 libreoffice-4-0

I tend to name the directory the same name as the branch name. Then

cd libreoffice-4-0
ln -s ../master/src
git-new-workdir ../master/helpconent2 helpcontent2 libreoffice-4-0

to checkout helpcontent2.

That's all.

Now, I do the same with my own unpublished local branch, in which case I 
don't normally bother to create a branch for it in helpcontent2 (or any 
other submodules) due to laziness.


I would think that you probably should create a branch for the 
submodules as well to get it to work, but I skip that step for now since 
not creating a branch in the submodules doesn't seem to create major 
pain for me.  Plus I don't really push anything to any of the submodules.


Now, I didn't really want to describe my settings because I don't really 
know if this is really not causing any issues.  And unlike my previous 
settings, there is no automatic script to set up a new workdir.  But it 
seems to just work fine for me.



To me git-new-workdir is not about diskspace, but about avoiding the
hassle of managing multiple local repos.  Keeping it in one local
repo make it easier to manage especially when I tend to cherry-pick
between branches quite a lot.


Exactly. In its absence, you have to constantly push/pull from one
local repo to the other, and that's a hassle.


Yup, and I prefer not having to do it.

Kohei

--
Kohei Yoshida, LibreOffice hacker, Calc
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: oox/inc oox/source sc/source

2012-12-13 Thread Libreoffice Gerrit user
 oox/inc/oox/core/relations.hxx|2 --
 oox/source/core/relations.cxx |6 --
 oox/source/drawingml/hyperlinkcontext.cxx |4 +---
 sc/source/filter/inc/worksheetbuffer.hxx  |4 
 sc/source/filter/oox/drawingfragment.cxx  |   22 ++
 sc/source/filter/oox/worksheetbuffer.cxx  |   19 ---
 6 files changed, 3 insertions(+), 54 deletions(-)

New commits:
commit 45b448b202b87ef66812d25c69e308f405ac221e
Author: Noel Power noel.po...@suse.com
Date:   Thu Dec 13 17:04:58 2012 +

Revert fix fdo#58237 import hyperlinks for shapes in xlsx documents

This reverts commit 2551fec6d2a73d10cfa2aec85ae7d00e237e0294.

diff --git a/oox/inc/oox/core/relations.hxx b/oox/inc/oox/core/relations.hxx
index 94b9908..f840448 100644
--- a/oox/inc/oox/core/relations.hxx
+++ b/oox/inc/oox/core/relations.hxx
@@ -79,8 +79,6 @@ public:
 
 /** Returns the external target of the relation with the passed relation 
identifier. */
 ::rtl::OUString getExternalTargetFromRelId( const ::rtl::OUString 
rRelId ) const;
-/** Returns the internal target of the relation with the passed relation 
identifier. */
-::rtl::OUString getInternalTargetFromRelId( const ::rtl::OUString 
rRelId ) const;
 
 /** Returns the full fragment path for the target of the passed relation. 
*/
 ::rtl::OUString getFragmentPathFromRelation( const Relation rRelation 
) const;
diff --git a/oox/source/core/relations.cxx b/oox/source/core/relations.cxx
index c49fe63..501f515 100644
--- a/oox/source/core/relations.cxx
+++ b/oox/source/core/relations.cxx
@@ -83,12 +83,6 @@ OUString Relations::getExternalTargetFromRelId( const 
OUString rRelId ) const
 return (pRelation  pRelation-mbExternal) ? pRelation-maTarget : 
OUString();
 }
 
-OUString Relations::getInternalTargetFromRelId( const OUString rRelId ) const
-{
-const Relation* pRelation = getRelationFromRelId( rRelId );
-return (pRelation  !pRelation-mbExternal) ? pRelation-maTarget : 
OUString();
-}
-
 OUString Relations::getFragmentPathFromRelation( const Relation rRelation ) 
const
 {
 // no target, no fragment path
diff --git a/oox/source/drawingml/hyperlinkcontext.cxx 
b/oox/source/drawingml/hyperlinkcontext.cxx
index 231b7f9..f2d715c 100644
--- a/oox/source/drawingml/hyperlinkcontext.cxx
+++ b/oox/source/drawingml/hyperlinkcontext.cxx
@@ -44,12 +44,10 @@ HyperLinkContext::HyperLinkContext( ContextHandler rParent,
 {
 OSL_TRACE(OOX: URI rId %s, OUStringToOString (aRelId, 
RTL_TEXTENCODING_UTF8).pData-buffer);
 sHref = getRelations().getExternalTargetFromRelId( aRelId );
-OUString sExtHref = getRelations().getExternalTargetFromRelId( aRelId 
);
-sURL = getRelations().getInternalTargetFromRelId( aRelId );
 if( !sHref.isEmpty() )
 {
 OSL_TRACE(OOX: URI href %s, OUStringToOString (sHref, 
RTL_TEXTENCODING_UTF8).pData-buffer);
-sURL = getFilter().getAbsoluteUrl( sExtHref );
+sURL = getFilter().getAbsoluteUrl( sHref );
 }
 }
 OUString sTooltip = xAttributes-getOptionalValue( R_TOKEN( tooltip ) );
diff --git a/sc/source/filter/inc/worksheetbuffer.hxx 
b/sc/source/filter/inc/worksheetbuffer.hxx
index d58a490..bceca55 100644
--- a/sc/source/filter/inc/worksheetbuffer.hxx
+++ b/sc/source/filter/inc/worksheetbuffer.hxx
@@ -81,10 +81,6 @@ public:
 sal_Int16   getCalcSheetIndex( const ::rtl::OUString 
rWorksheetName ) const;
 /** Returns the finalized name of the sheet with the passed worksheet 
name. */
 ::rtl::OUString getCalcSheetName( const ::rtl::OUString 
rWorksheetName ) const;
-/** Converts sSheetNameRef (e.g. '#SheetName!A1' to '#SheetName.A1' )
-if sSheetNameRef doesn't start with '#' it is ignored and not modified
-*/
-voidconvertSheetNameRef( ::rtl::OUString sSheetNameRef ) 
const;
 
 private:
 struct SheetInfo : public SheetInfoModel
diff --git a/sc/source/filter/oox/drawingfragment.cxx 
b/sc/source/filter/oox/drawingfragment.cxx
index a301326..bb8fe08 100644
--- a/sc/source/filter/oox/drawingfragment.cxx
+++ b/sc/source/filter/oox/drawingfragment.cxx
@@ -26,9 +26,6 @@
 #include com/sun/star/script/ScriptEventDescriptor.hpp
 #include com/sun/star/script/XEventAttacherManager.hpp
 #include rtl/strbuf.hxx
-#include svx/svdobj.hxx
-#include drwlayer.hxx
-#include userdat.hxx
 #include oox/drawingml/connectorshapecontext.hxx
 #include oox/drawingml/graphicshapecontext.hxx
 #include oox/helper/attributelist.hxx
@@ -39,10 +36,11 @@
 #include stylesbuffer.hxx
 #include themebuffer.hxx
 #include unitconverter.hxx
-#include worksheetbuffer.hxx
+
 namespace oox {
 namespace xls {
 
+using namespace ::com::sun::star::awt;
 using namespace ::com::sun::star::beans;
 using namespace ::com::sun::star::container;
 using namespace ::com::sun::star::document;
@@ -55,10 +53,6 @@ using namespace ::oox::core;
 using namespace 

[Libreoffice-commits] .: libvisio/ExternalProject_libvisio.mk

2012-12-13 Thread Libreoffice Gerrit user
 libvisio/ExternalProject_libvisio.mk |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 64f98aa1f492911c6ce1b76c394bd87abee330a2
Author: Tor Lillqvist tlillqv...@suse.com
Date:   Thu Dec 13 19:06:05 2012 +0200

/p:VisualStudioVersion=11.0

diff --git a/libvisio/ExternalProject_libvisio.mk 
b/libvisio/ExternalProject_libvisio.mk
index 191a712..3ad16c5 100644
--- a/libvisio/ExternalProject_libvisio.mk
+++ b/libvisio/ExternalProject_libvisio.mk
@@ -48,7 +48,7 @@ $(call gb_ExternalProject_get_state_target,libvisio,build) :
 export LIBWPG_INCLUDE_DIR=$(OUTDIR)/inc/external \
 export LIBXML_INCLUDE_DIR=$(OUTDIR)/inc/external \
 export ZLIB_INCLUDE_DIR=$(OUTDIR)/inc/external/zlib \
-msbuild.exe libvisio.vcxproj /p:PlatformToolset=v110 
/p:Configuration=Release \
+msbuild.exe libvisio.vcxproj /p:PlatformToolset=v110 
/p:VisualStudioVersion=11.0 /p:Configuration=Release \
 touch $@
 endif
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


minutes of ESC call ...

2012-12-13 Thread Michael Meeks
* Present:
+ Stephan, Petr, Bjoern, Caolan, Kohei, Michael M, Eike, Joel,
  Lionel, Michael S, Thorsten

* Completed Action Items
+ add completed technical 4.0 changes to features wiki page (Michael M)
+ grok the list of contributors for suitable certified hackers (Kendy)
+ have a list of people - bugs to process.
+ come up with a proposal for UNO breakage / ns timestamps (Stephan, 
Michael S, Lionel)
+ punt to next time
+ setup LibreOffice 4.1 schedule in the wiki (Petr)
+ check stlport situation with DudenCorrector (Thorsten)
+ thorough testing with Beta1  stlport removed planned
+ should get rid of stlport on all platforms in anticipation

* Pending Action Items
+ create a new AmbitiousHacks wiki page, based on GSOC page (Michael M)
+ minimal triage for good mentors for proposed easy hacks (Bjoern)
+ work on mail to encourage them to get certified (Kendy/Stephan/Bjoern)
+ [ in progess ]
+ disable Rhino / Beanshell unless in experimental mode (Michael)
+ tweak the configure defaults (Kendy)
+ no submodules by default, no gnome-vfs
+ persuade Thorsten of final MSI / pkg naming / resting place (Kendy)
+ update the 4.0 release notes wiki page to explain DocumentInfo 
(Michael S)

http://wiki.documentfoundation.org/ReleaseNotes/4.0#Extensions_.2F_API_changes
+ kill legacy stlport packaging (Stephan)

* Release Engineering update (Petr)
+ 4.0 timeline:
+ Monday is the deadline for Beta2
+ ideally the last deadline for delayed features
+ still no double etc. review needed until we get to RC1
+ 3.6.4 retrospective
+ a good release, needed an rc3 only down to a minor slip
+ please don't forget to cherry-pick fixes from master - 4.0 - 3.6
+ 3.6.5 - in January 14th
+ completing generic / Linux builds - RedHat (Caolan)
AI: + mail details on signing etc. to Caolan (Fridrich)
+ fdo#58165 - drop stdlibs foo

* UX input (Astron)
+ testing the Android remote
+ issues getting it connected, Android 2.3 + openSUSE
AI: + interactive debugging on IRC (Thorsten)
+ question from Mirek - someone to look into
+ https://bugs.freedesktop.org/show_bug.cgi?id=46517
+ full-word selection / editing ergonomics cf. UX-advise
+ template manager design concerns (Cedric)
+ ongoing discussion for 4.0
+ old design file-templates-organise, new-templates  
documents
+ new template manager replaces the latter, needs work
  to replace the 'organise' function; seems similar but
  really isn't.
+ missing fn is importing styles into documents.
+ at least the common UI is much improved.

* un-publishing DataPilot APIs (Kohei)
+ struggled to remove bits from types.rdb
+ done - thanks to Michael S.

* gnome-vfs / dependencies issues (Bjoern)
+ if adding libbonobo is the solution then we have the wrong problem
+ causing crash-on-starts etc. - bin it for gio ...
+ we need to get rid of gnome-vfs in our downloadable build
+ no gio in RHEL5 base-line ...
+ punt till next time - Fridrich input required.
+ could potentially build with newer glib
+ new dbus menu bits conditional on gio (Bjoern)

* bibisect build / brokenness (Bjoern)
+ 262 builds in a single bibisect repo 4GB small ...
+ poor download speed: 100k/sec (Joel)
+ bibisect could be improved to avoid compiling the center
  of back-to-back checkouts
+ some 1/4 of random checkouts don't build successfully

* Disabling Office 95 export filters (Brian Q)
+ disable word 6 export ? (Caolan)
+ why are people even selecting that ?
+ decided: disable it for now, leave code in in case of
  screaming for a release or so (agreed)

* Certification Committee update (Kendy/Stephan/Bjoern)
+ ongoing work as above.

* Features - please update:
+ http://wiki.documentfoundation.org/ReleaseNotes/4.0
+ with screenshots / feature details ...
AI: + add impress remote shots (Thorsten)

* 4.0 completed tasks:
+ un-publishing of data-pilot interfaces (Kohei)
+ liblangtag - fixed wrapper to not require loading until started (Eike)

* 4.0 pending tasks
+ binfilter:
+ warn on legacy file-types with helpful dialog (needs owner)
+ drop Mozilla mess in the tree:
+ https://bugs.freedesktop.org/show_bug.cgi?id=56902
+ lots of cleanup in configure
+ remove 

[Libreoffice-commits] .: Branch 'libreoffice-4-0' - offapi/com offapi/type_reference

2012-12-13 Thread Libreoffice Gerrit user
 offapi/com/sun/star/sheet/DataPilotSource.idl|2 +-
 offapi/com/sun/star/sheet/DataPilotSourceDimension.idl   |2 +-
 offapi/com/sun/star/sheet/DataPilotSourceDimensions.idl  |2 +-
 offapi/com/sun/star/sheet/DataPilotSourceHierarchies.idl |2 +-
 offapi/com/sun/star/sheet/DataPilotSourceHierarchy.idl   |2 +-
 offapi/com/sun/star/sheet/DataPilotSourceLevel.idl   |2 +-
 offapi/com/sun/star/sheet/DataPilotSourceLevels.idl  |2 +-
 offapi/com/sun/star/sheet/DataPilotSourceMember.idl  |2 +-
 offapi/com/sun/star/sheet/DataPilotSourceMembers.idl |2 +-
 offapi/com/sun/star/sheet/XDataPilotMemberResults.idl|2 +-
 offapi/com/sun/star/sheet/XDataPilotResults.idl  |2 +-
 offapi/com/sun/star/sheet/XDimensionsSupplier.idl|2 +-
 offapi/com/sun/star/sheet/XHierarchiesSupplier.idl   |2 +-
 offapi/com/sun/star/sheet/XLevelsSupplier.idl|2 +-
 offapi/com/sun/star/sheet/XMembersSupplier.idl   |2 +-
 offapi/type_reference/types.rdb  |binary
 16 files changed, 15 insertions(+), 15 deletions(-)

New commits:
commit c517b205ea4a9d73be54ef1d3ccb1cbcd9c4c60b
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Dec 12 20:27:34 2012 +0100

API CHANGE: unpublish Data Pilot interfaces

SERVICE: /UCR/com/sun/star/sheet/DataPilotSource
published in 1 but unpublished in 2
SERVICE: /UCR/com/sun/star/sheet/DataPilotSourceDimension
published in 1 but unpublished in 2
SERVICE: /UCR/com/sun/star/sheet/DataPilotSourceDimensions
published in 1 but unpublished in 2
SERVICE: /UCR/com/sun/star/sheet/DataPilotSourceHierarchies
published in 1 but unpublished in 2
SERVICE: /UCR/com/sun/star/sheet/DataPilotSourceHierarchy
published in 1 but unpublished in 2
SERVICE: /UCR/com/sun/star/sheet/DataPilotSourceLevel
published in 1 but unpublished in 2
SERVICE: /UCR/com/sun/star/sheet/DataPilotSourceLevels
published in 1 but unpublished in 2
SERVICE: /UCR/com/sun/star/sheet/DataPilotSourceMember
published in 1 but unpublished in 2
SERVICE: /UCR/com/sun/star/sheet/DataPilotSourceMembers
published in 1 but unpublished in 2
INTERFACE: /UCR/com/sun/star/sheet/XDataPilotMemberResults
published in 1 but unpublished in 2
INTERFACE: /UCR/com/sun/star/sheet/XDataPilotResults
published in 1 but unpublished in 2
INTERFACE: /UCR/com/sun/star/sheet/XDimensionsSupplier
published in 1 but unpublished in 2
INTERFACE: /UCR/com/sun/star/sheet/XHierarchiesSupplier
published in 1 but unpublished in 2
INTERFACE: /UCR/com/sun/star/sheet/XLevelsSupplier
published in 1 but unpublished in 2
INTERFACE: /UCR/com/sun/star/sheet/XMembersSupplier
published in 1 but unpublished in 2

(cherry picked from commit 0315ef5bbc168896b1d7d8ac5c1657f861fa6a93)
Conflicts:
offapi/type_reference/types.rdb

Change-Id: Ic7547e48f6b00409ed9fe5bf7c3e6921529c9598

diff --git a/offapi/com/sun/star/sheet/DataPilotSource.idl 
b/offapi/com/sun/star/sheet/DataPilotSource.idl
index 0aabeff..920c17d 100644
--- a/offapi/com/sun/star/sheet/DataPilotSource.idl
+++ b/offapi/com/sun/star/sheet/DataPilotSource.idl
@@ -34,7 +34,7 @@ module com {  module sun {  module star {  module sheet {
 pA component that implements this service can be used as data source
 for a data pilot table in a spreadsheet document./p
  */
-published service DataPilotSource
+service DataPilotSource
 {
 
 /** provides access to the collection of the dimensions of the data
diff --git a/offapi/com/sun/star/sheet/DataPilotSourceDimension.idl 
b/offapi/com/sun/star/sheet/DataPilotSourceDimension.idl
index e2c846b..f26c421 100644
--- a/offapi/com/sun/star/sheet/DataPilotSourceDimension.idl
+++ b/offapi/com/sun/star/sheet/DataPilotSourceDimension.idl
@@ -49,7 +49,7 @@
 @see com::sun::star::sheet::DataPilotSource
 @see com::sun::star::sheet::DataPilotTable
  */
-published service DataPilotSourceDimension
+service DataPilotSourceDimension
 {
 
 /** provides access to the name of the dimension, i.e. used in
diff --git a/offapi/com/sun/star/sheet/DataPilotSourceDimensions.idl 
b/offapi/com/sun/star/sheet/DataPilotSourceDimensions.idl
index 3d634d3..f9fc475 100644
--- a/offapi/com/sun/star/sheet/DataPilotSourceDimensions.idl
+++ b/offapi/com/sun/star/sheet/DataPilotSourceDimensions.idl
@@ -30,7 +30,7 @@ module com {  module sun {  module star {  module sheet {
 
 @see com::sun::star::sheet::DataPilotSource
  */
-published service DataPilotSourceDimensions
+service DataPilotSourceDimensions
 {
 
 /** provides access by name to the dimensions in the collection.
diff --git a/offapi/com/sun/star/sheet/DataPilotSourceHierarchies.idl 
b/offapi/com/sun/star/sheet/DataPilotSourceHierarchies.idl
index 1fbd6e6..9e2701a 100644
--- 

[Libreoffice-commits] .: Branch 'distro/suse/suse-3.6' - oox/source

2012-12-13 Thread Libreoffice Gerrit user
 oox/source/vml/vmlshape.cxx |8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

New commits:
commit 04b32936c55c1fe543f86ac077d75dda7e2fb49b
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Fri Oct 5 15:30:11 2012 +0200

vml import: only apply width-percent attribute if it's != 0

This fixes an issue with a shape defined with these attributes:
mso-width-percent:0;mso-height-percent:0 and
mso-width-relative:page;mso-height-relative:page;
where all points were then located in (0,0)

Change-Id: I51070ad2b2e4e05ab64f16813472ca1d7099fb09
Reviewed-on: https://gerrit.libreoffice.org/775
Reviewed-by: Petr Mladek pmla...@suse.cz
Tested-by: Petr Mladek pmla...@suse.cz

diff --git a/oox/source/vml/vmlshape.cxx b/oox/source/vml/vmlshape.cxx
index 1e9573e..fde972c 100644
--- a/oox/source/vml/vmlshape.cxx
+++ b/oox/source/vml/vmlshape.cxx
@@ -464,7 +464,9 @@ Reference XShape  SimpleShape::implConvertAndInsert( 
const Reference XShapes
 if ( maTypeModel.maWidthRelative.isEmpty() || 
maTypeModel.maWidthRelative.equalsAscii( page ) )
 {
 sal_Int16 nWidth = maTypeModel.maWidthPercent.toInt32() / 10;
-PropertySet( xShape ).setAnyProperty(PROP_RelativeWidth, 
makeAny( nWidth ) );
+// Only apply if nWidth != 0
+if ( nWidth )
+PropertySet( xShape ).setAnyProperty(PROP_RelativeWidth, 
makeAny( nWidth ) );
 }
 }
 if ( !maTypeModel.maHeightPercent.isEmpty( ) )
@@ -473,7 +475,9 @@ Reference XShape  SimpleShape::implConvertAndInsert( 
const Reference XShapes
 if ( maTypeModel.maHeightRelative.isEmpty() || 
maTypeModel.maHeightRelative.equalsAscii( page ) )
 {
 sal_Int16 nHeight = maTypeModel.maHeightPercent.toInt32() / 10;
-PropertySet( xShape ).setAnyProperty(PROP_RelativeHeight, 
makeAny( nHeight ) );
+// Only apply if nHeight != 0
+if ( nHeight )
+PropertySet( xShape ).setAnyProperty(PROP_RelativeHeight, 
makeAny( nHeight ) );
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: is 'make check' supposed to work on ~master ?

2012-12-13 Thread John Smith
Well, I re-tried 'make build ; make check' with './configure
--disable-online-update'.

But I run into the the following :
(Maybe I should leave this alone for now ...)




[build CUT] sw_subsequent_ooxmlimport
ods Test
xls Test
xlsx Test
csv Test
==5755== Conditional jump or move depends on uninitialised value(s)
==5755==at 0xE43D5E3: std::_Rb_treeunsigned long,
std::pairunsigned long const, SvNumberformat*,
std::_Select1ststd::pairunsigned long const, SvNumberformat* ,
std::lessunsigned long, std::allocatorstd::pairunsigned long
const, SvNumberformat* 
::_M_lower_bound(std::_Rb_tree_nodestd::pairunsigned long const,
SvNumberformat* *, std::_Rb_tree_nodestd::pairunsigned long const,
SvNumberformat* *, unsigned long const) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsvllo.so)
==5755==by 0xE437F69: std::_Rb_treeunsigned long,
std::pairunsigned long const, SvNumberformat*,
std::_Select1ststd::pairunsigned long const, SvNumberformat* ,
std::lessunsigned long, std::allocatorstd::pairunsigned long
const, SvNumberformat*  ::find(unsigned long const) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsvllo.so)
==5755==by 0xE432D2A: std::mapunsigned long, SvNumberformat*,
std::lessunsigned long, std::allocatorstd::pairunsigned long
const, SvNumberformat*  ::find(unsigned long const) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsvllo.so)
==5755==by 0xE4165E7: SvNumberFormatter::GetFormatEntry(unsigned
long) (in /usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsvllo.so)
==5755==by 0xE40BD31:
SvNumberFormatter::IsNumberFormat(rtl::OUString const, unsigned
long, double) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsvllo.so)
==5755==by 0xA532CC0: ScColumn::SetString(long, short, String
const, formula::FormulaGrammar::AddressConvention, ScSetStringParam*)
(in /usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsclo.so)
==5755==by 0xA967AD1: ScTable::SetString(short, long, short,
String const, ScSetStringParam*) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsclo.so)
==5755==by 0xA67C26C: ScDocument::SetString(short, long, short,
rtl::OUString const, ScSetStringParam*) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsclo.so)
==5755==by 0xB45C020: lcl_PutString(ScDocument*, short, long,
short, String const, unsigned char, SvNumberFormatter*, bool,
utl::TransliterationWrapper, CalendarWrapper,
utl::TransliterationWrapper*, CalendarWrapper*) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsclo.so)
==5755==by 0xB45EE90: ScImportExport::ExtText2Doc(SvStream) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsclo.so)
==5755==by 0xB4544D3: ScImportExport::ImportStream(SvStream,
String const, unsigned long) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsclo.so)
==5755==by 0xB3415CB: ScDocShell::ConvertFrom(SfxMedium) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsclo.so)
==5755==by 0xD8F5355: SfxObjectShell::DoLoad(SfxMedium*) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/libsfxlo.so)
==5755==by 0x65EF318: ScFiltersTest::load(rtl::OUString const,
rtl::OUString const, rtl::OUString const, rtl::OUString const,
unsigned int, unsigned int, unsigned int) (in
/usr/local/src/libreoffice/workdir/unxlngi6.pro/LinkTarget/CppunitTest/libtest_sc_subsequent_filters_test.so)
==5755==by 0x662804E: ScFiltersTest::testBrokenQuotesCSV() (in
/usr/local/src/libreoffice/workdir/unxlngi6.pro/LinkTarget/CppunitTest/libtest_sc_subsequent_filters_test.so)
==5755==by 0x666F4D8:
CppUnit::TestCallerScFiltersTest::runTest() (in
/usr/local/src/libreoffice/workdir/unxlngi6.pro/LinkTarget/CppunitTest/libtest_sc_subsequent_filters_test.so)
==5755==by 0x4B895E10:
CppUnit::TestCaseMethodFunctor::operator()() const (in
/usr/lib/libcppunit-1.12.so.1.0.0)
==5755==by 0x4C337CD: (anonymous
namespace)::Prot::protect(CppUnit::Functor const,
CppUnit::ProtectorContext const) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/unobootstrapprotector.so)
==5755==by 0x4B8930F1:
CppUnit::ProtectorChain::ProtectFunctor::operator()() const (in
/usr/lib/libcppunit-1.12.so.1.0.0)
==5755==by 0x4012ED5: (anonymous
namespace)::Prot::protect(CppUnit::Functor const,
CppUnit::ProtectorContext const) (in
/usr/local/src/libreoffice/solver/unxlngi6.pro/lib/unoexceptionprotector.so)
==5755==by 0x4B8930F1:
CppUnit::ProtectorChain::ProtectFunctor::operator()() const (in
/usr/lib/libcppunit-1.12.so.1.0.0)
==5755==by 0x4B88BDB9:
CppUnit::DefaultProtector::protect(CppUnit::Functor const,
CppUnit::ProtectorContext const) (in
/usr/lib/libcppunit-1.12.so.1.0.0)
==5755==by 0x4B8930F1:
CppUnit::ProtectorChain::ProtectFunctor::operator()() const (in
/usr/lib/libcppunit-1.12.so.1.0.0)
==5755==by 0x4B892DC8:
CppUnit::ProtectorChain::protect(CppUnit::Functor const,
CppUnit::ProtectorContext const) (in
/usr/lib/libcppunit-1.12.so.1.0.0)
==5755==by 0x4B89C5DD:

[Libreoffice-commits] .: Branch 'libreoffice-4-0' - helpcontent2

2012-12-13 Thread Libreoffice Gerrit user
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit c381a2f3063583d5e8750e3d13c46d02cd0cbb7c
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Dec 13 16:05:59 2012 +

Updated core
Project: help  84f109d2668f0bcbb11bbb5bff894466c538b052
the .tree files are xml, so need xml comments not c++ ones

diff --git a/helpcontent2 b/helpcontent2
index 32c4964..84f109d 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 32c4964e1415547af5dd956b590dc0edb3d10778
+Subproject commit 84f109d2668f0bcbb11bbb5bff894466c538b052
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - source/auxiliary

2012-12-13 Thread Libreoffice Gerrit user
 source/auxiliary/sbasic.tree   |7 +++
 source/auxiliary/scalc.tree|7 +++
 source/auxiliary/schart.tree   |7 +++
 source/auxiliary/shared.tree   |7 +++
 source/auxiliary/simpress.tree |7 +++
 source/auxiliary/smath.tree|7 +++
 source/auxiliary/swriter.tree  |7 +++
 7 files changed, 21 insertions(+), 28 deletions(-)

New commits:
commit 84f109d2668f0bcbb11bbb5bff894466c538b052
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Dec 13 16:05:59 2012 +

the .tree files are xml, so need xml comments not c++ ones

diff --git a/source/auxiliary/sbasic.tree b/source/auxiliary/sbasic.tree
index b228483..054daad 100644
--- a/source/auxiliary/sbasic.tree
+++ b/source/auxiliary/sbasic.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=sbasic id=07 title=Macros and 
Programming
node id=0701 title=General Information and User Interface 
Usage
diff --git a/source/auxiliary/scalc.tree b/source/auxiliary/scalc.tree
index 0da6d2a..fbaf76d 100644
--- a/source/auxiliary/scalc.tree
+++ b/source/auxiliary/scalc.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=scalc id=08 title=Spreadsheets
node id=0801 title=General Information and User Interface 
Usage
diff --git a/source/auxiliary/schart.tree b/source/auxiliary/schart.tree
index 4efafac..a470b99 100644
--- a/source/auxiliary/schart.tree
+++ b/source/auxiliary/schart.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=scalc id=05 title=Charts and Diagrams
node id=0501 title=General Information
diff --git a/source/auxiliary/shared.tree b/source/auxiliary/shared.tree
index ba7c734..5fc0ca1 100644
--- a/source/auxiliary/shared.tree
+++ b/source/auxiliary/shared.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=swriter id=01 title=Installation
 topic id=shared/text/shared/guide/ms_doctypes.xhpChanging the Association 
of Microsoft Office Document Types/topic
diff --git a/source/auxiliary/simpress.tree b/source/auxiliary/simpress.tree
index 520639c..b27417c 100644
--- a/source/auxiliary/simpress.tree
+++ b/source/auxiliary/simpress.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla Public
@@ -14,9 +15,7 @@
  *   License, Version 2.0 (the License); you may not use this file
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-?xml version=1.0 encoding=UTF-8?
+--
 tree_view version=24-Aug-2004
help_section application=simpress id=04 title=Presentations and 
Drawings
node id=0401 title=General Information and User Interface 
Usage
diff --git a/source/auxiliary/smath.tree b/source/auxiliary/smath.tree
index 6694bd8..07e61a0 100644
--- a/source/auxiliary/smath.tree
+++ b/source/auxiliary/smath.tree
@@ -1,4 +1,5 @@
-/*
+?xml version=1.0 encoding=UTF-8?
+!--
  * This file is part of the LibreOffice project.
  *
  * This Source Code Form is subject to the terms of the Mozilla 

[Libreoffice-commits] .: helpcontent2 oox/inc oox/source sc/source

2012-12-13 Thread Libreoffice Gerrit user
 helpcontent2  |2 +-
 oox/inc/oox/core/relations.hxx|2 ++
 oox/source/core/relations.cxx |6 ++
 oox/source/drawingml/hyperlinkcontext.cxx |5 +
 sc/source/filter/inc/worksheetbuffer.hxx  |4 
 sc/source/filter/oox/drawingfragment.cxx  |   22 --
 sc/source/filter/oox/worksheetbuffer.cxx  |   19 +++
 7 files changed, 57 insertions(+), 3 deletions(-)

New commits:
commit 5deba0e70c9287e6e933df458c21cc3e72f3aa70
Author: Noel Power noel.po...@suse.com
Date:   Thu Dec 13 17:32:00 2012 +

fix fdo#58237 import hyperlinks for shapes in xlsx documents

Change-Id: I9c64c0f4d1eb8533b65f35d5e85e1ab4881a2b67

diff --git a/helpcontent2 b/helpcontent2
index ca26126..56562e9 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit ca261267f5f67d4932e0046e0358e8d0fb8b4d1b
+Subproject commit 56562e9b04dcbfd63012eecddb53b76dc4cee84b
diff --git a/oox/inc/oox/core/relations.hxx b/oox/inc/oox/core/relations.hxx
index f840448..94b9908 100644
--- a/oox/inc/oox/core/relations.hxx
+++ b/oox/inc/oox/core/relations.hxx
@@ -79,6 +79,8 @@ public:
 
 /** Returns the external target of the relation with the passed relation 
identifier. */
 ::rtl::OUString getExternalTargetFromRelId( const ::rtl::OUString 
rRelId ) const;
+/** Returns the internal target of the relation with the passed relation 
identifier. */
+::rtl::OUString getInternalTargetFromRelId( const ::rtl::OUString 
rRelId ) const;
 
 /** Returns the full fragment path for the target of the passed relation. 
*/
 ::rtl::OUString getFragmentPathFromRelation( const Relation rRelation 
) const;
diff --git a/oox/source/core/relations.cxx b/oox/source/core/relations.cxx
index 501f515..c49fe63 100644
--- a/oox/source/core/relations.cxx
+++ b/oox/source/core/relations.cxx
@@ -83,6 +83,12 @@ OUString Relations::getExternalTargetFromRelId( const 
OUString rRelId ) const
 return (pRelation  pRelation-mbExternal) ? pRelation-maTarget : 
OUString();
 }
 
+OUString Relations::getInternalTargetFromRelId( const OUString rRelId ) const
+{
+const Relation* pRelation = getRelationFromRelId( rRelId );
+return (pRelation  !pRelation-mbExternal) ? pRelation-maTarget : 
OUString();
+}
+
 OUString Relations::getFragmentPathFromRelation( const Relation rRelation ) 
const
 {
 // no target, no fragment path
diff --git a/oox/source/drawingml/hyperlinkcontext.cxx 
b/oox/source/drawingml/hyperlinkcontext.cxx
index f2d715c..6da4db3 100644
--- a/oox/source/drawingml/hyperlinkcontext.cxx
+++ b/oox/source/drawingml/hyperlinkcontext.cxx
@@ -49,6 +49,11 @@ HyperLinkContext::HyperLinkContext( ContextHandler rParent,
 OSL_TRACE(OOX: URI href %s, OUStringToOString (sHref, 
RTL_TEXTENCODING_UTF8).pData-buffer);
 sURL = getFilter().getAbsoluteUrl( sHref );
 }
+else
+{
+// not sure if we also need to set sHref to the internal target
+sURL = getRelations().getInternalTargetFromRelId( aRelId );
+}
 }
 OUString sTooltip = xAttributes-getOptionalValue( R_TOKEN( tooltip ) );
 if ( !sTooltip.isEmpty() )
diff --git a/sc/source/filter/inc/worksheetbuffer.hxx 
b/sc/source/filter/inc/worksheetbuffer.hxx
index bceca55..d58a490 100644
--- a/sc/source/filter/inc/worksheetbuffer.hxx
+++ b/sc/source/filter/inc/worksheetbuffer.hxx
@@ -81,6 +81,10 @@ public:
 sal_Int16   getCalcSheetIndex( const ::rtl::OUString 
rWorksheetName ) const;
 /** Returns the finalized name of the sheet with the passed worksheet 
name. */
 ::rtl::OUString getCalcSheetName( const ::rtl::OUString 
rWorksheetName ) const;
+/** Converts sSheetNameRef (e.g. '#SheetName!A1' to '#SheetName.A1' )
+if sSheetNameRef doesn't start with '#' it is ignored and not modified
+*/
+voidconvertSheetNameRef( ::rtl::OUString sSheetNameRef ) 
const;
 
 private:
 struct SheetInfo : public SheetInfoModel
diff --git a/sc/source/filter/oox/drawingfragment.cxx 
b/sc/source/filter/oox/drawingfragment.cxx
index bb8fe08..a301326 100644
--- a/sc/source/filter/oox/drawingfragment.cxx
+++ b/sc/source/filter/oox/drawingfragment.cxx
@@ -26,6 +26,9 @@
 #include com/sun/star/script/ScriptEventDescriptor.hpp
 #include com/sun/star/script/XEventAttacherManager.hpp
 #include rtl/strbuf.hxx
+#include svx/svdobj.hxx
+#include drwlayer.hxx
+#include userdat.hxx
 #include oox/drawingml/connectorshapecontext.hxx
 #include oox/drawingml/graphicshapecontext.hxx
 #include oox/helper/attributelist.hxx
@@ -36,11 +39,10 @@
 #include stylesbuffer.hxx
 #include themebuffer.hxx
 #include unitconverter.hxx
-
+#include worksheetbuffer.hxx
 namespace oox {
 namespace xls {
 
-using namespace ::com::sun::star::awt;
 using namespace ::com::sun::star::beans;
 using namespace ::com::sun::star::container;
 using namespace ::com::sun::star::document;
@@ -53,6 +55,10 

[Libreoffice-commits] .: Branch 'libreoffice-4-0' - 2 commits - helpcontent2 oox/inc oox/source sc/source

2012-12-13 Thread Libreoffice Gerrit user
 helpcontent2  |2 +-
 oox/inc/oox/core/relations.hxx|2 ++
 oox/source/core/relations.cxx |6 ++
 oox/source/drawingml/hyperlinkcontext.cxx |5 +
 sc/source/filter/inc/worksheetbuffer.hxx  |4 
 sc/source/filter/oox/drawingfragment.cxx  |   22 --
 sc/source/filter/oox/workbookhelper.cxx   |4 ++--
 sc/source/filter/oox/worksheetbuffer.cxx  |   19 +++
 8 files changed, 59 insertions(+), 5 deletions(-)

New commits:
commit 08a84bfc4043dbae9a52a4559a7e5a5337464e92
Author: Noel Power noel.po...@suse.com
Date:   Thu Dec 13 18:15:45 2012 +

fix sometimes data corruption with xlsx import ( with scenario sheets )

formula import is buffered 'till the end of import. The processing of the
formula data need to happen before the scenario import happens. This is
necessary because sheet numbers stored in the formula addresses can become
invalid as scenario import insertes new hidden sheets upsetting the previous
table order

Change-Id: Id967a05ca06d1d918c959eaf1d1371ed2c6a4490

diff --git a/helpcontent2 b/helpcontent2
index 84f109d..d9d28eb 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 84f109d2668f0bcbb11bbb5bff894466c538b052
+Subproject commit d9d28ebb575e96e742d6590a28dd59c62b89dc4e
diff --git a/sc/source/filter/oox/workbookhelper.cxx 
b/sc/source/filter/oox/workbookhelper.cxx
index a9201ee..e5521e7 100644
--- a/sc/source/filter/oox/workbookhelper.cxx
+++ b/sc/source/filter/oox/workbookhelper.cxx
@@ -597,8 +597,6 @@ void WorkbookGlobals::finalize()
 // #i79826# enable updating automatic row height after loading the 
document
 aPropSet.setProperty( PROP_IsAdjustHeightEnabled, true );
 
-getFormulaBuffer().finalizeImport();
-
 // Insert all pivot tables. Must be done after loading all sheets and
 // formulas, because data pilots expect existing source data on
 // creation.
@@ -674,6 +672,8 @@ void WorkbookHelper::finalizeWorkbookImport()
 mrBookGlob.getWorkbookSettings().finalizeImport();
 mrBookGlob.getViewSettings().finalizeImport();
 
+// need to import formulas before scenarios
+mrBookGlob.getFormulaBuffer().finalizeImport();
 /*  Insert scenarios after all sheet processing is done, because new hidden
 sheets are created for scenarios which would confuse code that relies
 on certain sheet indexes. Must be done after pivot tables too. */
commit 691f7a1554cee0863258f2bfbfb3ab656ead57a1
Author: Noel Power noel.po...@suse.com
Date:   Thu Dec 13 16:26:58 2012 +

fix fdo#58237 import hyperlinks for shapes in xlsx documents

Change-Id: Ib0c661dbb3ce9a2f8c8d29707a1cf0c65aadc81f

diff --git a/oox/inc/oox/core/relations.hxx b/oox/inc/oox/core/relations.hxx
index f840448..94b9908 100644
--- a/oox/inc/oox/core/relations.hxx
+++ b/oox/inc/oox/core/relations.hxx
@@ -79,6 +79,8 @@ public:
 
 /** Returns the external target of the relation with the passed relation 
identifier. */
 ::rtl::OUString getExternalTargetFromRelId( const ::rtl::OUString 
rRelId ) const;
+/** Returns the internal target of the relation with the passed relation 
identifier. */
+::rtl::OUString getInternalTargetFromRelId( const ::rtl::OUString 
rRelId ) const;
 
 /** Returns the full fragment path for the target of the passed relation. 
*/
 ::rtl::OUString getFragmentPathFromRelation( const Relation rRelation 
) const;
diff --git a/oox/source/core/relations.cxx b/oox/source/core/relations.cxx
index 501f515..c49fe63 100644
--- a/oox/source/core/relations.cxx
+++ b/oox/source/core/relations.cxx
@@ -83,6 +83,12 @@ OUString Relations::getExternalTargetFromRelId( const 
OUString rRelId ) const
 return (pRelation  pRelation-mbExternal) ? pRelation-maTarget : 
OUString();
 }
 
+OUString Relations::getInternalTargetFromRelId( const OUString rRelId ) const
+{
+const Relation* pRelation = getRelationFromRelId( rRelId );
+return (pRelation  !pRelation-mbExternal) ? pRelation-maTarget : 
OUString();
+}
+
 OUString Relations::getFragmentPathFromRelation( const Relation rRelation ) 
const
 {
 // no target, no fragment path
diff --git a/oox/source/drawingml/hyperlinkcontext.cxx 
b/oox/source/drawingml/hyperlinkcontext.cxx
index 209e733..1329c30 100644
--- a/oox/source/drawingml/hyperlinkcontext.cxx
+++ b/oox/source/drawingml/hyperlinkcontext.cxx
@@ -50,6 +50,11 @@ HyperLinkContext::HyperLinkContext( ContextHandler rParent,
 OSL_TRACE(OOX: URI href %s, ::rtl::OUStringToOString (sHref, 
RTL_TEXTENCODING_UTF8).pData-buffer);
 sURL = getFilter().getAbsoluteUrl( sHref );
 }
+else
+{
+// not sure if we also need to set sHref to the internal target
+sURL = getRelations().getInternalTargetFromRelId( aRelId );
+}
 }
 OUString sTooltip = 

[Libreoffice-commits] .: cui/source desktop/source editeng/source

2012-12-13 Thread Libreoffice Gerrit user
 cui/source/tabpages/autocdlg.cxx   |6 ++---
 desktop/source/migration/migration.cxx |8 +++
 editeng/source/misc/svxacorr.cxx   |   35 +++--
 3 files changed, 27 insertions(+), 22 deletions(-)

New commits:
commit 623410669fa2d5da9a2ce4e3c4b81ce23605a6df
Author: Eike Rathke er...@redhat.com
Date:   Thu Dec 13 19:51:52 2012 +0100

fdo#58060 use acor_und.dat and LANGUAGE_UNDETERMINED

Previous versions used an empty language tag for LANGUAGE_DONTKNOW with the
[All] autocorrection entry and read/wrote from/to arco_.dat file. An empty
language tag otherwise is used for system locale and doesn't convert 
flawlessly
with the new LanguageTag system. Instead use LANGUAGE_UNDETERMINED with the 
ISO
639 code 'und' so the file name is acor_und.dat.

During user profile migration an existing 3/user/autocorr/acor_.dat is 
copied
to the new 4/user/autocorr/acor_und.dat

Change-Id: I593f24829c6efd58f36e93ebd3385a3c925f7217

diff --git a/cui/source/tabpages/autocdlg.cxx b/cui/source/tabpages/autocdlg.cxx
index 66bf164..48bd0c7 100644
--- a/cui/source/tabpages/autocdlg.cxx
+++ b/cui/source/tabpages/autocdlg.cxx
@@ -111,7 +111,7 @@ OfaAutoCorrDlg::OfaAutoCorrDlg(Window* pParent, const 
SfxItemSet* _pSet ) :
 
 // initialize languages
 //! LANGUAGE_NONE is displayed as '[All]' and the LanguageType
-//! will be set to LANGUAGE_DONTKNOW
+//! will be set to LANGUAGE_UNDETERMINED
 sal_Int16 nLangList = LANG_LIST_WESTERN;
 
 if( SvtLanguageOptions().IsCTLFontEnabled() )
@@ -120,13 +120,13 @@ OfaAutoCorrDlg::OfaAutoCorrDlg(Window* pParent, const 
SfxItemSet* _pSet ) :
 aLanguageLB.SelectLanguage( LANGUAGE_NONE );
 sal_uInt16 nPos = aLanguageLB.GetSelectEntryPos();
 DBG_ASSERT( LISTBOX_ENTRY_NOTFOUND != nPos, listbox entry missing );
-aLanguageLB.SetEntryData( nPos, (void*)(long) LANGUAGE_DONTKNOW );
+aLanguageLB.SetEntryData( nPos, (void*)(long) LANGUAGE_UNDETERMINED );
 
 // Initializing doesn't work for static on linux - therefore here
 if( LANGUAGE_SYSTEM == eLastDialogLanguage )
 eLastDialogLanguage = 
Application::GetSettings().GetLanguageTag().getLanguageType();
 
-LanguageType nSelectLang = LANGUAGE_DONTKNOW;
+LanguageType nSelectLang = LANGUAGE_UNDETERMINED;
 nPos = aLanguageLB.GetEntryPos( (void*)(long) eLastDialogLanguage );
 if (LISTBOX_ENTRY_NOTFOUND != nPos)
 nSelectLang = eLastDialogLanguage;
diff --git a/desktop/source/migration/migration.cxx 
b/desktop/source/migration/migration.cxx
index 6036ef5..1b4115a 100644
--- a/desktop/source/migration/migration.cxx
+++ b/desktop/source/migration/migration.cxx
@@ -922,6 +922,14 @@ void MigrationImpl::copyFiles()
 {
 // remove installation prefix from file
 localName = i_file-copy(m_aInfo.userdata.getLength());
+if (localName.endsWith( /autocorr/acor_.dat))
+{
+// Previous versions used an empty language tag for
+// LANGUAGE_DONTKNOW with the [All] autocorrection entry.
+// As of LibreOffice 4.0 it is 'und' for LANGUAGE_UNDETERMINED
+// so the file name is acor_und.dat.
+localName = localName.copy( 0, localName.getLength() - 4) + 
und.dat;
+}
 destName = userInstall + localName;
 INetURLObject aURL(destName);
 // check whether destination directory exists
diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index b4fa5d0..50c5b02 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -1481,11 +1481,11 @@ sal_Bool SvxAutoCorrect::AddCplSttException( const 
String rNew,
 pLists = nTmpVal-second;
 else
 {
-nTmpVal = pLangTable-find(LANGUAGE_DONTKNOW);
+nTmpVal = pLangTable-find(LANGUAGE_UNDETERMINED);
 if(nTmpVal != pLangTable-end())
 pLists = nTmpVal-second;
-else if(CreateLanguageFile(LANGUAGE_DONTKNOW, sal_True))
-pLists = pLangTable-find(LANGUAGE_DONTKNOW)-second;
+else if(CreateLanguageFile(LANGUAGE_UNDETERMINED, sal_True))
+pLists = pLangTable-find(LANGUAGE_UNDETERMINED)-second;
 }
 OSL_ENSURE(pLists, No auto correction data);
 return pLists-AddToCplSttExceptList(rNew);
@@ -1502,11 +1502,11 @@ sal_Bool SvxAutoCorrect::AddWrtSttException( const 
String rNew,
 pLists = nTmpVal-second;
 else
 {
-nTmpVal = pLangTable-find(LANGUAGE_DONTKNOW);
+nTmpVal = pLangTable-find(LANGUAGE_UNDETERMINED);
 if(nTmpVal != pLangTable-end())
 pLists = nTmpVal-second;
-else if(CreateLanguageFile(LANGUAGE_DONTKNOW, sal_True))
-pLists = pLangTable-find(LANGUAGE_DONTKNOW)-second;
+else if(CreateLanguageFile(LANGUAGE_UNDETERMINED, sal_True))
+pLists = 

[Libreoffice-commits] .: Branch 'libreoffice-4-0' - cui/source desktop/source editeng/source

2012-12-13 Thread Libreoffice Gerrit user
 cui/source/tabpages/autocdlg.cxx   |6 ++---
 desktop/source/migration/migration.cxx |8 +++
 editeng/source/misc/svxacorr.cxx   |   35 +++--
 3 files changed, 27 insertions(+), 22 deletions(-)

New commits:
commit cbd9177a3985910312d4f6a4b7c2068bbab50619
Author: Eike Rathke er...@redhat.com
Date:   Thu Dec 13 19:51:52 2012 +0100

fdo#58060 use acor_und.dat and LANGUAGE_UNDETERMINED

Previous versions used an empty language tag for LANGUAGE_DONTKNOW with the
[All] autocorrection entry and read/wrote from/to arco_.dat file. An empty
language tag otherwise is used for system locale and doesn't convert 
flawlessly
with the new LanguageTag system. Instead use LANGUAGE_UNDETERMINED with the 
ISO
639 code 'und' so the file name is acor_und.dat.

During user profile migration an existing 3/user/autocorr/acor_.dat is 
copied
to the new 4/user/autocorr/acor_und.dat

Change-Id: I593f24829c6efd58f36e93ebd3385a3c925f7217
(cherry picked from commit 623410669fa2d5da9a2ce4e3c4b81ce23605a6df)

diff --git a/cui/source/tabpages/autocdlg.cxx b/cui/source/tabpages/autocdlg.cxx
index 66bf164..48bd0c7 100644
--- a/cui/source/tabpages/autocdlg.cxx
+++ b/cui/source/tabpages/autocdlg.cxx
@@ -111,7 +111,7 @@ OfaAutoCorrDlg::OfaAutoCorrDlg(Window* pParent, const 
SfxItemSet* _pSet ) :
 
 // initialize languages
 //! LANGUAGE_NONE is displayed as '[All]' and the LanguageType
-//! will be set to LANGUAGE_DONTKNOW
+//! will be set to LANGUAGE_UNDETERMINED
 sal_Int16 nLangList = LANG_LIST_WESTERN;
 
 if( SvtLanguageOptions().IsCTLFontEnabled() )
@@ -120,13 +120,13 @@ OfaAutoCorrDlg::OfaAutoCorrDlg(Window* pParent, const 
SfxItemSet* _pSet ) :
 aLanguageLB.SelectLanguage( LANGUAGE_NONE );
 sal_uInt16 nPos = aLanguageLB.GetSelectEntryPos();
 DBG_ASSERT( LISTBOX_ENTRY_NOTFOUND != nPos, listbox entry missing );
-aLanguageLB.SetEntryData( nPos, (void*)(long) LANGUAGE_DONTKNOW );
+aLanguageLB.SetEntryData( nPos, (void*)(long) LANGUAGE_UNDETERMINED );
 
 // Initializing doesn't work for static on linux - therefore here
 if( LANGUAGE_SYSTEM == eLastDialogLanguage )
 eLastDialogLanguage = 
Application::GetSettings().GetLanguageTag().getLanguageType();
 
-LanguageType nSelectLang = LANGUAGE_DONTKNOW;
+LanguageType nSelectLang = LANGUAGE_UNDETERMINED;
 nPos = aLanguageLB.GetEntryPos( (void*)(long) eLastDialogLanguage );
 if (LISTBOX_ENTRY_NOTFOUND != nPos)
 nSelectLang = eLastDialogLanguage;
diff --git a/desktop/source/migration/migration.cxx 
b/desktop/source/migration/migration.cxx
index 6036ef5..1b4115a 100644
--- a/desktop/source/migration/migration.cxx
+++ b/desktop/source/migration/migration.cxx
@@ -922,6 +922,14 @@ void MigrationImpl::copyFiles()
 {
 // remove installation prefix from file
 localName = i_file-copy(m_aInfo.userdata.getLength());
+if (localName.endsWith( /autocorr/acor_.dat))
+{
+// Previous versions used an empty language tag for
+// LANGUAGE_DONTKNOW with the [All] autocorrection entry.
+// As of LibreOffice 4.0 it is 'und' for LANGUAGE_UNDETERMINED
+// so the file name is acor_und.dat.
+localName = localName.copy( 0, localName.getLength() - 4) + 
und.dat;
+}
 destName = userInstall + localName;
 INetURLObject aURL(destName);
 // check whether destination directory exists
diff --git a/editeng/source/misc/svxacorr.cxx b/editeng/source/misc/svxacorr.cxx
index 4e59f09..cf486af 100644
--- a/editeng/source/misc/svxacorr.cxx
+++ b/editeng/source/misc/svxacorr.cxx
@@ -1481,11 +1481,11 @@ sal_Bool SvxAutoCorrect::AddCplSttException( const 
String rNew,
 pLists = nTmpVal-second;
 else
 {
-nTmpVal = pLangTable-find(LANGUAGE_DONTKNOW);
+nTmpVal = pLangTable-find(LANGUAGE_UNDETERMINED);
 if(nTmpVal != pLangTable-end())
 pLists = nTmpVal-second;
-else if(CreateLanguageFile(LANGUAGE_DONTKNOW, sal_True))
-pLists = pLangTable-find(LANGUAGE_DONTKNOW)-second;
+else if(CreateLanguageFile(LANGUAGE_UNDETERMINED, sal_True))
+pLists = pLangTable-find(LANGUAGE_UNDETERMINED)-second;
 }
 OSL_ENSURE(pLists, No auto correction data);
 return pLists-AddToCplSttExceptList(rNew);
@@ -1502,11 +1502,11 @@ sal_Bool SvxAutoCorrect::AddWrtSttException( const 
String rNew,
 pLists = nTmpVal-second;
 else
 {
-nTmpVal = pLangTable-find(LANGUAGE_DONTKNOW);
+nTmpVal = pLangTable-find(LANGUAGE_UNDETERMINED);
 if(nTmpVal != pLangTable-end())
 pLists = nTmpVal-second;
-else if(CreateLanguageFile(LANGUAGE_DONTKNOW, sal_True))
-pLists = pLangTable-find(LANGUAGE_DONTKNOW)-second;
+else 

[Libreoffice-commits] Changes to 'suse-3.6.3'

2012-12-13 Thread Libreoffice Gerrit user
New branch 'suse-3.6.3' available with the following commits:
commit 7553ac8d6d214fa2cba0b9775d17a87ddb10208e
Author: Petr Mladek pmla...@suse.cz
Date:   Thu Dec 13 20:40:58 2012 +0100

Branch suse-3.6.3

This is 'suse-3.6.3' - branch for the final stabilisation
of the fist SUSE 3.6 release. Only very important and/or very safe changes
are allowed.

Please, keep pushing into suse-3.6 branch which will be used for the
next potential bugfix releases.

Change-Id: I1d12f03ace181059716a3ace194c8c382ac370e9

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] Changes to 'suse-3.6.3'

2012-12-13 Thread Libreoffice Gerrit user
New branch 'suse-3.6.3' available with the following commits:
commit b44c28aa94749d9436f47603e7456abe5280018d
Author: Petr Mladek pmla...@suse.cz
Date:   Thu Dec 13 20:40:59 2012 +0100

Branch suse-3.6.3

This is 'suse-3.6.3' - branch for the final stabilisation
of the fist SUSE 3.6 release. Only very important and/or very safe changes
are allowed.

Please, keep pushing into suse-3.6 branch which will be used for the
next potential bugfix releases.

Change-Id: I5223a1411b005055b229dce20f6f6f91e2628d30

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] Changes to 'suse-3.6.3'

2012-12-13 Thread Libreoffice Gerrit user
New branch 'suse-3.6.3' available with the following commits:
commit 9d7f51cf1959793f6a0376e21af98b672b7f25fe
Author: Petr Mladek pmla...@suse.cz
Date:   Thu Dec 13 20:40:59 2012 +0100

Branch suse-3.6.3

This is 'suse-3.6.3' - branch for the final stabilisation
of the fist SUSE 3.6 release. Only very important and/or very safe changes
are allowed.

Please, keep pushing into suse-3.6 branch which will be used for the
next potential bugfix releases.

Change-Id: I83439660556b99fab3a46b00bc96399add1f6f88

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] Changes to 'suse-3.6.3'

2012-12-13 Thread Libreoffice Gerrit user
New branch 'suse-3.6.3' available with the following commits:
commit 45ed353464e662fb428873a7258cc3f334dfa95d
Author: Petr Mladek pmla...@suse.cz
Date:   Thu Dec 13 20:40:59 2012 +0100

Branch suse-3.6.3

This is 'suse-3.6.3' - branch for the final stabilisation
of the fist SUSE 3.6 release. Only very important and/or very safe changes
are allowed.

Please, keep pushing into suse-3.6 branch which will be used for the
next potential bugfix releases.

Change-Id: I7473206f8101bdfaaa3b4ccf74e7f146fca6f3a9

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] Changes to 'suse-3.6.3'

2012-12-13 Thread Libreoffice Gerrit user
New branch 'suse-3.6.3' available with the following commits:
commit 8f52724d8a080fdc8fb55586515a8a5b6048112b
Author: Petr Mladek pmla...@suse.cz
Date:   Thu Dec 13 20:41:00 2012 +0100

Branch suse-3.6.3

This is 'suse-3.6.3' - branch for the final stabilisation
of the fist SUSE 3.6 release. Only very important and/or very safe changes
are allowed.

Please, keep pushing into suse-3.6 branch which will be used for the
next potential bugfix releases.

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


  1   2   3   4   >