Re: replacing OUString::valueOf(static_castsal_Int32) ??

2013-01-10 Thread Noel Grandin


On 2013-01-09 19:58, Michael Meeks wrote:
At least in my mind :-) but we're starting to bike-shed here... I 
didn't see anyone volunteering to do the actual batch cleanup there 
;-) Regards, Michael. 


Created a proof-of-concept patch along with some unit tests and pushed 
to gerrit:


https://gerrit.libreoffice.org/#/c/1625/

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


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


Re: gmake: CXXFLAGS override gb_Library_add_cxxflags

2013-01-10 Thread Stephan Bergmann

On 01/09/2013 06:28 PM, Lubos Lunak wrote:

On Wednesday 09 of January 2013, Stephan Bergmann wrote:

On 01/09/2013 03:41 PM, Petr Mladek wrote:

+ openSUSE uses:

CXXFLAGS=-fomit-frame-pointer ...


What do you mean with openSUSE uses [that]?  Is the env var preset
when you run a shell?  Why would openSUSE do that in the first place?


  It's when building RPM packages, which are supposed to be built with
$RPM_OPT_FLAGS.


Ah, so Petr's IMHO, it would make sense to use CXXFLAGS after the 
default global flags but before the source file specific flags sounds 
reasonable to me.


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


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

2013-01-10 Thread Libreoffice Gerrit user
 svx/source/dialog/docrecovery.cxx |2 +-
 svx/source/inc/docrecovery.hxx|2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit fe3bcdac912580607611475bfc3b0fa478a54ec1
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 8 14:04:09 2013 +

Resolves: fdo#58589 recovery dialog title area text is missing

regression since efc6b840a0423099d6c3e0d33c14d34637fbba4c
as that changes the order in which windows are drawn

Change-Id: I37388229533127ab915bec351f094fcd4786b2a3
(cherry picked from commit f8a4ce67987e5cb47f07dc61eeb450e1d5b9c541)
Reviewed-on: https://gerrit.libreoffice.org/1606
Reviewed-by: Tor Lillqvist t...@iki.fi
Tested-by: Tor Lillqvist t...@iki.fi

diff --git a/svx/source/dialog/docrecovery.cxx 
b/svx/source/dialog/docrecovery.cxx
index 3f9d4de..277d9ec 100644
--- a/svx/source/dialog/docrecovery.cxx
+++ b/svx/source/dialog/docrecovery.cxx
@@ -987,8 +987,8 @@ short impl_askUserForWizardCancel(Window* pParent, 
sal_Int16 nRes)
 RecoveryDialog::RecoveryDialog(Window*   pParent,
RecoveryCore* pCore  )
 : IExtendedTabPage( pParent  , SVX_RES( 
RID_SVXPAGE_DOCRECOVERY_RECOVER ) )
-, m_aTitleWin   ( this   , SVX_RES  ( WIN_RECOV_TITLE  
  ) )
 , m_aTitleFT( this   , SVX_RES  ( FT_RECOV_TITLE   
  ) )
+, m_aTitleWin   ( this   , SVX_RES  ( WIN_RECOV_TITLE  
  ) )
 , m_aTitleFL( this   , SVX_RES  ( FL_RECOV_TITLE   
  ) )
 , m_aDescrFT( this   , SVX_RES  ( FT_RECOV_DESCR   
  ) )
 , m_aProgressFT ( this   , SVX_RES  ( FT_RECOV_PROGR   
  ) )
diff --git a/svx/source/inc/docrecovery.hxx b/svx/source/inc/docrecovery.hxx
index f3e47e0..171c34d 100644
--- a/svx/source/inc/docrecovery.hxx
+++ b/svx/source/inc/docrecovery.hxx
@@ -606,8 +606,8 @@ class RecoveryDialog : public IExtendedTabPage
 //---
 // member
 private:
-Window  m_aTitleWin;
 FixedText   m_aTitleFT;
+Window  m_aTitleWin;
 FixedLine   m_aTitleFL;
 FixedText   m_aDescrFT;
 FixedText   m_aProgressFT;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-4-0]: Resolves: fdo#58589 recovery dialog title area text is missi...

2013-01-10 Thread Miklos Vajna (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/1606

Approvals:
  Tor Lillqvist: Verified; Looks good to me, approved
  Miklos Vajna: 


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I37388229533127ab915bec351f094fcd4786b2a3
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Tor Lillqvist t...@iki.fi

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


Re: [libreoffice-dev] - questions about calc extension

2013-01-10 Thread Stephan Bergmann

On 01/09/2013 07:41 PM, Kohei Yoshida wrote:

On 01/09/2013 01:12 PM, Rai, Neeraj wrote:

What is the correct way to get a
handle to spreadsheet in extension code.


I don't want you to look too close to this (since I no longer maintain
this), bug this code of mine

http://code.google.com/p/scsolver/source/browse/source/ui/xcalc.cxx

may give you some hint, especially the top 3 methods,
getCurrentComponent(), updateCurrentComponent(), and getServiceManager()
should give you how to get the current component object from the current
context object.  As an extension, you are given access to the current
context object, and you can reach the current component from that one
way or another.


Some comments on the above code, for the benefit of fresh development:

Obtaining the com.sun.star.frame.Desktop service (variable xDesktop) in 
updateCurrentComponent can be greatly simplified through the use of 
so-called new-style service constructors (and Noel's ongoing work of 
changing existing old-style services into new-style ones in UNOIDL):


  Reference frame::XDesktop2  xDesktop(
  frame::Desktop::create(m_xCC));

- It already returns the proper interface of the service (in this case 
XDesktop2, derived from XDesktop), so no need for querying from 
XInterface to the correct type.


- It directly uses the component context, no need for the service 
manager.  (In general, demand for explicitly using the service manager 
should go away more and more over time.)


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


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - vcl/generic vcl/inc vcl/unx

2013-01-10 Thread Libreoffice Gerrit user
 vcl/generic/print/genprnpsp.cxx |1 +
 vcl/inc/vcl/jobdata.hxx |3 ++-
 vcl/unx/generic/printer/jobdata.cxx |8 +++-
 3 files changed, 10 insertions(+), 2 deletions(-)

New commits:
commit 4c93acdba9274f5571a01a7ed5738957d8ebb0cd
Author: Caolán McNamara caol...@redhat.com
Date:   Wed Jan 9 12:44:09 2013 +

Resolves: fdo#59131 resolve default backend on printer-setup

For setting permanent defaults via spadmin we don't want to resolve the
default backend on saving default, this happens during the print
initialization stuff.

But when we tweak the printer settings for a single print job and reset 
back to
default, we need to resolve that to what backend we want

Change-Id: Iac2370624b6d248c9658fb0fbc2f2d449849d81a
(cherry picked from commit 9c4d77f22d58f5b500c95f65a431628a8d8d6e6e)
Reviewed-on: https://gerrit.libreoffice.org/1609
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/vcl/generic/print/genprnpsp.cxx b/vcl/generic/print/genprnpsp.cxx
index 9949eae..6800e01 100644
--- a/vcl/generic/print/genprnpsp.cxx
+++ b/vcl/generic/print/genprnpsp.cxx
@@ -584,6 +584,7 @@ sal_Bool PspSalInfoPrinter::Setup( SalFrame* pFrame, 
ImplJobSetup* pJobSetup )
 
 if( pSetupFunction( aInfo ) )
 {
+aInfo.resolveDefaultBackend();
 rtl_freeMemory( pJobSetup-mpDriverData );
 pJobSetup-mpDriverData = NULL;
 
diff --git a/vcl/inc/vcl/jobdata.hxx b/vcl/inc/vcl/jobdata.hxx
index 577e0fa..3f81877 100644
--- a/vcl/inc/vcl/jobdata.hxx
+++ b/vcl/inc/vcl/jobdata.hxx
@@ -68,7 +68,8 @@ struct VCL_DLLPUBLIC JobData
 void setCollate( bool bCollate );
 bool setPaper( int nWidth, int nHeight ); // dimensions in pt
 bool setPaperBin( int nPaperBin ); // dimensions in pt
-void setDefaultBackend( bool bUsePDF );
+void resolveDefaultBackend();
+void setDefaultBackend(bool bUsePDF);
 
 // creates a new buffer using new
 // it is up to the user to delete it again
diff --git a/vcl/unx/generic/printer/jobdata.cxx 
b/vcl/unx/generic/printer/jobdata.cxx
index 385ed02..c55f64a 100644
--- a/vcl/unx/generic/printer/jobdata.cxx
+++ b/vcl/unx/generic/printer/jobdata.cxx
@@ -18,9 +18,9 @@
  */
 
 
+#include officecfg/Office/Common.hxx
 #include vcl/jobdata.hxx
 #include vcl/printerinfomanager.hxx
-
 #include tools/stream.hxx
 
 #include sal/alloca.h
@@ -269,6 +269,12 @@ bool JobData::constructFromStreamBuffer( void* pData, int 
bytes, JobData rJobDa
 return bVersion  bPrinter  bOrientation  bCopies  bContext  
bMargin  bPSLevel  bPDFDevice  bColorDevice  bColorDepth;
 }
 
+void JobData::resolveDefaultBackend()
+{
+if (m_nPSLevel == 0  m_nPDFDevice == 0)
+
setDefaultBackend(officecfg::Office::Common::Print::Option::Printer::PDFAsStandardPrintJobFormat::get());
+}
+
 void JobData::setDefaultBackend(bool bUsePDF)
 {
 if (bUsePDF  m_nPSLevel == 0  m_nPDFDevice == 0)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-4-0]: Resolves: fdo#59131 resolve default backend on printer-setu...

2013-01-10 Thread Miklos Vajna (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/1609

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iac2370624b6d248c9658fb0fbc2f2d449849d81a
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - sw/source

2013-01-10 Thread Libreoffice Gerrit user
 sw/source/core/unocore/unocrsrhelper.cxx |   11 ---
 sw/source/core/unocore/unoobj.cxx|4 
 sw/source/core/unocore/unoobj2.cxx   |7 ---
 3 files changed, 16 insertions(+), 6 deletions(-)

New commits:
commit b58dc1b9aee370a1a2aa9d8f951cede3ae24d61d
Author: Michael Stahl mst...@redhat.com
Date:   Wed Jan 9 15:45:32 2013 +0100

fdo#58242: sw: fix more crashes when not on SwTxtNode

In getCrsrPropertyValue etc.; also add a unit test for the problem.

(cherry picked from commit 68d40d2cae3700f4134375fcaf9649ac626ada7d)

Conflicts:
sw/qa/complex/writer/CheckTable.java

Change-Id: Ibd459a43393c39b4fed9fb89aae4a5f7bacff007
Reviewed-on: https://gerrit.libreoffice.org/1615
Reviewed-by: Noel Power noel.po...@suse.com
Tested-by: Noel Power noel.po...@suse.com

diff --git a/sw/source/core/unocore/unocrsrhelper.cxx 
b/sw/source/core/unocore/unocrsrhelper.cxx
index e41d237..aa9cc45 100644
--- a/sw/source/core/unocore/unocrsrhelper.cxx
+++ b/sw/source/core/unocore/unocrsrhelper.cxx
@@ -497,9 +497,9 @@ sal_Bool getCrsrPropertyValue(const 
SfxItemPropertySimpleEntry rEntry
 case FN_UNO_CHARFMT_SEQUENCE:
 {
 
-SwTxtNode* pTxtNode;
-if((pTxtNode = (SwTxtNode*)rPam.GetNode( sal_True )) == 
rPam.GetNode(sal_False) 
-pTxtNode-GetpSwpHints())
+SwTxtNode *const pTxtNode = rPam.GetNode()-GetTxtNode();
+if (rPam.GetNode(sal_True) == rPam.GetNode(sal_False)
+ pTxtNode  pTxtNode-GetpSwpHints())
 {
 sal_uInt16 nPaMStart = rPam.GetPoint()-nContent.GetIndex();
 sal_uInt16 nPaMEnd = rPam.GetMark() ? 
rPam.GetMark()-nContent.GetIndex() : nPaMStart;
@@ -951,6 +951,11 @@ sal_Bool DocInsertStringSplitCR(
 xub_StrLen nStartIdx = 0;
 SwTxtNode* const pTxtNd =
 rNewCursor.GetPoint()-nNode.GetNode().GetTxtNode();
+if (!pTxtNd)
+{
+SAL_INFO(sw.uno, DocInsertStringSplitCR: need a text node);
+return false;
+}
 const xub_StrLen nMaxLength = ( pTxtNd )
 ? STRING_LEN - pTxtNd-GetTxt().Len()
 : STRING_LEN;
diff --git a/sw/source/core/unocore/unoobj.cxx 
b/sw/source/core/unocore/unoobj.cxx
index 3bba084..d400286 100644
--- a/sw/source/core/unocore/unoobj.cxx
+++ b/sw/source/core/unocore/unoobj.cxx
@@ -556,6 +556,10 @@ throw (lang::IllegalArgumentException)
 {
 // multi selection is not considered
 SwTxtNode *const pTxtNd = rPam.GetNode()-GetTxtNode();
+if (!pTxtNd)
+{
+throw lang::IllegalArgumentException();
+}
 if (FN_UNO_NUM_LEVEL == rEntry.nWID)
 {
 sal_Int16 nLevel = 0;
diff --git a/sw/source/core/unocore/unoobj2.cxx 
b/sw/source/core/unocore/unoobj2.cxx
index ea121c1..6e0bf2f 100644
--- a/sw/source/core/unocore/unoobj2.cxx
+++ b/sw/source/core/unocore/unoobj2.cxx
@@ -1851,9 +1851,10 @@ lcl_FillFrame(SwClient  rEnum, SwUnoCrsr rUnoCrsr,
 FrameDependList_t  rFrames)
 {
 // search for objects at the cursor - anchored at/as char
-SwTxtAttr const*const pTxtAttr =
-rUnoCrsr.GetNode()-GetTxtNode()-GetTxtAttrForCharAt(
-rUnoCrsr.GetPoint()-nContent.GetIndex(), RES_TXTATR_FLYCNT);
+SwTxtAttr const*const pTxtAttr = (rUnoCrsr.GetNode()-IsTxtNode())
+? rUnoCrsr.GetNode()-GetTxtNode()-GetTxtAttrForCharAt(
+rUnoCrsr.GetPoint()-nContent.GetIndex(), RES_TXTATR_FLYCNT)
+: 0;
 if (pTxtAttr)
 {
 const SwFmtFlyCnt rFlyCnt = pTxtAttr-GetFlyCnt();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-3-6]: fdo#58242: sw: fix more crashes when not on SwTxtNode

2013-01-10 Thread Noel Power (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/1615

Approvals:
  Noel Power: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibd459a43393c39b4fed9fb89aae4a5f7bacff007
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Noel Power noel.po...@suse.com

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


Removed some RTL_* macros and rtl:: prefixes in dbaccess

2013-01-10 Thread Miklos Vajna (via Code Review)
Miklos Vajna has abandoned this change.

Change subject: Removed some RTL_* macros and rtl:: prefixes in dbaccess
..


Patch Set 2: Abandoned

Uhm, sorry, when resolving the conflicts, the change-id line moved up, so 
gerrit didn't notice this change is pushed. So even if I technically abandon 
this change, it's in fact merged: 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=0f2a3a3bdc157d997fe1967e10300fc6fecf2bec

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: I467ad9307536e6cdd32ebcdd40539bd5f8614c85
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Marcos Souza marcos.souza@gmail.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


[Libreoffice-commits] .: bridges/Module_bridges.mk Makefile.in solenv/gbuild

2013-01-10 Thread Libreoffice Gerrit user
 Makefile.in   |2 +-
 bridges/Module_bridges.mk |2 +-
 solenv/gbuild/Module.mk   |   17 +++--
 3 files changed, 17 insertions(+), 4 deletions(-)

New commits:
commit a5838b19bc774efd261c1d6c78a2b6eb62f1b502
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Wed Jan 9 14:24:33 2013 +0100

add rule for generating fresh pot (po template) files

Moved from translations' custom makefile.

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

diff --git a/Makefile.in b/Makefile.in
index 9c12258..013548b 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -422,7 +422,7 @@ check: dev-install subsequentcheck
 subsequentcheck :| $(if $(filter-out 
subsequentcheck,$(MAKECMDGOALS)),dev-install)
$(GNUMAKE) -j $(CHECK_PARALLELISM) $(GMAKE_OPTIONS) -f 
$(SRC_ROOT)/Makefile.post $@
 
-unitcheck slowcheck debugrun:
+unitcheck slowcheck debugrun translations:
$(GNUMAKE) -j $(PARALLELISM) $(GMAKE_OPTIONS) -f 
$(SRC_ROOT)/Makefile.post $@
 
 endif # not clean or distclean
diff --git a/bridges/Module_bridges.mk b/bridges/Module_bridges.mk
index 118e82f..14a7255 100644
--- a/bridges/Module_bridges.mk
+++ b/bridges/Module_bridges.mk
@@ -96,7 +96,7 @@ $(eval $(call gb_Module_add_targets,bridges,\
) \
 ))
 
-ifneq (showmodules,$(MAKECMDGOALS))
+ifeq ($(gb_FULL_BUILD),$(true))
 ifeq ($(strip $(bridges_SELECTED_BRIDGE)),)
 $(call gb_Output_error,no bridge selected for build: bailing out)
 endif
diff --git a/solenv/gbuild/Module.mk b/solenv/gbuild/Module.mk
index fad1821..2863be8 100644
--- a/solenv/gbuild/Module.mk
+++ b/solenv/gbuild/Module.mk
@@ -89,7 +89,7 @@ $(call gb_Module_get_target,%) :
mkdir -p $(dir $@)  \
touch $@)
 
-.PHONY : build all clean unitcheck slowcheck subsequentcheck dev-install 
showmodules
+.PHONY : build all clean unitcheck slowcheck subsequentcheck dev-install 
showmodules translations
 .DEFAULT_GOAL := all
 
 ifeq ($(strip $(gb_PARTIALBUILD)),)
@@ -188,6 +188,17 @@ showmodules :
$(info $(strip $(gb_Module_ALLMODULES)))
@true 
 
+translations : $(WORKDIR)/pot.done
+
+$(WORKDIR)/pot.done : $(foreach exec,cfgex helpex localize transex3 \
+   propex 
uiex ulfex xrmex treex, \
+   $(call 
gb_Executable_get_target_for_build,$(exec)))
+   $(call gb_Output_announce,$(subst .pot,,$(subst 
$(WORKDIR)/,,$@)),$(true),POT,1)
+   $(call gb_Helper_abbreviate_dirs,\
+   mkdir -p $(dir $@)  $(call gb_Helper_execute,localize) 
$(SRCDIR) $(dir $@)/pot \
+find $(dir $@)/pot -type f -printf %P\n | sed -e 
s/\.pot/.po/  $(dir $@)/LIST \
+touch $@)
+
 # enable if: no -MODULE/ defined AND [all defined OR MODULE/ defined]
 gb_Module__debug_enabled = \
  $(and $(if $(filter -$(1)/,$(ENABLE_DEBUGINFO_FOR)),,$(true)),\
@@ -227,7 +238,9 @@ endif
 
 endef
 
-ifneq (showmodules,$(MAKECMDGOALS))
+gb_FULL_BUILD := $(if $(filter showmodules 
translations,$(MAKECMDGOALS)),$(false),$(true))
+
+ifeq ($(gb_FULL_BUILD),$(true))
 define gb_Module_add_target
 $(call gb_Module__read_targetfile,$(1),$(2),target)
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] add rule for generating fresh pot (po template) files

2013-01-10 Thread Miklos Vajna (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/1622

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ife58bfa3ee607b06617d7db80408afb453b9a10c
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Matúš Kukan matus.ku...@gmail.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


[PUSHED] Kill the ancient StarOffice patch concept

2013-01-10 Thread Miklos Vajna (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/1605

Approvals:
  Miklos Vajna: Verified
  Michael Stahl: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I09e949e601a969f88eff60067faa2352f4f89537
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Tor Lillqvist t...@iki.fi
Gerrit-Reviewer: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


[Libreoffice-commits] .: slideshow/README

2013-01-10 Thread Libreoffice Gerrit user
 slideshow/README |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b3d5e78dd78390513021b46bc44036cdde2f1b45
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Jan 10 11:38:19 2013 +0200

It's not that new any longer;)

Change-Id: Ide189c5f3ace0a5a7478ae1a70c315fe3e461667

diff --git a/slideshow/README b/slideshow/README
index 54a1994..f6d19bb 100644
--- a/slideshow/README
+++ b/slideshow/README
@@ -1 +1 @@
-Contains the new impress slideshow engine introduced in OO.o 2.0
+The Impress slideshow engine
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - odk/cfgWin.js

2013-01-10 Thread Libreoffice Gerrit user
 odk/cfgWin.js |   50 +-
 1 file changed, 1 insertion(+), 49 deletions(-)

New commits:
commit fc2b654da5482f6d8c202c9fd30e1a1812c5b8c2
Author: Stephan Bergmann sberg...@redhat.com
Date:   Wed Jan 9 17:45:09 2013 +0100

Fix oo_sdk_ure_home calculation in SDK's cfgWin.js

This appears to be broken at least since
bde5b815873450dc9520c4dd300829eb5bd72daa INTEGRATION: CWS jsc21 (1.11.2); 
FILE
MERGED: 2008/05/27 16:59:34 jsc 1.11.2.1: #i88797# adapt for new structure?

Change-Id: I6385ca10cdded13e6d1aaf08d353f5029843d38d
(cherry picked from commit 0c22641b4c44938924187ddfb945f494a183ba47)
Reviewed-on: https://gerrit.libreoffice.org/1620
Reviewed-by: Tor Lillqvist t...@iki.fi
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/odk/cfgWin.js b/odk/cfgWin.js
index b6dd4e7..fa1c081 100644
--- a/odk/cfgWin.js
+++ b/odk/cfgWin.js
@@ -53,7 +53,7 @@ var oo_user_sdk_dir=WshSysEnv(APPDATA) + \\ + oo_sdk_name;
 var oo_user_sdk_env_script=oo_user_sdk_dir + \\setsdkenv_windows.bat;
 
 var office_home=getOfficeHome();
-var oo_sdk_ure_home=getUreHome();
+var oo_sdk_ure_home=office_home + \\URE;
 
 var oo_sdk_make_home=getMakeHome();
 var oo_sdk_zip_home=getZipHome();
@@ -140,23 +140,6 @@ function getSdkHome()
 }   
 }
 
-function getOfficeOrUre()
-{
-var suggestion = Office;
-var choice;
-while (choice != office  choice != ure) {
-stdout.Write(
-\n Use an installed Office or an installed UNO Runtime
-+  Environment (Office/URE) [ + suggestion + ]:);
-choice = stdin.ReadLine();
-if (choice == ) {
-choice = suggestion;
-}
-choice = choice.toLowerCase();
-}
-return choice;
-}
-
 function getOfficeHome()
 {
 var sSuggestedHome = WshSysEnv(OFFICE_HOME);
@@ -239,37 +222,6 @@ function searchOffice()
return ;
 }
 
-
-function getUreHome()
-{
-   var tmpure = oo_sdk_home;
-   var ure = ;
-
-if (aFileSystemObject.FileExists(tmpure + \\URE\\bin\\uno.exe)) {
-  ure = tmpure + \URE;
-   }
-
-   return ure;
-
-/*
-var suggestion = WshSysEnv(OO_SDK_URE_HOME);
-var choice;
-for (;;) {
-stdout.Write(
-\n Enter the URE installation directory [ + suggestion + ]:);
-choice = stdin.ReadLine();
-if (choice == ) {
-choice = suggestion;
-}
-if (aFileSystemObject.FileExists(choice + \\bin\\uno.exe)) {
-break;
-}
-stdout.WriteLine(\n Error: A valid URE installation is required.);
-}
-return choice;
-*/
-}
-
 function getMakeHome()
 {
 var sSuggestedHome = WshSysEnv(OO_SDK_MAKE_HOME);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-4-0]: Fix oo_sdk_ure_home calculation in SDK's cfgWin.js

2013-01-10 Thread Miklos Vajna (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/1620

Approvals:
  Tor Lillqvist: Looks good to me, approved
  Miklos Vajna: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6385ca10cdded13e6d1aaf08d353f5029843d38d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Tor Lillqvist t...@iki.fi

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


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

2013-01-10 Thread Libreoffice Gerrit user
 sfx2/source/control/thumbnailview.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 96633bf2d11b375a608240d1c1e83fd6e627fd75
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Wed Jan 9 10:57:44 2013 +0100

Template Manager: double click on the whole item instead of only thumbnail

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

diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index c3957f0..62324a7 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -517,7 +517,6 @@ void ThumbnailView::MouseButtonDown( const MouseEvent 
rMEvt )
 maItemStateHdl.Call(pItem);
 
 Rectangle aRect(pItem-getDrawArea());
-aRect.SetSize(Size(mnItemWidth,mnThumbnailHeight));
 
 if (aRect.IsInside(rMEvt.GetPosPixel()))
 OnItemDblClicked(pItem);
___
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

2013-01-10 Thread Libreoffice Gerrit user
 sfx2/source/doc/templatedlg.cxx |   18 --
 1 file changed, 8 insertions(+), 10 deletions(-)

New commits:
commit 62dc5dfcf2af93a35beb96bebe63b7e8ac652fb5
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Wed Jan 9 11:58:39 2013 +0100

Template Manager: don't deselect item after setting default template

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

diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 282372d..bff2dc1 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -1069,19 +1069,17 @@ void SfxTemplateManagerDlg::OnTemplateDelete ()
 
 void SfxTemplateManagerDlg::OnTemplateAsDefault ()
 {
-assert(!maSelTemplates.empty());
-
-const TemplateViewItem *pItem = static_castconst 
TemplateViewItem*(*(maSelTemplates.begin()));
-
-OUString aServiceName;
-if (lcl_getServiceName(pItem-getPath(),aServiceName))
+if (!maSelTemplates.empty())
 {
-SfxObjectFactory::SetStandardTemplate(aServiceName,pItem-getPath());
+const TemplateViewItem *pItem = static_castconst 
TemplateViewItem*(*(maSelTemplates.begin()));
 
-createDefaultTemplateMenu();
+OUString aServiceName;
+if (lcl_getServiceName(pItem-getPath(),aServiceName))
+{
+
SfxObjectFactory::SetStandardTemplate(aServiceName,pItem-getPath());
 
-// clear selection and display view/action toolbars
-maView-deselectOverlayItem(pItem-mnId);
+createDefaultTemplateMenu();
+}
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-4-0]: Template Manager: double click on the whole item instead of ...

2013-01-10 Thread Miklos Vajna (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/1610

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie07d41e5dcb09977d89752e724fcbabd6c5d7df1
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


[PUSHED] Change in core[libreoffice-4-0]: Template Manager: don't deselect item after setting default ...

2013-01-10 Thread Miklos Vajna (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/1611

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I860fb2c91d6843a1d5345ab2cf9270473f73f5bc
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric cedric.bosdon...@free.fr
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


RE: [libreoffice-dev] - questions about calc extension

2013-01-10 Thread Michael Meeks

On Wed, 2013-01-09 at 23:18 +, Rai, Neeraj wrote:
 I looked a little more into your code and copied the getactiveSheet function. 
 Now it takes 12s. I am happy.
 Sorry for the spam - you have good code sitting there.

12s sounds like a long time to me ;- If you can run your macro under
callgrind:

export OOO_DISABLE_RECOVERY=1
valgrind --tool=callgrind --simulate-cache=yes  \
 --dump-instr=yes ./soffice.bin -writer --splash-pipe=0

(needs write-permission to the installed libreoffice directory to drop
it's callgrind.12345.txt file)

Then perhaps kcachegrind will show you something interesting for your
use-case.

HTH,

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] .: Branch 'libreoffice-3-6' - odk/cfgWin.js

2013-01-10 Thread Libreoffice Gerrit user
 odk/cfgWin.js |   50 +-
 1 file changed, 1 insertion(+), 49 deletions(-)

New commits:
commit 89964f8e4f439bcd4b023ef8fd45bfacff01bd5d
Author: Stephan Bergmann sberg...@redhat.com
Date:   Wed Jan 9 17:45:09 2013 +0100

Fix oo_sdk_ure_home calculation in SDK's cfgWin.js

This appears to be broken at least since
bde5b815873450dc9520c4dd300829eb5bd72daa INTEGRATION: CWS jsc21 (1.11.2); 
FILE
MERGED: 2008/05/27 16:59:34 jsc 1.11.2.1: #i88797# adapt for new structure?

Change-Id: I6385ca10cdded13e6d1aaf08d353f5029843d38d
(cherry picked from commit 0c22641b4c44938924187ddfb945f494a183ba47)
Reviewed-on: https://gerrit.libreoffice.org/1619
Reviewed-by: Tor Lillqvist t...@iki.fi
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/odk/cfgWin.js b/odk/cfgWin.js
index 81fed05..f333a85 100644
--- a/odk/cfgWin.js
+++ b/odk/cfgWin.js
@@ -36,7 +36,7 @@ var oo_user_sdk_dir=WshSysEnv(APPDATA) + \\ + oo_sdk_name;
 var oo_user_sdk_env_script=oo_user_sdk_dir + \\setsdkenv_windows.bat;
 
 var office_home=getOfficeHome();
-var oo_sdk_ure_home=getUreHome();
+var oo_sdk_ure_home=office_home + \\URE;
 
 var oo_sdk_make_home=getMakeHome();
 var oo_sdk_zip_home=getZipHome();
@@ -123,23 +123,6 @@ function getSdkHome()
 }   
 }
 
-function getOfficeOrUre()
-{
-var suggestion = Office;
-var choice;
-while (choice != office  choice != ure) {
-stdout.Write(
-\n Use an installed Office or an installed UNO Runtime
-+  Environment (Office/URE) [ + suggestion + ]:);
-choice = stdin.ReadLine();
-if (choice == ) {
-choice = suggestion;
-}
-choice = choice.toLowerCase();
-}
-return choice;
-}
-
 function getOfficeHome()
 {
 var sSuggestedHome = WshSysEnv(OFFICE_HOME);
@@ -222,37 +205,6 @@ function searchOffice()
return ;
 }
 
-
-function getUreHome()
-{
-   var tmpure = oo_sdk_home;
-   var ure = ;
-
-if (aFileSystemObject.FileExists(tmpure + \\URE\\bin\\uno.exe)) {
-  ure = tmpure + \URE;
-   }
-
-   return ure;
-
-/*
-var suggestion = WshSysEnv(OO_SDK_URE_HOME);
-var choice;
-for (;;) {
-stdout.Write(
-\n Enter the URE installation directory [ + suggestion + ]:);
-choice = stdin.ReadLine();
-if (choice == ) {
-choice = suggestion;
-}
-if (aFileSystemObject.FileExists(choice + \\bin\\uno.exe)) {
-break;
-}
-stdout.WriteLine(\n Error: A valid URE installation is required.);
-}
-return choice;
-*/
-}
-
 function getMakeHome()
 {
 var sSuggestedHome = WshSysEnv(OO_SDK_MAKE_HOME);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-3-6]: Fix oo_sdk_ure_home calculation in SDK's cfgWin.js

2013-01-10 Thread Miklos Vajna (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/1619

Approvals:
  Tor Lillqvist: Looks good to me, approved
  Miklos Vajna: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6385ca10cdded13e6d1aaf08d353f5029843d38d
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Tor Lillqvist t...@iki.fi

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


Re: FYI: Make for faster windows build

2013-01-10 Thread Michael Meeks

On Wed, 2013-01-09 at 20:01 +0100, Lubos Lunak wrote:
 Looks what I found under the tree! Faster make for Cygwin. Granted, I had put 
 it there first, but still nice.

Nice work ! :-)

I'd love to see a configure option: --with-custom-gnumake that does the
git clone, pre-builds, and uses our own gnumake (with the speedups). Did
you have any joy getting the fixes up-stream to gnumake ? ;-)

Thanks !

Michael.

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

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


Need confirmation about non-clearing of Character Styles

2013-01-10 Thread Fernand Vanrie

Sinds all or editors use LO 3.6.2 (before OO3.3)

We have complains about the fact that parts of the text, who are 
formated with a Character Style, can not been cleared to no-formating,

not by Clear formating
not by Clear Direct Formating (cntrl+M)

There is a dirty work-around by applying the Default character style

Paragraph Styles can been cleared
Is this intentional behavior or a regression open for a issue ?

Greetz

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


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - sd/source

2013-01-10 Thread Libreoffice Gerrit user
 sd/source/ui/slidesorter/controller/SlideSorterController.cxx |5 +++--
 sd/source/ui/slidesorter/inc/controller/SlideSorterController.hxx |2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 72980e72a2d380d5e5a9c7b56a8df0a3ac1e93d0
Author: Caolán McNamara caol...@redhat.com
Date:   Wed Aug 22 11:46:50 2012 +0100

Resolves: fdo#45911 rhbz#846775 Clipboard must be disposed before Selection

(cherry picked from commit 7887a4e0adb0d60e6b013808dc27eccddd1c0324)

Change-Id: I968bc6da85cd444d504b08f85300a1fe251cdeee
Reviewed-on: https://gerrit.libreoffice.org/1612
Reviewed-by: Tor Lillqvist t...@iki.fi
Tested-by: Tor Lillqvist t...@iki.fi

diff --git a/sd/source/ui/slidesorter/controller/SlideSorterController.cxx 
b/sd/source/ui/slidesorter/controller/SlideSorterController.cxx
index 2932e20..7ab51b4 100644
--- a/sd/source/ui/slidesorter/controller/SlideSorterController.cxx
+++ b/sd/source/ui/slidesorter/controller/SlideSorterController.cxx
@@ -110,10 +110,10 @@ SlideSorterController::SlideSorterController 
(SlideSorter rSlideSorter)
   mpPageSelector(),
   mpFocusManager(),
   mpSlotManager(),
-  mpClipboard(),
   mpScrollBarManager(),
   mpCurrentSlideManager(),
   mpSelectionManager(),
+  mpClipboard(),
   mpInsertionIndicatorHandler(new InsertionIndicatorHandler(rSlideSorter)),
   mpAnimator(new Animator(rSlideSorter)),
   mpVisibleAreaManager(new VisibleAreaManager(rSlideSorter)),
@@ -156,9 +156,9 @@ void SlideSorterController::Init (void)
 mpPageSelector.reset(new PageSelector(mrSlideSorter));
 mpFocusManager.reset(new FocusManager(mrSlideSorter));
 mpSlotManager.reset(new SlotManager(mrSlideSorter));
-mpClipboard.reset(new Clipboard(mrSlideSorter));
 mpScrollBarManager.reset(new ScrollBarManager(mrSlideSorter));
 mpSelectionManager.reset(new SelectionManager(mrSlideSorter));
+mpClipboard.reset(new Clipboard(mrSlideSorter));
 
 mpScrollBarManager-LateInitialization();
 
@@ -202,6 +202,7 @@ SlideSorterController::~SlideSorterController (void)
 void SlideSorterController::Dispose (void)
 {
 mpInsertionIndicatorHandler-End(Animator::AM_Immediate);
+mpClipboard.reset();
 mpSelectionManager.reset();
 mpAnimator-Dispose();
 }
diff --git a/sd/source/ui/slidesorter/inc/controller/SlideSorterController.hxx 
b/sd/source/ui/slidesorter/inc/controller/SlideSorterController.hxx
index e962623..c515480 100644
--- a/sd/source/ui/slidesorter/inc/controller/SlideSorterController.hxx
+++ b/sd/source/ui/slidesorter/inc/controller/SlideSorterController.hxx
@@ -245,10 +245,10 @@ private:
 ::boost::scoped_ptrPageSelector mpPageSelector;
 ::boost::scoped_ptrFocusManager mpFocusManager;
 ::boost::shared_ptrSlotManager mpSlotManager;
-::boost::scoped_ptrcontroller::Clipboard mpClipboard;
 ::boost::scoped_ptrScrollBarManager mpScrollBarManager;
 mutable ::boost::shared_ptrCurrentSlideManager mpCurrentSlideManager;
 ::boost::shared_ptrSelectionManager mpSelectionManager;
+::boost::scoped_ptrcontroller::Clipboard mpClipboard;
 ::boost::shared_ptrInsertionIndicatorHandler mpInsertionIndicatorHandler;
 ::boost::shared_ptrAnimator mpAnimator;
 ::boost::scoped_ptrVisibleAreaManager mpVisibleAreaManager;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: config_host.mk.in configure.ac solenv/gbuild

2013-01-10 Thread Libreoffice Gerrit user
 config_host.mk.in   |1 +
 configure.ac|   19 +++
 solenv/gbuild/gbuild.mk |4 
 3 files changed, 24 insertions(+)

New commits:
commit 6af200ec0cbc78fee7b835de06a1588c7645fdc7
Author: Tomas Chvatal tchva...@suse.cz
Date:   Mon Dec 10 10:41:35 2012 +0100

Add configure switch that allows hardlink deliver

This switch just makes possible to configure in ln delivery instead of
cp based one.

The feature was already in the place, we just make it visible as
configure option now.

Change-Id: I778ef3b03e6b9a57057e66ff6307e7da46bcd935

diff --git a/config_host.mk.in b/config_host.mk.in
index 1c98744..55d0fa0 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -218,6 +218,7 @@ export GTK_PRINT_LIBS=$(gb_SPACE)@GTK_PRINT_LIBS@
 export GUI=@GUI@
 export GUIBASE=@GUIBASE@
 export GUI_FOR_BUILD=@GUI_FOR_BUILD@
+export HARDLINKDELIVER=@HARDLINKDELIVER@
 export HAVE_CXX0X=@HAVE_CXX0X@
 export HAVE_GCC_AVX=@HAVE_GCC_AVX@
 export HAVE_GCC_BUILTIN_ATOMIC=@HAVE_GCC_BUILTIN_ATOMIC@
diff --git a/configure.ac b/configure.ac
index 2fa044f..688f2e4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -678,6 +678,12 @@ AC_ARG_ENABLE(ext-wiki-publisher,
 
 dnl -- *** --
 
+AC_ARG_ENABLE([hardlink-deliver],
+AS_HELP_STRING([--enable-hardlink-deliver],
+[Put files into deliver folder as hardlinks instead of copying them
+over. Saves space and speeds up build.])
+)
+
 AC_ARG_ENABLE(mergelibs,
 AS_HELP_STRING([--enable-mergelibs],
 [Enables linking of big, merged, library. Experimental feature tested
@@ -12240,6 +12246,19 @@ else
 fi
 AC_SUBST(MERGELIBS)
 
+# ===
+# Create hardlinks on deliver instead of copying for smaller size and speed up
+# ===
+AC_MSG_CHECKING([whether to create hardlinks for delivering files])
+HARDLINKDELIVER=
+if test $enable_hardlink_deliver = yes; then
+HARDLINKDELIVER=TRUE
+AC_MSG_RESULT([yes])
+else
+AC_MSG_RESULT([no])
+fi
+AC_SUBST(HARDLINKDELIVER)
+
 dnl ===
 dnl icerun is a wrapper that stops us spawning tens of processes
 dnl locally - for tools that can't be executed on the compile cluster
diff --git a/solenv/gbuild/gbuild.mk b/solenv/gbuild/gbuild.mk
index d7b804d..eebcfe9 100644
--- a/solenv/gbuild/gbuild.mk
+++ b/solenv/gbuild/gbuild.mk
@@ -133,6 +133,10 @@ ENABLE_DEBUGINFO_FOR := all
 endif
 endif
 
+ifeq ($(HARDLINKDELIVER),TRUE)
+gb_Deliver_HARDLINK := $(true)
+endif
+
 ifeq ($(or $(ENABLE_SYMBOLS),$(enable_symbols)),FALSE)
 gb_SYMBOL := $(false)
 else
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - svx/source

2013-01-10 Thread Libreoffice Gerrit user
 svx/source/dialog/docrecovery.cxx |2 +-
 svx/source/inc/docrecovery.hxx|2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 26b0cacbceb39974fee8c974edb73c3428d9b137
Author: Caolán McNamara caol...@redhat.com
Date:   Tue Jan 8 14:04:09 2013 +

Resolves: fdo#58589 recovery dialog title area text is missing

regression since efc6b840a0423099d6c3e0d33c14d34637fbba4c
as that changes the order in which windows are drawn

Change-Id: I37388229533127ab915bec351f094fcd4786b2a3
(cherry picked from commit f8a4ce67987e5cb47f07dc61eeb450e1d5b9c541)
Reviewed-on: https://gerrit.libreoffice.org/1607
Reviewed-by: Tor Lillqvist t...@iki.fi
Tested-by: Tor Lillqvist t...@iki.fi

diff --git a/svx/source/dialog/docrecovery.cxx 
b/svx/source/dialog/docrecovery.cxx
index 4c0c96a..412c938 100644
--- a/svx/source/dialog/docrecovery.cxx
+++ b/svx/source/dialog/docrecovery.cxx
@@ -998,8 +998,8 @@ short impl_askUserForWizardCancel(Window* pParent, 
sal_Int16 nRes)
 RecoveryDialog::RecoveryDialog(Window*   pParent,
RecoveryCore* pCore  )
 : IExtendedTabPage( pParent  , SVX_RES( 
RID_SVXPAGE_DOCRECOVERY_RECOVER ) )
-, m_aTitleWin   ( this   , SVX_RES  ( WIN_RECOV_TITLE  
  ) )
 , m_aTitleFT( this   , SVX_RES  ( FT_RECOV_TITLE   
  ) )
+, m_aTitleWin   ( this   , SVX_RES  ( WIN_RECOV_TITLE  
  ) )
 , m_aTitleFL( this   , SVX_RES  ( FL_RECOV_TITLE   
  ) )
 , m_aDescrFT( this   , SVX_RES  ( FT_RECOV_DESCR   
  ) )
 , m_aProgressFT ( this   , SVX_RES  ( FT_RECOV_PROGR   
  ) )
diff --git a/svx/source/inc/docrecovery.hxx b/svx/source/inc/docrecovery.hxx
index 1159f01..9eeb3c3 100644
--- a/svx/source/inc/docrecovery.hxx
+++ b/svx/source/inc/docrecovery.hxx
@@ -631,8 +631,8 @@ class RecoveryDialog : public IExtendedTabPage
 //---
 // member
 private:
-Window  m_aTitleWin;
 FixedText   m_aTitleFT;
+Window  m_aTitleWin;
 FixedLine   m_aTitleFL;
 FixedText   m_aDescrFT;
 FixedText   m_aProgressFT;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-3-6]: Resolves: fdo#58589 recovery dialog title area text is missi...

2013-01-10 Thread Miklos Vajna (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/1607

Approvals:
  Tor Lillqvist: Verified; Looks good to me, approved
  Miklos Vajna: 


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I37388229533127ab915bec351f094fcd4786b2a3
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Tor Lillqvist t...@iki.fi

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


[PUSHED] Change in core[libreoffice-3-6]: Resolves: fdo#45911 rhbz#846775 Clipboard must be disposed b...

2013-01-10 Thread Miklos Vajna (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/1612

Approvals:
  Tor Lillqvist: Verified; Looks good to me, approved
  Miklos Vajna: 


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I968bc6da85cd444d504b08f85300a1fe251cdeee
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Tor Lillqvist t...@iki.fi

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


[PUSHED] Add configure switch that allows hardlink deliver

2013-01-10 Thread Petr Mladek (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/1283


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I778ef3b03e6b9a57057e66ff6307e7da46bcd935
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Tomáš Chvátal tchva...@suse.cz
Gerrit-Reviewer: Luboš Luňák l.lu...@suse.cz
Gerrit-Reviewer: Norbert Thiebaud nthieb...@gmail.com
Gerrit-Reviewer: Petr Mladek pmla...@suse.cz

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


[Libreoffice-commits] .: sfx2/source

2013-01-10 Thread Libreoffice Gerrit user
 sfx2/source/appl/appuno.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 4f6cd9a3dc5e83e7394dfb35161abb94699e59fb
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Thu Jan 10 10:52:18 2013 +0100

fdo#58328: fixed items types made versioning work again

Change-Id: Ib74f8f1a79ca04c42db9bd2c35ae6c4729edad69

diff --git a/sfx2/source/appl/appuno.cxx b/sfx2/source/appl/appuno.cxx
index 2e6ba66..fb16aad 100644
--- a/sfx2/source/appl/appuno.cxx
+++ b/sfx2/source/appl/appuno.cxx
@@ -130,8 +130,8 @@ SfxFormalArgument aFormalArgs[] = {
 SFX_ARGUMENT(SID_DEFAULTFILENAME,SuggestedSaveAsName,SfxStringItem),
 SFX_ARGUMENT(SID_DEFAULTFILEPATH,SuggestedSaveAsDir,SfxStringItem),
 SFX_ARGUMENT(SID_DOCINFO_AUTHOR,VersionAuthor,SfxStringItem),
-SFX_ARGUMENT(SID_DOCINFO_COMMENTS,VersionComment,SfxBoolItem),
-SFX_ARGUMENT(SID_DOCINFO_MAJOR,VersionMajor,SfxStringItem),
+SFX_ARGUMENT(SID_DOCINFO_COMMENTS,VersionComment,SfxStringItem),
+SFX_ARGUMENT(SID_DOCINFO_MAJOR,VersionMajor,SfxBoolItem),
 SFX_ARGUMENT(SID_FILE_FILTEROPTIONS,FilterOptions,SfxStringItem),
 SFX_ARGUMENT(SID_FILTER_NAME,FilterName,SfxStringItem),
 //SFX_ARGUMENT(SID_FILE_NAME,FileName,SfxStringItem),
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2013-01-10 Thread Libreoffice Gerrit user
 writerfilter/source/dmapper/DomainMapper_Impl.cxx |8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

New commits:
commit db67dad3065e3578f1321822ffb63eef83fe5377
Author: Michael Stahl mst...@redhat.com
Date:   Thu Jan 10 00:43:12 2013 +0100

fdo#58327: writerfilter: RemoveLastParagraph is tricky:

What happens here is that the xCursor-goLeft( 1, true ) selects the
entire inserted table, which is then deleted.  That causes annoyances,
such as an assertion from ~SwIndexReg because as collateral damage the
SwTxtNode that SwReader::Read's pUndoPam points to is also deleted, as
well as missing tables in the document.

DomainMapper_Impl::RemoveLastParagraph() can avoid this by not using
setString() but instead disposing the paragraph, like it already does in
other cases.  But don't do it for new (loaded) documents, because it may
kill flys in that case, which i strangely couldn't reproduce when
pasting as RTF.

Change-Id: I87302b8916e39ca42bc4fedfee40ac38fadd0939
(cherry picked from commit e521930ea1c855c236efb67793e540d07c201d35)

Signed-off-by: Miklos Vajna vmik...@suse.cz

diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx 
b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
index ef16af5..f8730b9 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
@@ -283,7 +283,13 @@ void DomainMapper_Impl::RemoveLastParagraph( )
 // Keep the character properties of the last but one paragraph, even if
 // it's empty. This works for headers/footers, and maybe in other cases
 // as well, but surely not in textboxes.
-if (m_bInHeaderFooterImport  xEnumerationAccess.is())
+// fdo#58327: also do this at the end of the document: when pasting,
+// a table before the cursor position would be deleted
+// (but only for paste/insert, not load; otherwise it can happen that
+// flys anchored at the disposed paragraph are deleted (fdo47036.rtf))
+bool const bEndOfDocument(m_aTextAppendStack.size() == 1);
+if ((m_bInHeaderFooterImport || (bEndOfDocument  !m_bIsNewDoc))
+ xEnumerationAccess.is())
 {
 uno::Referencecontainer::XEnumeration xEnumeration = 
xEnumerationAccess-createEnumeration();
 uno::Referencelang::XComponent 
xParagraph(xEnumeration-nextElement(), uno::UNO_QUERY);
___
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

2013-01-10 Thread Libreoffice Gerrit user
 sfx2/source/appl/appuno.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 6bfd791162fa392600ef8b8df6c1ac04bb4d4099
Author: Cédric Bosdonnat cedric.bosdon...@free.fr
Date:   Thu Jan 10 10:52:18 2013 +0100

fdo#58328: fixed items types made versioning work again

Change-Id: Ib74f8f1a79ca04c42db9bd2c35ae6c4729edad69
Signed-off-by: Michael Meeks michael.me...@suse.com

diff --git a/sfx2/source/appl/appuno.cxx b/sfx2/source/appl/appuno.cxx
index 2e6ba66..fb16aad 100644
--- a/sfx2/source/appl/appuno.cxx
+++ b/sfx2/source/appl/appuno.cxx
@@ -130,8 +130,8 @@ SfxFormalArgument aFormalArgs[] = {
 SFX_ARGUMENT(SID_DEFAULTFILENAME,SuggestedSaveAsName,SfxStringItem),
 SFX_ARGUMENT(SID_DEFAULTFILEPATH,SuggestedSaveAsDir,SfxStringItem),
 SFX_ARGUMENT(SID_DOCINFO_AUTHOR,VersionAuthor,SfxStringItem),
-SFX_ARGUMENT(SID_DOCINFO_COMMENTS,VersionComment,SfxBoolItem),
-SFX_ARGUMENT(SID_DOCINFO_MAJOR,VersionMajor,SfxStringItem),
+SFX_ARGUMENT(SID_DOCINFO_COMMENTS,VersionComment,SfxStringItem),
+SFX_ARGUMENT(SID_DOCINFO_MAJOR,VersionMajor,SfxBoolItem),
 SFX_ARGUMENT(SID_FILE_FILTEROPTIONS,FilterOptions,SfxStringItem),
 SFX_ARGUMENT(SID_FILTER_NAME,FilterName,SfxStringItem),
 //SFX_ARGUMENT(SID_FILE_NAME,FileName,SfxStringItem),
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 54157] LibreOffice 3.7/4.0 most annoying bugs

2013-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

Bug 54157 depends on bug 58328, which changed state.

Bug 58328 Summary: FILESAVE: Creating a new document version no longer working
https://bugs.freedesktop.org/show_bug.cgi?id=58328

   What|Removed |Added

 Status|NEW |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


[Bug 44446] LibreOffice 3.6 most annoying bugs

2013-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=6

Michael Meeks michael.me...@novell.com changed:

   What|Removed |Added

 Depends on||58819

-- 
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


[Bug 54157] LibreOffice 3.7/4.0 most annoying bugs

2013-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

Michael Meeks michael.me...@novell.com changed:

   What|Removed |Added

 Depends on|58819   |

-- 
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] .: CustomTarget_translate.mk .gitignore localization_present.mk Makefile Module_translations.mk prj/build.lst prj/d.lst prj/makefile.mk

2013-01-10 Thread Libreoffice Gerrit user
 .gitignore|   11 ---
 CustomTarget_translate.mk |   44 
 Makefile  |   11 ---
 Module_translations.mk|   38 --
 localization_present.mk   |1 -
 prj/build.lst |2 --
 prj/makefile.mk   |1 -
 7 files changed, 108 deletions(-)

New commits:
commit 620e5cb6954b4fdb1be1cf38b83844ae319600d8
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Tue Jan 8 22:20:14 2013 +0100

remove unused files, now that we don't build translations module

Rule for generating .pot files moved to core repo.

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

diff --git a/.gitignore b/.gitignore
deleted file mode 100644
index ebc0c42..000
--- a/.gitignore
+++ /dev/null
@@ -1,11 +0,0 @@
-# backup and temporary files
-*~
-.*.sw[op]
-
-# the build directories
-/unxlng??
-/unxlng??.pro
-/wntmsc???
-/wntmsc???.pro
-/unxmac??.pro
-/unxmac??
diff --git a/CustomTarget_translate.mk b/CustomTarget_translate.mk
deleted file mode 100644
index 562b6b3..000
--- a/CustomTarget_translate.mk
+++ /dev/null
@@ -1,44 +0,0 @@
-# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
-#
-# Version: MPL 1.1 / GPLv3+ / LGPLv3+
-#
-# The contents of this file are subject to the Mozilla Public License Version
-# 1.1 (the License); you may not use this file except in compliance with
-# the License or as specified alternatively below. You may obtain a copy of
-# the License at http://www.mozilla.org/MPL/
-#
-# Software distributed under the License is distributed on an AS IS basis,
-# WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License
-# for the specific language governing rights and limitations under the
-# License.
-#
-# Major Contributor(s):
-# Copyright (C) 2012 Matúš Kukan matus.ku...@gmail.com (initial developer)
-#
-# All Rights Reserved.
-#
-# For minor contributions see the git repository.
-#
-# Alternatively, the contents of this file may be used under the terms of
-# either the GNU General Public License Version 3 or later (the GPLv3+), or
-# the GNU Lesser General Public License Version 3 or later (the LGPLv3+),
-# in which case the provisions of the GPLv3+ or the LGPLv3+ are applicable
-# instead of those above.
-
-$(eval $(call gb_CustomTarget_CustomTarget,translations/translate))
-
-translations_DIR := $(call gb_CustomTarget_get_workdir,translations/translate)
-
-$(call gb_CustomTarget_get_target,translations/translate) : \
-   $(translations_DIR)/pot.done
-
-$(translations_DIR)/pot.done : $(foreach exec,cfgex helpex localize transex3 \
- propex uiex ulfex xrmex treex, \
-   $(call gb_Executable_get_target_for_build,$(exec)))
-   $(call gb_Output_announce,$(subst .pot,,$(subst 
$(WORKDIR)/,,$@)),$(true),POT,1)
-   $(call gb_Helper_abbreviate_dirs, \
-   mkdir -p $(dir $@)  $(call gb_Helper_execute,localize) 
$(SRCDIR) $(dir $@)/pot) \
-find $(dir $@)/pot -type f -printf %P\n | sed -e 
s/\.pot/.po/  $(dir $@)/LIST \
-touch $@
-
-# vim: set noet sw=4 ts=4:
diff --git a/Makefile b/Makefile
deleted file mode 100644
index 6b38ac5..000
--- a/Makefile
+++ /dev/null
@@ -1,11 +0,0 @@
-# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
-
-module_directory=$(dir $(realpath $(firstword $(MAKEFILE_LIST
-
-ifeq ($(origin SRC_ROOT),undefined)
-SRC_ROOT=$(module_directory)../../..
-endif
-
-include $(SRC_ROOT)/solenv/gbuild/partial_build.mk
-
-# vim: set noet sw=4 ts=4:
diff --git a/Module_translations.mk b/Module_translations.mk
deleted file mode 100644
index df8c839..000
--- a/Module_translations.mk
+++ /dev/null
@@ -1,38 +0,0 @@
-# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
-#
-# Version: MPL 1.1 / GPLv3+ / LGPLv3+
-#
-# The contents of this file are subject to the Mozilla Public License Version
-# 1.1 (the License); you may not use this file except in compliance with
-# the License or as specified alternatively below. You may obtain a copy of
-# the License at http://www.mozilla.org/MPL/
-#
-# Software distributed under the License is distributed on an AS IS basis,
-# WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License
-# for the specific language governing rights and limitations under the
-# License.
-#
-# Major Contributor(s):
-# Copyright (C) 2012 Matúš Kukan matus.ku...@gmail.com (initial developer)
-#
-# All Rights Reserved.
-#
-# For minor contributions see the git repository.
-#
-# Alternatively, the contents of this file may be used under the terms of
-# either the GNU General Public License Version 3 or later (the GPLv3+), or
-# the GNU Lesser General Public License Version 3 or later (the LGPLv3+),

[Libreoffice-commits] .: translations

2013-01-10 Thread Libreoffice Gerrit user
 translations |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit c85fa091125330f0764ac7b58b79a3b0dcef41ca
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Tue Jan 8 22:20:14 2013 +0100

Updated core
Project: translations  620e5cb6954b4fdb1be1cf38b83844ae319600d8

remove unused files, now that we don't build translations module

Rule for generating .pot files moved to core repo.

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

diff --git a/translations b/translations
index e5f69f7..620e5cb 16
--- a/translations
+++ b/translations
@@ -1 +1 @@
-Subproject commit e5f69f7b241f21f0b85f7a62ccff9de841875828
+Subproject commit 620e5cb6954b4fdb1be1cf38b83844ae319600d8
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[ANN] LibreOffice 4.0.0 RC1 test builds available

2013-01-10 Thread Fridrich Strba
Hi *,

for the upcoming new version 4.0.0, the RC1 builds now start to be
available on pre-releases. This build is slated to be first release
candidate build on the way towards 4.0.0, please refer to our release
plan timings here:

 http://wiki.documentfoundation.org/ReleasePlan#4.0_release

Builds are now being uploaded to a public (but non-mirrored - so don't
spread news too widely!) place, as soon as they're available. Grab
them here:

 http://dev-builds.libreoffice.org/pre-releases/

If you've a bit of time, please give them a try  report *critical*
bugs not yet in bugzilla here, so we can incorporate them into the
release notes. Please note that it takes approximately 24 hours to
populate the mirrors, so that's about the time we have to collect
feedback.

NOTE: This build is in a release configuration and _will_ replace your
existing LibreOffice install on Windows.

The list of fixed bugs relative to 3.6.3 is here:

 
http://dev-builds.libreoffice.org/pre-releases/src/bugs-libreoffice-4-0-release-4.0.0.1.log


So playing with the areas touched there also greatly appreciated - and
validation that those bugs are really fixed.

Thanks a lot for your help,
Fridrich

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


Re: Failure to build 3.6 branch

2013-01-10 Thread Alexander Thurgood
Le 09/01/13 23:19, Lubos Lunak a écrit :

Hi Lubos,

  Should be fixed now.
 

This still fails for me on Mac today with a fresh pull and complete
clean. First few lines of error message below :

/Users/Shared/Repos/LO/core/unotools/source/misc/fontcvt.cxx:1535:
internal compiler error: Segmentation fault: 11
Please submit a full bug report,
with preprocessed source if appropriate.
See URL:http://developer.apple.com/bugreporter for instructions.
{standard input}:4636:non-relocatable subtraction expression,
L__ZTV23StarSymbolToMSMultiFont$non_lazy_ptr minus L011$pb
{standard input}:4636:symbol:
L__ZTV23StarSymbolToMSMultiFont$non_lazy_ptr can't be undefined in a
subtraction expression
{standard input}:1842:non-relocatable subtraction expression,
L__ZTV23StarSymbolToMSMultiFont$non_lazy_ptr minus L004$pb
{standard input}:1842:symbol:
L__ZTV23StarSymbolToMSMultiFont$non_lazy_ptr can't be undefined in a
subtraction expression
{standard input}:1814:non-relocatable subtraction expression,
L__ZTV23StarSymbolToMSMultiFont$non_lazy_ptr minus L003$pb
{standard input}:1814:symbol:
L__ZTV23StarSymbolToMSMultiFont$non_lazy_ptr can't be undefined in a
subtraction expression
{standard input}:8747:non-relocatable subtraction expression,
L___gxx_personality_v0$non_lazy_ptr minus L0
{standard input}:8747:symbol: L___gxx_personality_v0$non_lazy_ptr
can't be undefined in a subtraction expression
{standard input}:unknown:Undefined local symbol
L__ZTV23StarSymbolToMSMultiFont$non_lazy_ptr
{standard input}:unknown:Undefined local symbol
L___gxx_personality_v0$non_lazy_ptr
{standard input}:unknown:Undefined local symbol LASF0
{standard input}:unknown:Undefined local symbol LASF1
{standard input}:unknown:Undefined local symbol LASF2
{standard input}:unknown:Undefined local symbol LASF3
{standard input}:unknown:Undefined local symbol LASF4...ad nauseam



Alex

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


Re: Failure to build 3.6 branch

2013-01-10 Thread Alexander Thurgood
Le 09/01/13 23:19, Lubos Lunak a écrit :

Ooops, forgot to mention that I'm building from master, sorry.

Alex




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


[Libreoffice-commits] .: dictionaries

2013-01-10 Thread Libreoffice Gerrit user
 dictionaries |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit bcd01e43810272e7219c44bafb3d40e6914c911d
Author: Petr Mladek pmla...@suse.cz
Date:   Thu Jan 10 11:44:14 2013 +0100

Add Swedish hyphen dictionary provided by Niklas Johansson

Change-Id: I0a425c066726e0364dce414f622106e954aa29d3

diff --git a/dictionaries b/dictionaries
index f310b73..9cfd428 16
--- a/dictionaries
+++ b/dictionaries
@@ -1 +1 @@
-Subproject commit f310b731bbbaf968b513bea1dd460e5d96b0c61c
+Subproject commit 9cfd4282a7a522bed2dc4f4b9ede2c8ee444c29b
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .:

2013-01-10 Thread Libreoffice Gerrit user
 0 files changed

New commits:
commit fbd9b4b9977fc12d826a00f6c4b2c4563eb74ff7
Author: Petr Mladek pmla...@suse.cz
Date:   Thu Jan 10 11:28:41 2013 +0100

Updated core
Project: dictionaries  9cfd4282a7a522bed2dc4f4b9ede2c8ee444c29b
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'feature/androidapp' - vcl/android

2013-01-10 Thread Libreoffice Gerrit user
 vcl/android/androidinst.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b5956d061735306bd9b49d7aa99f3ecc98efa189
Author: Michael Meeks michael.me...@suse.com
Date:   Thu Jan 10 11:14:33 2013 +

increase random default window size.

Change-Id: I960d11c6098681356fea0634970545aa9af9bacb

diff --git a/vcl/android/androidinst.cxx b/vcl/android/androidinst.cxx
index d7b5ea4..6bc7754 100644
--- a/vcl/android/androidinst.cxx
+++ b/vcl/android/androidinst.cxx
@@ -385,7 +385,7 @@ void AndroidSalInstance::GetWorkArea( Rectangle rRect )
 {
 if (!mpApp || !mpApp-window)
 rRect = Rectangle( Point( 0, 0 ),
-   Size( 800, 600 ) );
+   Size( 1280, 750 ) );
 else
 rRect = Rectangle( Point( 0, 0 ),
Size( ANativeWindow_getWidth( mpApp-window ),
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2013-01-10 Thread Libreoffice Gerrit user
 desktop/source/deployment/manager/dp_manager.cxx |2 +-
 scp2/source/ooo/common_brand.scp |8 
 scp2/source/ooo/profileitem_ooo.scp  |9 +
 3 files changed, 18 insertions(+), 1 deletion(-)

New commits:
commit 8dd405d32cb331818dc8f9bd1af3c5a074100f0f
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Jan 10 11:54:28 2013 +0100

fdo#58909: Missing BAK_EXTENSIONS bootstrap variable

daeed90f4586eb9533041fb89bee163a5193596c re-base on ALv2 code had 
incorporated
from http://svn.apache.org/viewvc?view=revisionrevision=1172102 imported
patch extensions_i117681.patch the parts using BAK_EXTENSIONS in
desktop/source/deployment/manager/, but not the parts setting 
BAK_EXTENSIONS in
scp2/source/ooo/.

Also, setting the stamp for bak in PackageManagerImpl::create needed to be
adapted to bee742eb7a0d5dfe23e61d9ee49a29286de90256 Fix sense of r/o 
detection
code, and clean up and 61eeb689d7605a23c3e71c652b57ee65cf5b28dc fix 
smoketest
- need to check read-only-ness of non-existent paths.

Change-Id: If7cc363b3cf21fd13fb467f5f672eda5312515c8

diff --git a/desktop/source/deployment/manager/dp_manager.cxx 
b/desktop/source/deployment/manager/dp_manager.cxx
index 89e46c9..87c107c 100644
--- a/desktop/source/deployment/manager/dp_manager.cxx
+++ b/desktop/source/deployment/manager/dp_manager.cxx
@@ -379,7 +379,7 @@ Referencedeployment::XPackageManager 
PackageManagerImpl::create(
 that-m_activePackages = 
vnd.sun.star.expand:$BAK_EXTENSIONS/extensions;
 that-m_registrationData = vnd.sun.star.expand:$BAK_EXTENSIONS;
 that-m_registryCache = vnd.sun.star.expand:$BAK_EXTENSIONS/registry;
-stamp = vnd.sun.star.expand:$BAK_EXTENSIONS/stamp.sys;
+stamp = $BAK_EXTENSIONS;
 }
 
 else if (! context.match(vnd.sun.star.tdoc:/)) {
diff --git a/scp2/source/ooo/common_brand.scp b/scp2/source/ooo/common_brand.scp
index eae79e7..025ad7f 100644
--- a/scp2/source/ooo/common_brand.scp
+++ b/scp2/source/ooo/common_brand.scp
@@ -1180,6 +1180,14 @@ ProfileItem 
gid_Brand_Profileitem_Fundamental_Uno_Tmp_Extensions
 Value = ${$ORIGIN/ PROFILENAME(uno) :TMP_EXTENSIONS};
 End
 
+ProfileItem gid_Brand_Profileitem_Fundamental_Uno_Bak_Extensions
+ModuleID = gid_Module_Root_Brand;
+ProfileID = gid_Brand_Profile_Fundamental_Ini;
+Section = Bootstrap;
+Key = BAK_EXTENSIONS;
+Value = ${$ORIGIN/ PROFILENAME(uno) :TMP_EXTENSIONS};
+End
+
 ProfileItem gid_Brand_Profileitem_Fundamental_Ure_More_Types
 ModuleID = gid_Module_Root_Brand;
 ProfileID = gid_Brand_Profile_Fundamental_Ini;
diff --git a/scp2/source/ooo/profileitem_ooo.scp 
b/scp2/source/ooo/profileitem_ooo.scp
index 69ca5c6..abe5df4 100644
--- a/scp2/source/ooo/profileitem_ooo.scp
+++ b/scp2/source/ooo/profileitem_ooo.scp
@@ -166,6 +166,15 @@ ProfileItem gid_Profileitem_Uno_Uno_Bundled_Tmp_Extensions
 Value = ${$BRAND_BASE_DIR/program/ PROFILENAME(bootstrap) 
:UserInstallation}/user/extensions/tmp;
 End
 
+ProfileItem gid_Profileitem_Uno_Uno_Bundled_Bak_Extensions
+ProfileID = gid_Profile_Uno_Ini;
+ModuleID = gid_Module_Root;
+Section = Bootstrap;
+Order = 1;
+Key = BAK_EXTENSIONS;
+Value = ${$BRAND_BASE_DIR/program/ PROFILENAME(bootstrap) 
:UserInstallation}/user/extensions/bak;
+End
+
 ProfileItem gid_Profileitem_Uno_Uno_User_Packages_Cache
 ProfileID = gid_Profile_Uno_Ini;
 ModuleID = gid_Module_Root;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - lingucomponent/source linguistic/inc linguistic/source linguistic/workben

2013-01-10 Thread Libreoffice Gerrit user
 lingucomponent/source/hyphenator/altlinuxhyph/hyphen/hyphenimp.cxx |8 +-
 lingucomponent/source/lingutil/lingutil.cxx|2 
 lingucomponent/source/spellcheck/macosxspell/macspellimp.mm|4 -
 lingucomponent/source/spellcheck/spell/sspellimp.cxx   |4 -
 lingucomponent/source/thesaurus/libnth/nthesimp.cxx|4 -
 linguistic/inc/linguistic/misc.hxx |   18 
 linguistic/source/convdic.cxx  |2 
 linguistic/source/convdiclist.cxx  |2 
 linguistic/source/dicimp.cxx   |7 +
 linguistic/source/dlistimp.cxx |6 -
 linguistic/source/gciterator.cxx   |6 -
 linguistic/source/hyphdsp.cxx  |   18 ++--
 linguistic/source/lngsvcmgr.cxx|   10 +-
 linguistic/source/misc.cxx |   38 
--
 linguistic/source/spelldsp.cxx |   14 +--
 linguistic/source/spelldta.cxx |4 -
 linguistic/source/thesdsp.cxx  |   10 +-
 linguistic/workben/sspellimp.cxx   |4 -
 18 files changed, 106 insertions(+), 55 deletions(-)

New commits:
commit f4078cb3a2366263227328a53e6cab4631e04bd6
Author: Eike Rathke er...@redhat.com
Date:   Wed Jan 9 16:48:52 2013 +0100

resolved fdo#58503 restore awkward handling of empty locale for legacy

Partly reverts d7a5ec62e91ce3dc5b784815254218f16181f676

An empty locale was treated as LANGUAGE_NONE, effectively being
interpreted as absence or undetermined or multiple or all depending on
context. Restore this behavior as it was an undocumented feature of the
API.

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

diff --git a/lingucomponent/source/hyphenator/altlinuxhyph/hyphen/hyphenimp.cxx 
b/lingucomponent/source/hyphenator/altlinuxhyph/hyphen/hyphenimp.cxx
index e7fe7a4..990b45b 100644
--- a/lingucomponent/source/hyphenator/altlinuxhyph/hyphen/hyphenimp.cxx
+++ b/lingucomponent/source/hyphenator/altlinuxhyph/hyphen/hyphenimp.cxx
@@ -485,13 +485,13 @@ Reference XHyphenatedWord  SAL_CALL 
Hyphenator::hyphenate( const ::rtl::OUStri
 sal_Int16 nPos = (sal_Int16) ((nHyphenationPosAltHyph  
nHyphenationPos) ?
 nHyphenationPosAltHyph : nHyphenationPos);
 // dicretionary hyphenation
-xRes = HyphenatedWord::CreateHyphenatedWord( aWord, 
LanguageTag( aLocale ).getLanguageType(), nPos,
+xRes = HyphenatedWord::CreateHyphenatedWord( aWord, 
LinguLocaleToLanguage( aLocale ), nPos,
 aWord.replaceAt(nHyphenationPosAlt + 1, 
cut[nHyphenationPos], repHyph),
 (sal_Int16) nHyphenationPosAltHyph);
 }
 else
 {
-xRes = HyphenatedWord::CreateHyphenatedWord( aWord, 
LanguageTag( aLocale ).getLanguageType(),
+xRes = HyphenatedWord::CreateHyphenatedWord( aWord, 
LinguLocaleToLanguage( aLocale ),
 (sal_Int16)nHyphenationPos, aWord, (sal_Int16) 
nHyphenationPos);
 }
 }
@@ -542,7 +542,7 @@ Reference XPossibleHyphens  SAL_CALL 
Hyphenator::createPossibleHyphens( const
 //well as hyphenate
 if (aWord.getLength()  minLen)
 {
-return PossibleHyphens::CreatePossibleHyphens( aWord, LanguageTag( 
aLocale ).getLanguageType(),
+return PossibleHyphens::CreatePossibleHyphens( aWord, 
LinguLocaleToLanguage( aLocale ),
   aWord, Sequence sal_Int16 () );
 }
 
@@ -689,7 +689,7 @@ Reference XPossibleHyphens  SAL_CALL 
Hyphenator::createPossibleHyphens( const
 OUString hyphenatedWord = hyphenatedWordBuffer.makeStringAndClear();
 
 Reference XPossibleHyphens  xRes = 
PossibleHyphens::CreatePossibleHyphens(
-aWord, LanguageTag( aLocale ).getLanguageType(), hyphenatedWord, 
aHyphPos);
+aWord, LinguLocaleToLanguage( aLocale ), hyphenatedWord, aHyphPos);
 
 delete[] hyphens;
 delete[] lcword;
diff --git a/lingucomponent/source/lingutil/lingutil.cxx 
b/lingucomponent/source/lingutil/lingutil.cxx
index 194f0b7..8754442 100644
--- a/lingucomponent/source/lingutil/lingutil.cxx
+++ b/lingucomponent/source/lingutil/lingutil.cxx
@@ -216,7 +216,7 @@ void MergeNewStyleDicsAndOldStyleDics(
 {
 LanguageType nLang = LanguageTag( aIt2-aLocaleNames[0] 
).getLanguageType();
 
-if (nLang == LANGUAGE_DONTKNOW || nLang == LANGUAGE_NONE)
+if (nLang == LANGUAGE_DONTKNOW || linguistic::LinguIsUnspecified( 
nLang))
 {
 OSL_FAIL( old 

[REVIEW 4-0] Re: Adding hyphenation patterns for Swedish

2013-01-10 Thread Petr Mladek
Hi Niklas,

I have added the dictionary into master, see
http://cgit.freedesktop.org/libreoffice/dictionaries/commit/?id=9cfd4282a7a522bed2dc4f4b9ede2c8ee444c29b

We need one review to get it into 4-0 branch.

Thanks for the nice improvement.

Best Regards,
Petr


Niklas Johansson píše v Po 07. 01. 2013 v 20:02 +0100:
 What would be the easiest way to add the hyphenation patterns bundled 
 with LibreOffice?
 I've updated the hyphenation extension on the LibreOffice extension site:
 http://extensions.libreoffice.org/extension-center/swedish-hyphenation
 
 Regards,
 Niklas Johansson
 
 Petr Mladek skrev 2012-12-03 12:25:
  Niklas Johansson píše v Ne 02. 12. 2012 v 21:07 +0100:
  I thought I would add Swedish hyphenation patterns to the next release
  of LibreOffice. The plan was to do it together with an update of the
  spelling dictionary that is planed in mid January.
  Sounds great.
 
  Since Swedish doesn't have any patterns bundled today, will there be
  any problems with the feature freeze? Or will it be possible to add it
  together with the update of the spelling dictionary?
  It should not be a problem. We are less strict in this area. I am not
  aware about any breakage or regression caused by updating or adding a
  dictionary.
 
  We are looking forward to update and add the dictionary.
 
 
  Best Regards,
  Petr
 
 
 


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


[PUSHED] Change in core[libreoffice-4-0]: resolved fdo#58503 restore awkward handling of empty locale ...

2013-01-10 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/1616


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I256b352961f09e41cadb3e44c9ef01ee7677fd31
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Eike Rathke er...@redhat.com
Gerrit-Reviewer: Stephan Bergmann sberg...@redhat.com

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


[Bug 44446] LibreOffice 3.6 most annoying bugs

2013-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=6

Michael Meeks michael.me...@novell.com changed:

   What|Removed |Added

 Depends on||45600

--- Comment #194 from Michael Meeks michael.me...@novell.com ---
Added - missing jump-list in Windows 7 - apparently a rather annoying
regression.

-- 
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


Re: Failure to build 3.6 branch

2013-01-10 Thread Caolán McNamara
On Thu, 2013-01-10 at 11:38 +0100, Alexander Thurgood wrote:
 /Users/Shared/Repos/LO/core/unotools/source/misc/fontcvt.cxx:1535:
 internal compiler error: Segmentation fault: 11
 Please submit a full bug report,

I wonder if that file has grown too big for your toolchain. Try
splitting up the file into two and see if you can get it to compile that
way.

C.

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


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

2013-01-10 Thread Libreoffice Gerrit user
 dictionaries |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 5af51534d0bca3fdf5b9758287ce9e23a89ff117
Author: Petr Mladek pmla...@suse.cz
Date:   Thu Jan 10 11:28:41 2013 +0100

Updated core
Project: dictionaries  5cf7cd4323ac7ff01c317951c6cb9d35f5647235

diff --git a/dictionaries b/dictionaries
index 7e895dc..5cf7cd4 16
--- a/dictionaries
+++ b/dictionaries
@@ -1 +1 @@
-Subproject commit 7e895dc439c87d229b7396076fdd8945b3b51f3f
+Subproject commit 5cf7cd4323ac7ff01c317951c6cb9d35f5647235
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 54157] LibreOffice 3.7/4.0 most annoying bugs

2013-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

Bug 54157 depends on bug 58909, which changed state.

Bug 58909 Summary: Extension upgrade fails with error Cannot create folder
https://bugs.freedesktop.org/show_bug.cgi?id=58909

   What|Removed |Added

 Status|NEW |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


Re: Failure to build 3.6 branch

2013-01-10 Thread Alexander Thurgood
Le 10/01/13 12:24, Caolán McNamara a écrit :

Hi Caolán,


 I wonder if that file has grown too big for your toolchain. Try
 splitting up the file into two and see if you can get it to compile that
 way.

Hmm, if I try and build just unotools, I get the following error when
building unotools_inc :
gb_Deliver-deliver : file does not exist in solver, and cannot be
delivered :
/Users/Shared/Repos/LO/core/solver/unxmacxi.pro/lib/libcomphelpgcc3.dylib


It appears that libcomphelpgcc3.dylib is in /solver/unxmacxi and not
unxmacxi.pro. Copying that file to unxmacxi and re-attempting make in
unotools then throws another error that libtllo.dylib does not exist in
solver. Sorry, but I'm a bit lost here.


Alex









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


[Libreoffice-commits] .: src/docrecord.py src/docstream.py

2013-01-10 Thread Miklos Vajna
 src/docrecord.py |   27 +++
 src/docstream.py |5 -
 2 files changed, 31 insertions(+), 1 deletion(-)

New commits:
commit 81b1156dba0f77787ad0171be85e4f5c59b751de
Author: Miklos Vajna vmik...@suse.cz
Date:   Thu Jan 10 12:32:25 2013 +0100

dump PlcfGram

diff --git a/src/docrecord.py b/src/docrecord.py
index 6dccd2d..d5742f7 100644
--- a/src/docrecord.py
+++ b/src/docrecord.py
@@ -2358,4 +2358,31 @@ class PlcfSpl(DOCDirStream, PLC):
 print '/aCP'
 print '/plcfSpl'
 
+class PlcfGram(DOCDirStream, PLC):
+The PlcfGram structure is a Plc structure whose data elements are 
GrammarSpls structures.
+def __init__(self, mainStream):
+DOCDirStream.__init__(self, 
mainStream.doc.getDirectoryStreamByName(1Table).bytes, mainStream = 
mainStream)
+PLC.__init__(self, mainStream.lcbPlcfGram, 2) # 2 is defined by 2.8.21
+self.pos = mainStream.fcPlcfGram
+self.size = mainStream.lcbPlcfGram
+
+def dump(self):
+print 'plcfGram type=PlcfGram offset=%d size=%d bytes' % 
(self.pos, self.size)
+offset = self.mainStream.fcMin
+pos = self.pos
+for i in range(self.getElements()):
+# aCp
+start = self.getuInt32(pos = pos)
+end = self.getuInt32(pos = pos + 4)
+print 'aCP index=%d start=%d end=%d' % (i, start, end)
+pos += 4
+
+# aGrammarSpls
+aGrammarSpls = SPLS(GrammarSpls, self, self.getOffset(self.pos, 
i))
+aGrammarSpls.dump()
+
+print 'transformed value=%s/' % 
self.quoteAttr(self.mainStream.retrieveText(offset + start, offset + end, 
logicalLength = True))
+print '/aCP'
+print '/plcfGram'
+
 # vim:set filetype=python shiftwidth=4 softtabstop=4 expandtab:
diff --git a/src/docstream.py b/src/docstream.py
index a690b8d..c5c9fcc 100644
--- a/src/docstream.py
+++ b/src/docstream.py
@@ -384,7 +384,7 @@ class WordDocumentStream(DOCDirStream):
 [fcPlcfAsumy],
 [lcbPlcfAsumy],
 [fcPlcfGram],
-[lcbPlcfGram],
+[lcbPlcfGram, self.handleLcbPlcfGram],
 [fcSttbListNames],
 [lcbSttbListNames],
 [fcSttbfUssr],
@@ -515,6 +515,9 @@ class WordDocumentStream(DOCDirStream):
 def handleLcbPlcfSpl(self):
 docrecord.PlcfSpl(self).dump()
 
+def handleLcbPlcfGram(self):
+docrecord.PlcfGram(self).dump()
+
 def dumpFibRgFcLcb97(self, name):
 print '%s type=FibRgFcLcb97 size=744 bytes' % name
 self.__dumpFibRgFcLcb97()
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 54157] LibreOffice 3.7/4.0 most annoying bugs

2013-01-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

Joren De Cuyper joren.libreoff...@telenet.be changed:

   What|Removed |Added

 Depends on|59141   |

-- 
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


Re: Failure to build 3.6 branch

2013-01-10 Thread Lubos Lunak
On Thursday 10 of January 2013, Alexander Thurgood wrote:
 Le 10/01/13 12:24, Caolán McNamara a écrit :

 Hi Caolán,

  I wonder if that file has grown too big for your toolchain. Try
  splitting up the file into two and see if you can get it to compile that
  way.

 Hmm, if I try and build just unotools, I get the following error when
 building unotools_inc :
 gb_Deliver-deliver : file does not exist in solver, and cannot be
 delivered :
 /Users/Shared/Repos/LO/core/solver/unxmacxi.pro/lib/libcomphelpgcc3.dylib

 make unotools.all

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


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

2013-01-10 Thread Libreoffice Gerrit user
 desktop/source/deployment/manager/dp_manager.cxx |2 +-
 scp2/source/ooo/common_brand.scp |8 
 scp2/source/ooo/profileitem_ooo.scp  |9 +
 3 files changed, 18 insertions(+), 1 deletion(-)

New commits:
commit 147ce2ebeeac686d0a8e9e3d6d804aeba0fcbcdc
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Jan 10 11:54:28 2013 +0100

fdo#58909: Missing BAK_EXTENSIONS bootstrap variable

daeed90f4586eb9533041fb89bee163a5193596c re-base on ALv2 code had 
incorporated
from http://svn.apache.org/viewvc?view=revisionrevision=1172102 imported
patch extensions_i117681.patch the parts using BAK_EXTENSIONS in
desktop/source/deployment/manager/, but not the parts setting 
BAK_EXTENSIONS in
scp2/source/ooo/.

Also, setting the stamp for bak in PackageManagerImpl::create needed to be
adapted to bee742eb7a0d5dfe23e61d9ee49a29286de90256 Fix sense of r/o 
detection
code, and clean up and 61eeb689d7605a23c3e71c652b57ee65cf5b28dc fix 
smoketest
- need to check read-only-ness of non-existent paths.

(cherry picked from commit 8dd405d32cb331818dc8f9bd1af3c5a074100f0f)
Conflicts:
desktop/source/deployment/manager/dp_manager.cxx

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

diff --git a/desktop/source/deployment/manager/dp_manager.cxx 
b/desktop/source/deployment/manager/dp_manager.cxx
index ba67f9e..b9949df 100644
--- a/desktop/source/deployment/manager/dp_manager.cxx
+++ b/desktop/source/deployment/manager/dp_manager.cxx
@@ -398,7 +398,7 @@ Referencedeployment::XPackageManager 
PackageManagerImpl::create(
 vnd.sun.star.expand:$BAK_EXTENSIONS);
 that-m_registryCache = OUSTR(
 vnd.sun.star.expand:$BAK_EXTENSIONS/registry);
-stamp = OUSTR(vnd.sun.star.expand:$BAK_EXTENSIONS/stamp.sys);
+stamp = OUSTR($BAK_EXTENSIONS);
 }
 
 else if (! context.matchAsciiL(
diff --git a/scp2/source/ooo/common_brand.scp b/scp2/source/ooo/common_brand.scp
index 7ba7e76..0c21df8 100644
--- a/scp2/source/ooo/common_brand.scp
+++ b/scp2/source/ooo/common_brand.scp
@@ -1198,6 +1198,14 @@ ProfileItem 
gid_Brand_Profileitem_Fundamental_Uno_Tmp_Extensions
 Value = ${$ORIGIN/ PROFILENAME(uno) :TMP_EXTENSIONS};
 End
 
+ProfileItem gid_Brand_Profileitem_Fundamental_Uno_Bak_Extensions
+ModuleID = gid_Module_Root_Brand;
+ProfileID = gid_Brand_Profile_Fundamental_Ini;
+Section = Bootstrap;
+Key = BAK_EXTENSIONS;
+Value = ${$ORIGIN/ PROFILENAME(uno) :TMP_EXTENSIONS};
+End
+
 ProfileItem gid_Brand_Profileitem_Fundamental_Ure_More_Types
 ModuleID = gid_Module_Root_Brand;
 ProfileID = gid_Brand_Profile_Fundamental_Ini;
diff --git a/scp2/source/ooo/profileitem_ooo.scp 
b/scp2/source/ooo/profileitem_ooo.scp
index 69ca5c6..abe5df4 100644
--- a/scp2/source/ooo/profileitem_ooo.scp
+++ b/scp2/source/ooo/profileitem_ooo.scp
@@ -166,6 +166,15 @@ ProfileItem gid_Profileitem_Uno_Uno_Bundled_Tmp_Extensions
 Value = ${$BRAND_BASE_DIR/program/ PROFILENAME(bootstrap) 
:UserInstallation}/user/extensions/tmp;
 End
 
+ProfileItem gid_Profileitem_Uno_Uno_Bundled_Bak_Extensions
+ProfileID = gid_Profile_Uno_Ini;
+ModuleID = gid_Module_Root;
+Section = Bootstrap;
+Order = 1;
+Key = BAK_EXTENSIONS;
+Value = ${$BRAND_BASE_DIR/program/ PROFILENAME(bootstrap) 
:UserInstallation}/user/extensions/bak;
+End
+
 ProfileItem gid_Profileitem_Uno_Uno_User_Packages_Cache
 ProfileID = gid_Profile_Uno_Ini;
 ModuleID = gid_Module_Root;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-4-0]: fdo#58909: Missing BAK_EXTENSIONS bootstrap variable

2013-01-10 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/1626

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie73ce50b6c26bbee6f3b7230a4d163715180ef52
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Caolán McNamara caol...@redhat.com

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


[PUSHED] Added Table Properties Widget

2013-01-10 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/1511

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I019b99e6194458b8b311b0cc3f2e231ef898ece0
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: gokul s gswaminat...@kacst.edu.sa
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' - setup_native/source

2013-01-10 Thread Libreoffice Gerrit user
 setup_native/source/packinfo/spellchecker_selection.pl |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 9299707ce09077f10c49ec836e26f6197ed20a1a
Author: Stephan Bergmann sberg...@redhat.com
Date:   Wed Jan 9 15:09:58 2013 +0100

Always default-select English dictionary

It was discovered that on a Russian Windows, by default only Russian and
German dictionaries would be installed, but no English dictionary.  This is
because since e2fac98819c00b4fb50f9de9d0f32d20092f3191 fdo#53009: For msi
installer, only default-select a subset of dictionaries only those 
dictionaries
mentioned in setup_native/source/packinfo/spellchecker_selection.txt were
default-selected, but it apparently was a misunderstanding that
spellcheker_selection.txt for many locales does not list en even though an
English dictionary should always be default-installed.

Quoting Andras Timar ati...@suse.com:  AFAIK en-US localization with 
English
dictionaries is always installed in all 3 supported platforms by default.  
On
Windows default install should always select en-US and English 
dictionaries.  If
it does not do so under certain circumstances, then it is a bug in my 
opinion.
We removed en-US entry from the spellchecker selection file for this very
reason.  It would have only bloated Mac and Linux langpacks.

So this fix simply makes sure that en is included in every line of the
generated spellchecker_selection.hxx that is used by
setup_native/source/win32/customactions/sellang/sellang.cxx to determine 
which
dictionaries to select by default.

Change-Id: I3db68585dc6269a7db2f6cbf6b79796dc09d05e8
(cherry picked from commit d8385c2cfc63b25414f37699faf0143ee9f5ded9)
Reviewed-on: https://gerrit.libreoffice.org/1614
Reviewed-by: Eike Rathke er...@redhat.com
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/setup_native/source/packinfo/spellchecker_selection.pl 
b/setup_native/source/packinfo/spellchecker_selection.pl
index b30a843..87f7cc2 100644
--- a/setup_native/source/packinfo/spellchecker_selection.pl
+++ b/setup_native/source/packinfo/spellchecker_selection.pl
@@ -27,8 +27,9 @@ while () {
 @{$map{$lang}} = ();
 } else {
 @{$map{$lang}} = split(/,/, $dicts);
-$max = max($max, scalar(@{$map{$lang}}));
 }
+push(@{$map{$lang}}, ('en')) unless grep($_ eq 'en', @{$map{$lang}});
+$max = max($max, scalar(@{$map{$lang}}));
 }
 
 ++$max;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - setup_native/source

2013-01-10 Thread Libreoffice Gerrit user
 setup_native/source/packinfo/spellchecker_selection.pl |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 86ca8cfe7ff5bfb889f3d65b1647839965fc88a7
Author: Stephan Bergmann sberg...@redhat.com
Date:   Wed Jan 9 15:09:58 2013 +0100

Always default-select English dictionary

It was discovered that on a Russian Windows, by default only Russian and
German dictionaries would be installed, but no English dictionary.  This is
because since e2fac98819c00b4fb50f9de9d0f32d20092f3191 fdo#53009: For msi
installer, only default-select a subset of dictionaries only those 
dictionaries
mentioned in setup_native/source/packinfo/spellchecker_selection.txt were
default-selected, but it apparently was a misunderstanding that
spellcheker_selection.txt for many locales does not list en even though an
English dictionary should always be default-installed.

Quoting Andras Timar ati...@suse.com:  AFAIK en-US localization with 
English
dictionaries is always installed in all 3 supported platforms by default.  
On
Windows default install should always select en-US and English 
dictionaries.  If
it does not do so under certain circumstances, then it is a bug in my 
opinion.
We removed en-US entry from the spellchecker selection file for this very
reason.  It would have only bloated Mac and Linux langpacks.

So this fix simply makes sure that en is included in every line of the
generated spellchecker_selection.hxx that is used by
setup_native/source/win32/customactions/sellang/sellang.cxx to determine 
which
dictionaries to select by default.

Change-Id: I3db68585dc6269a7db2f6cbf6b79796dc09d05e8
(cherry picked from commit d8385c2cfc63b25414f37699faf0143ee9f5ded9)
Reviewed-on: https://gerrit.libreoffice.org/1613
Reviewed-by: Eike Rathke er...@redhat.com
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/setup_native/source/packinfo/spellchecker_selection.pl 
b/setup_native/source/packinfo/spellchecker_selection.pl
index b30a843..87f7cc2 100644
--- a/setup_native/source/packinfo/spellchecker_selection.pl
+++ b/setup_native/source/packinfo/spellchecker_selection.pl
@@ -27,8 +27,9 @@ while () {
 @{$map{$lang}} = ();
 } else {
 @{$map{$lang}} = split(/,/, $dicts);
-$max = max($max, scalar(@{$map{$lang}}));
 }
+push(@{$map{$lang}}, ('en')) unless grep($_ eq 'en', @{$map{$lang}});
+$max = max($max, scalar(@{$map{$lang}}));
 }
 
 ++$max;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-4-0]: Always default-select English dictionary

2013-01-10 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/1614

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3db68585dc6269a7db2f6cbf6b79796dc09d05e8
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Eike Rathke er...@redhat.com

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


[PUSHED] Change in core[libreoffice-3-6]: Always default-select English dictionary

2013-01-10 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/1613

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3db68585dc6269a7db2f6cbf6b79796dc09d05e8
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Eike Rathke er...@redhat.com

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


[Libreoffice-commits] .: officecfg/registry

2013-01-10 Thread Libreoffice Gerrit user
 officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit dcd974c3ca70065ba1d984ad6bd5514848cd58a9
Author: Rob Snelders programm...@ertai.nl
Date:   Thu Dec 20 21:40:22 2012 +0100

- fdo#46460 - Missing Circles and Ovals toolbar

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

diff --git 
a/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu 
b/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
index b035510..42358d2d 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
@@ -252,7 +252,7 @@
 valuefalse/value
 /prop
 prop oor:name=HideFromToolbarMenu oor:type=xs:boolean
-valuetrue/value
+valuefalse/value
 /prop
 /node
 node oor:name=private:resource/toolbar/fontworkobjectbar 
oor:op=replace
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] Change in core[libreoffice-4-0]: - fdo#46460 - Missing Circles and Ovals toolbar

2013-01-10 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1627

To pull it, you can do:

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

- fdo#46460 - Missing Circles and Ovals toolbar

Change-Id: I08f2cef9c16ac998663f634653dad37a201f50c5
Reviewed-on: https://gerrit.libreoffice.org/1456
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit dcd974c3ca70065ba1d984ad6bd5514848cd58a9)
---
M officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git 
a/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu 
b/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
index b035510..42358d2d 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/DrawWindowState.xcu
@@ -252,7 +252,7 @@
 valuefalse/value
 /prop
 prop oor:name=HideFromToolbarMenu oor:type=xs:boolean
-valuetrue/value
+valuefalse/value
 /prop
 /node
 node oor:name=private:resource/toolbar/fontworkobjectbar 
oor:op=replace

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I08f2cef9c16ac998663f634653dad37a201f50c5
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Rob Snelders libreoff...@ertai.nl

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


[PUSHED] - fdo#46460 - Missing Circles and Ovals toolbar

2013-01-10 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/1456

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I08f2cef9c16ac998663f634653dad37a201f50c5
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rob Snelders libreoff...@ertai.nl
Gerrit-Reviewer: Caolán McNamara caol...@redhat.com

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


[Libreoffice-commits] .: slideshow/Library_OGLTrans.mk slideshow/source

2013-01-10 Thread Libreoffice Gerrit user
 slideshow/Library_OGLTrans.mk |2 
 slideshow/source/engine/OGLTrans/win/OGLTrans_Shaders.cxx |  142 --
 2 files changed, 1 insertion(+), 143 deletions(-)

New commits:
commit 6312695ad694598951f2cdd4ac26c53457fedb79
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Jan 10 14:55:33 2013 +0200

The unx and win OGLTrans_Shaders.cxx files were effectively identical

Just some OSL_TRACE/SAL_INFO changes in the unx one. And it isn't
like the win one would actually compile for Windows as is anyway, as
it unconditionally uses glX stuff. So certainly it is better to have
just one copy, and if we want to make the OpenGL transitions build on
Windows then add ifdefs for Windows to it.

Change-Id: If6887e24a0fc03c7e0e77f003d08feb51c264f30

diff --git a/slideshow/Library_OGLTrans.mk b/slideshow/Library_OGLTrans.mk
index bae4d99..44db60b 100644
--- a/slideshow/Library_OGLTrans.mk
+++ b/slideshow/Library_OGLTrans.mk
@@ -86,7 +86,7 @@ $(eval $(call gb_Library_use_system_win32_libs,OGLTrans,\
 ))
 
 $(eval $(call gb_Library_add_exception_objects,OGLTrans,\
-slideshow/source/engine/OGLTrans/win/OGLTrans_Shaders \
+slideshow/source/engine/OGLTrans/unx/OGLTrans_Shaders \
 slideshow/source/engine/OGLTrans/win/OGLTrans_TransitionerImpl \
 slideshow/source/engine/OGLTrans/win/OGLTrans_TransitionImpl \
 ))
diff --git a/slideshow/source/engine/OGLTrans/win/OGLTrans_Shaders.cxx 
b/slideshow/source/engine/OGLTrans/win/OGLTrans_Shaders.cxx
deleted file mode 100644
index c1beae1..000
--- a/slideshow/source/engine/OGLTrans/win/OGLTrans_Shaders.cxx
+++ /dev/null
@@ -1,142 +0,0 @@
-/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
-/*
- *
- * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- *
- * Copyright 2008 by Sun Microsystems, Inc.
- *
- * OpenOffice.org - a multi-platform office productivity suite
- *
- * This file is part of OpenOffice.org.
- *
- * OpenOffice.org is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License version 3
- * only, as published by the Free Software Foundation.
- *
- * OpenOffice.org is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License version 3 for more details
- * (a copy is included in the LICENSE file that accompanied this code).
- *
- * You should have received a copy of the GNU Lesser General Public License
- * version 3 along with OpenOffice.org.  If not, see
- * http://www.openoffice.org/license.html
- * for a copy of the LGPLv3 License.
- *
- /
-
-#include osl/diagnose.hxx
-
-#include OGLTrans_Shaders.hxx
-
-namespace unx
-{
-#include GL/glx.h
-#include GL/glxext.h
-}
-
-bool OGLShaders::cbInitialized = false;
-
-#ifdef GL_VERSION_2_0
-
-PFNGLCREATESHADERPROC OGLShaders::glCreateShader = NULL;
-PFNGLSHADERSOURCEPROC OGLShaders::glShaderSource = NULL;
-PFNGLCOMPILESHADERPROC OGLShaders::glCompileShader = NULL;
-PFNGLGETSHADERIVPROC OGLShaders::glGetShaderiv = NULL;
-PFNGLGETSHADERINFOLOGPROC OGLShaders::glGetShaderInfoLog = NULL;
-PFNGLDELETESHADERPROC OGLShaders::glDeleteShader = NULL;
-PFNGLCREATEPROGRAMPROC OGLShaders::glCreateProgram = NULL;
-PFNGLATTACHSHADERPROC OGLShaders::glAttachShader = NULL;
-PFNGLLINKPROGRAMPROC OGLShaders::glLinkProgram = NULL;
-PFNGLGETPROGRAMIVPROC OGLShaders::glGetProgramiv = NULL;
-PFNGLGETPROGRAMINFOLOGPROC OGLShaders::glGetProgramInfoLog = NULL;
-PFNGLUSEPROGRAMPROC OGLShaders::glUseProgram = NULL;
-PFNGLDELETEPROGRAMPROC OGLShaders::glDeleteProgram = NULL;
-PFNGLGETUNIFORMLOCATIONPROC OGLShaders::glGetUniformLocation = NULL;
-PFNGLUNIFORM1IPROC OGLShaders::glUniform1i = NULL;
-PFNGLUNIFORM1FPROC OGLShaders::glUniform1f = NULL;
-#endif
-
-bool OGLShaders::Initialize()
-{
-#ifdef GL_VERSION_2_0
-if( !cbInitialized ) {
-glCreateShader = (PFNGLCREATESHADERPROC) unx::glXGetProcAddress( 
(unsigned char *) glCreateShader );
-glShaderSource = (PFNGLSHADERSOURCEPROC) unx::glXGetProcAddress( 
(unsigned char *) glShaderSource );
-glCompileShader = (PFNGLCOMPILESHADERPROC) unx::glXGetProcAddress( 
(unsigned char *) glCompileShader );
-glGetShaderiv = (PFNGLGETSHADERIVPROC) unx::glXGetProcAddress( 
(unsigned char *) glGetShaderiv );
-glGetShaderInfoLog = (PFNGLGETSHADERINFOLOGPROC) 
unx::glXGetProcAddress( (unsigned char *) glGetShaderInfoLog );
-glDeleteShader = (PFNGLDELETESHADERPROC) unx::glXGetProcAddress( 
(unsigned char *) glDeleteShader );
-glCreateProgram = (PFNGLCREATEPROGRAMPROC) unx::glXGetProcAddress( 
(unsigned char *) glCreateProgram );
-glAttachShader = (PFNGLATTACHSHADERPROC) 

WITH_MOZAB4WIN needing msvc?80.dll

2013-01-10 Thread Lubos Lunak

 Hello,

 as far as I understand it, the current problem with 
W2008R2_20-With-Symbol-Bytemark-Hosting [*] , is caused by needing 
specifically *80 msvc dlls for WITH_MOZAB4WIN (grep under scp2 for the 
only 'msvcp' there). Does somebody have any idea why it's specifically this 
version and if it's really needed this way? It dates back to 2009, so repo 
history is as usually useless.

[*]
ERROR: The following files could not be found: 
ERROR: File not found: Microsoft.VC80.CRT.manifest
ERROR: File not found: msvcp80.dll
ERROR: File not found: msvcr80.dll

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


Re: [libreoffice-l10n] Translation of the names of the keys on Windows

2013-01-10 Thread Martin Srebotnjak
Hi,

here is the Slovenian replacement table, please add it to the code:

static const struct KeysNameReplacement aImplReplacements_Slovenian[] =
{
 { KEY_ESC, Ubežnica },
 { KEY_BACK, Vračalka },
 { KEY_ENTER, Vnašalka },
 { KEY_SPACEBAR, Preslednica },
 { KEY_HOME, Začetek },
 { KEY_UP, Navzgor },
 { KEY_PAGEUP, Prejšnja stran },
 { KEY_LEFT, Levo },
 { KEY_RIGHT, Desno },
 { KEY_END, Konec },
 { KEY_DOWN, Navzdol },
 { KEY_PAGEDOWN, Naslednja stran },
 { KEY_INSERT, Vrivalka },
 { KEY_DELETE, Brisalka },
 { KEY_CONTROL, Krmilka },
 { KEY_SHIFT, Dvigalka },
 { KEY_ALT, Izmenjalka },
};

Please also add the following line to the aKeyboards[]:
{ sl, aImplReplacements_Slovenian,
SAL_N_ELEMENTS(aImplReplacements_Slovenian) },

Please notify when this is in, so I can check that special characters look ok.

Thanks, m.

2013/1/8 Jesús Corrius je...@softcatala.org:
 Hi all,

 Since today it's possible to provide custom translations of the names of
 the keys on Windows. This is useful because Windows gets the name of the
 key from the keyboard layout and, in some cases, they don't match properly.

 For example, Catalan users don't have a proper keyboard layout (well, they
 do have one but it's just an alias for the Spanish keyboard) so if a user
 uses the Spanish or French keyboard, the name of the keys in the UI will be
 in Spanish or French, but never in Catalan. I am sure other languages have
 this problem too.

 So it's now possible to provide custom names for keys for Windows using the
 same way we do for Linux. The file to modify is the following one and the
 process is exactly the same:

 http://cgit.freedesktop.org/libreoffice/core/tree/vcl/win/source/window/keynames.cxx

 If you want to implement this, but you are not a developer, just send me
 the name of the keys with its translation and I will apply the changes
 myself.

 Thanks and Happy 2013 to everybody!

 --
 Jesús Corrius je...@softcatala.org

 --
 Unsubscribe instructions: E-mail to l10n+h...@global.libreoffice.org
 Problems? 
 http://www.libreoffice.org/get-help/mailing-lists/how-to-unsubscribe/
 Posting guidelines + more: http://wiki.documentfoundation.org/Netiquette
 List archive: http://listarchives.libreoffice.org/global/l10n/
 All messages sent to this list will be publicly archived and cannot be deleted

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


[Libreoffice-commits] .: sdext/source

2013-01-10 Thread Libreoffice Gerrit user
 sdext/source/presenter/PresenterSlideSorter.cxx |8 +++-
 1 file changed, 3 insertions(+), 5 deletions(-)

New commits:
commit eb0d07d3f2937315700dabf72d366f5ef4d4200d
Author: Abdulelah Alarifi asalar...@kacst.edu.sa
Date:   Wed Dec 12 13:26:55 2012 +0300

fdo#57422 - presenter console RTL support needs completion

This patch will fix the text direction of the text that
appears when the mouse come over the slide in
Slide sorter view.

Change-Id: I2d3c5563b22dcd40ae4f70ed0ed06ad17f6c4132
Change-Id: I812e011b4e0aaeeb7ec6114a655240effc8c606a
Reviewed-on: https://gerrit.libreoffice.org/1284
Reviewed-by: Faisal al-otaibi fmalota...@kacst.edu.sa
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sdext/source/presenter/PresenterSlideSorter.cxx 
b/sdext/source/presenter/PresenterSlideSorter.cxx
index eb0ce72..4148152 100644
--- a/sdext/source/presenter/PresenterSlideSorter.cxx
+++ b/sdext/source/presenter/PresenterSlideSorter.cxx
@@ -1583,12 +1583,10 @@ Referencerendering::XBitmap 
PresenterSlideSorter::MouseOverManager::CreateBitm
 rendering::CompositeOperation::SOURCE);
 PresenterCanvasHelper::SetDeviceColor(aRenderState, mpFont-mnColor);
 
-xBitmapCanvas-drawText(
-aContext,
-mpFont-mxFont,
+xBitmapCanvas-drawTextLayout (
+xLayout,
 aViewState,
-aRenderState,
-rendering::TextDirection::WEAK_LEFT_TO_RIGHT);
+aRenderState);
 }
 
 return xBitmap;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sdext/source

2013-01-10 Thread Libreoffice Gerrit user
 sdext/source/presenter/PresenterToolBar.cxx |  162 +++-
 1 file changed, 41 insertions(+), 121 deletions(-)

New commits:
commit b0a827aa839afefd580c438ea72a7e5ca0b88aea
Author: Faisal M. Al-Otaibi fmalota...@kacst.edu.sa
Date:   Sun Dec 23 12:18:20 2012 +0300

Fix some errors and remove duplicate code on Presenter

Fix the toolbar text if the interface were RTL.
Remove duplicate code from PresenterToolBar.cxx

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

diff --git a/sdext/source/presenter/PresenterToolBar.cxx 
b/sdext/source/presenter/PresenterToolBar.cxx
index ac69c76..34dfb7e 100644
--- a/sdext/source/presenter/PresenterToolBar.cxx
+++ b/sdext/source/presenter/PresenterToolBar.cxx
@@ -1760,47 +1760,21 @@ void Text::Paint (
 aContext,
 rendering::TextDirection::WEAK_LEFT_TO_RIGHT,
 0));
-/** this responsible of the toolbar and the zoom
-that in the note mode.
-check whether RTL interface or not. */
-if(!Application::GetSettings().GetLayoutRTL()){
-geometry::RealRectangle2D aBox (xLayout-queryTextBounds());
-const double nTextWidth = aBox.X2 - aBox.X1;
-const double nY = rBoundingBox.Y + rBoundingBox.Height - aBox.Y2;
-const double nX = rBoundingBox.X + (rBoundingBox.Width - nTextWidth)/2;
-
-rendering::RenderState aRenderState(
-geometry::AffineMatrix2D(1,0,nX, 0,1,nY),
-NULL,
-Sequencedouble(4),
-rendering::CompositeOperation::SOURCE);
-PresenterCanvasHelper::SetDeviceColor(aRenderState, mpFont-mnColor);
-rxCanvas-drawText(
-aContext,
-mpFont-mxFont,
-rViewState,
-aRenderState,
-rendering::TextDirection::WEAK_LEFT_TO_RIGHT);
-}
-else {
-geometry::RealRectangle2D aBox (xLayout-queryTextBounds());
-const double nTextWidth = aBox.X2 - aBox.X1;
-const double nY = rBoundingBox.Y + rBoundingBox.Height - aBox.Y2;
-const double nX = rBoundingBox.X + (rBoundingBox.Width + nTextWidth)/2;
-
-rendering::RenderState aRenderState(
-geometry::AffineMatrix2D(1,0,nX, 0,1,nY),
-NULL,
-Sequencedouble(4),
-rendering::CompositeOperation::SOURCE);
-PresenterCanvasHelper::SetDeviceColor(aRenderState, mpFont-mnColor);
-rxCanvas-drawText(
-aContext,
-mpFont-mxFont,
-rViewState,
-aRenderState,
-rendering::TextDirection::WEAK_RIGHT_TO_LEFT);
-}
+geometry::RealRectangle2D aBox (xLayout-queryTextBounds());
+const double nTextWidth = aBox.X2 - aBox.X1;
+const double nY = rBoundingBox.Y + rBoundingBox.Height - aBox.Y2;
+const double nX = rBoundingBox.X + (rBoundingBox.Width - nTextWidth)/2;
+
+rendering::RenderState aRenderState(
+geometry::AffineMatrix2D(1,0,nX, 0,1,nY),
+NULL,
+Sequencedouble(4),
+rendering::CompositeOperation::SOURCE);
+PresenterCanvasHelper::SetDeviceColor(aRenderState, mpFont-mnColor);
+rxCanvas-drawTextLayout(
+xLayout,
+rViewState,
+aRenderState);
 }
 
 geometry::RealRectangle2D Text::GetBoundingBox (const 
Referencerendering::XCanvas rxCanvas)
@@ -1811,25 +1785,13 @@ geometry::RealRectangle2D Text::GetBoundingBox (const 
Referencerendering::XCanv
 mpFont-PrepareFont(rxCanvas);
 if (mpFont-mxFont.is())
 {
-/// check whether RTL interface or not
-if(!Application::GetSettings().GetLayoutRTL()){
-rendering::StringContext aContext (msText, 0, 
msText.getLength());
-Referencerendering::XTextLayout xLayout (
-mpFont-mxFont-createTextLayout(
-aContext,
-rendering::TextDirection::WEAK_LEFT_TO_RIGHT,
-0));
-return xLayout-queryTextBounds();
-}
-else {
-rendering::StringContext aContext (msText, 0, 
msText.getLength());
-Referencerendering::XTextLayout xLayout (
-mpFont-mxFont-createTextLayout(
-aContext,
-rendering::TextDirection::WEAK_RIGHT_TO_LEFT,
-0));
-return xLayout-queryTextBounds();
-}
+rendering::StringContext aContext (msText, 0, msText.getLength());
+Referencerendering::XTextLayout xLayout (
+mpFont-mxFont-createTextLayout(
+aContext,
+rendering::TextDirection::WEAK_LEFT_TO_RIGHT,
+0));
+return xLayout-queryTextBounds();
 }
 }

[PUSHED] fdo#57422 - presenter console RTL support needs completion

2013-01-10 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/1284

Approvals:
  Faisal al-otaibi: Looks good to me, but someone else must approve
  Caolán McNamara: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I812e011b4e0aaeeb7ec6114a655240effc8c606a
Gerrit-PatchSet: 4
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Abdulelah Alarifi asalar...@kacst.edu.sa
Gerrit-Reviewer: Abdulelah Alarifi asalar...@kacst.edu.sa
Gerrit-Reviewer: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Faisal al-otaibi fmalota...@kacst.edu.sa
Gerrit-Reviewer: Kohei Yoshida kohei.yosh...@gmail.com

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


[PUSHED] Fix some errors and remove duplicate code on Presenter

2013-01-10 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/1474

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icd6f71ce2daa896494c8efe76779d49e16c547bf
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Faisal al-otaibi fmalota...@kacst.edu.sa
Gerrit-Reviewer: Caolán McNamara caol...@redhat.com
Gerrit-Reviewer: Faisal al-otaibi fmalota...@kacst.edu.sa

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


Re: WITH_MOZAB4WIN needing msvc?80.dll

2013-01-10 Thread Michael Stahl
On 10/01/13 14:01, Lubos Lunak wrote:
 
  Hello,
 
  as far as I understand it, the current problem with 
 W2008R2_20-With-Symbol-Bytemark-Hosting [*] , is caused by needing 
 specifically *80 msvc dlls for WITH_MOZAB4WIN (grep under scp2 for the 
 only 'msvcp' there). Does somebody have any idea why it's specifically this 
 version and if it's really needed this way? It dates back to 2009, so repo 
 history is as usually useless.

AFAIK the Mozilla stuff (moz/) does not build with MSVC 2008, only with
MSVC2005; that's why it has the unique distinction of allowing a binary
zip file containing all mozilla libraries to be used instead of building
it (--disable-build-mozilla); actually i've never built moz on windows
so i don't even know what you would need.

since it's built with MSVC 2005 and every release of MSVC brings a new
incompatible version of msvcrt libraries, the installer wants to bundles
these ones if you don't disable mozilla completely:

 [*]
 ERROR: The following files could not be found: 
 ERROR: File not found: Microsoft.VC80.CRT.manifest
 ERROR: File not found: msvcp80.dll
 ERROR: File not found: msvcr80.dll


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


Re: FYI: Make for faster windows build

2013-01-10 Thread Lubos Lunak
On Thursday 10 of January 2013, Noel Grandin wrote:
 I think the WINDOWS32 #define is for building a native windows binary.
 (There is also stuff in there for building under AmigaOS and DOS, the
 gnumake code is pretty grotty)

 The cywin stuff is probably using a #ifdef CYGWIN.

 I don't see anything cygwin-specific there, except for handling the cygwin 
shell. And I expect the WINDOWS32 code should work just fine for cygwin make 
as well.

 You can't use the Win32 API as-is under cygwin, because you need to call
 cygpath() on the path argument first to convert from the cygwin
 filesystem structure to the Win32 representation.
 i.e. from /cygdrive/C/libo to C:\libo

 I have not done this, apparently make always gets windows paths when building 
LO, or does somebody have a problem with this (builtins have (Built-in) 
prefix when doing verbose make)?

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


Re: FYI: Make for faster windows build

2013-01-10 Thread Lubos Lunak
On Thursday 10 of January 2013, Michael Meeks wrote:
 On Wed, 2013-01-09 at 20:01 +0100, Lubos Lunak wrote:
  Looks what I found under the tree! Faster make for Cygwin. Granted, I had
  put it there first, but still nice.

   Nice work ! :-)

   I'd love to see a configure option: --with-custom-gnumake that does the
 git clone, pre-builds, and uses our own gnumake (with the speedups).

 If I'm not mistaken, we need our custom make on Windows anyway, since LO 
build fails with the cygwin one for some reason (may or may not be our fault, 
I don't know). Moreover, this is most probably not going to work anyway, 
since it's the developer who runs the make, not the buildsystem (eventually, 
after we get rid of dmake). And Makefile.top has shown that make forwarding 
does not quite work as it should.

 Did you have any joy getting the fixes up-stream to gnumake ? ;-)

 I have not tried, the commands are rather LO-specific in some cases, so we 
need a patch anyway. I also consider it to be a bit of an ugly hack.

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


Re: Need confirmation about non-clearing of Character Styles

2013-01-10 Thread Caolán McNamara
On Thu, 2013-01-10 at 10:55 +0100, Fernand Vanrie wrote:
 Sinds all or editors use LO 3.6.2 (before OO3.3)
 
 We have complains about the fact that parts of the text, who are
 formated with a Character Style, can not been cleared to
 no-formating,
 not by Clear Direct Formating (cntrl+M)
 
 There is a dirty work-around by applying the Default character style
 
 Paragraph Styles can been cleared
 Is this intentional behavior or a regression open for a issue ?

My understanding is
a) that its intentional, see
http://lists.freedesktop.org/archives/ooo-build/2010-February/000540.html
and the commit
http://cgit.freedesktop.org/libreoffice/core/commit/?id=6e690e0909459d46c53d17e7939891abea11e566
b) that paragraph styles are not cleared, i.e. the text should remain
using the original paragraph and character styles, but any additional
properties applied on top of that is reset to the underlying paragraph
and character styles properties.

C.

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


[Libreoffice-commits] .: 3 commits - configure.ac

2013-01-10 Thread Libreoffice Gerrit user
 configure.ac |  109 +--
 1 file changed, 61 insertions(+), 48 deletions(-)

New commits:
commit 417943f071cc6c7ebe4dad69dfdec411cc7de975
Author: Tor Lillqvist tlillqv...@suse.com
Date:   Thu Jan 10 15:41:02 2013 +0200

Get rid of last actual TABs in this file

Change-Id: Ic92c47834add4f5d0350343e6b7ea8294477b495

diff --git a/configure.ac b/configure.ac
index 7267536..2ef61b1 100644
--- a/configure.ac
+++ b/configure.ac
@@ -4575,6 +4575,8 @@ if test -z $GNUMAKE; then
 AC_MSG_ERROR([not found. install GNU make.])
 fi
 
+TAB=`printf '\t'`
+
 AC_MSG_CHECKING([the GNU make version])
 _make_version=`$GNUMAKE --version | grep GNU | $GREP -v GPL | $SED -e 
's@^[[^0-9]]*@@' -e 's@ .*@@' -e 's@,.*@@'`
 _make_longver=`echo $_make_version | $AWK -F. '{ print \$1*1+\$2*100+\$3 
}'`
@@ -4596,16 +4598,16 @@ elif test $_make_longver -ge 038100; then
 fi
 AC_MSG_CHECKING([for GNU make bug 20033])
 TESTGMAKEBUG20033=`mktemp -d tmp.XX`
-cat  $TESTGMAKEBUG20033/Makefile  EOF
+$SED -e s/TAB/$TAB/  $TESTGMAKEBUG20033/Makefile  EOF
 A := \$(wildcard *.a)
 
 .PHONY: all
 all: \$(A:.a=.b)
-   @echo survived bug20033. #dont kill these tabs, you will break the 
Makefile
+TAB@echo survived bug20033. #dont kill these tabs, you will break the 
Makefile
 
 .PHONY: setup
 setup:
-   @touch 1.a 2.a 3.a 4.a 5.a 6.a
+TAB@touch 1.a 2.a 3.a 4.a 5.a 6.a
 
 define d1
 @echo lala \$(1)
@@ -4618,9 +4620,9 @@ define d2
 endef
 
 %.b : %.a
-   \$(eval CHECKSUM := \$(word 1,\$(shell cat \$^ | $SHA1SUM))) \$(if 
\$(wildcard \$(CACHEDIR)/\$(CHECKSUM)),\
-   \$(call d1,\$(CHECKSUM)),\
-   \$(call d2,\$(CHECKSUM)))
+TAB\$(eval CHECKSUM := \$(word 1,\$(shell cat \$^ | $SHA1SUM))) \$(if 
\$(wildcard \$(CACHEDIR)/\$(CHECKSUM)),\
+TAB\$(call d1,\$(CHECKSUM)),\
+TAB\$(call d2,\$(CHECKSUM)))
 EOF
 if test -z `(cd $TESTGMAKEBUG20033  $GNUMAKE setup  $GNUMAKE -j)|grep 
survived`; then
 no_parallelism_make=YES
@@ -4636,12 +4638,12 @@ fi
 # find if gnumake support file function
 AC_MSG_CHECKING([whether GNU make supports the 'file' function])
 TESTGMAKEFILEFUNC=`mktemp -d -t tst.XX`
-cat  $TESTGMAKEFILEFUNC/Makefile  EOF
+$SED -e s/TAB/$TAB/  $TESTGMAKEFILEFUNC/Makefile  EOF
 \$(file test.txt,Success )
 
 .PHONY: all
 all:
-   @cat test.txt
+TAB@cat test.txt
 
 EOF
 $GNUMAKE -C $TESTGMAKEFILEFUNC 2/dev/null 12
commit f340cbb6af86c7046d34202d2781a68b0d991001
Author: Tor Lillqvist tlillqv...@suse.com
Date:   Thu Jan 10 15:13:49 2013 +0200

De-tab

Change-Id: If0e93181affadd7c608625c494daf6da78ba9bbc

diff --git a/configure.ac b/configure.ac
index 60daa23..7267536 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8293,9 +8293,9 @@ if test $with_system_nss = yes; then
 libo_MINGW_CHECK_DLL([SSL3], [ssl3])
 else
 SYSTEM_NSS=NO
-   NSS_MAJOR=3
-   NSS_MINOR=13
-   NSS_PATCH=5
+NSS_MAJOR=3
+NSS_MINOR=13
+NSS_PATCH=5
 BUILD_TYPE=$BUILD_TYPE NSS
 AC_MSG_RESULT([internal])
 fi # system nss
commit dc7e074fc89677d87194b2836c18755788c22f1c
Author: Tor Lillqvist tlillqv...@suse.com
Date:   Thu Jan 10 15:09:46 2013 +0200

Make it possible to even try building the OpenGL transitions for Windows

Change-Id: Id9b688a22518aedb8959120f909832f0cd84b324

diff --git a/configure.ac b/configure.ac
index 688f2e4..60daa23 100644
--- a/configure.ac
+++ b/configure.ac
@@ -867,8 +867,7 @@ AC_ARG_ENABLE(cairo-canvas,
 
 AC_ARG_ENABLE(opengl,
 AS_HELP_STRING([--disable-opengl],
-[Determines whether to build the OpenGL 3D slide transitions 
component.]),
-,enable_opengl=yes)
+[Determines whether to build the OpenGL 3D slide transitions 
component.]))
 
 AC_ARG_ENABLE(dbus,
 AS_HELP_STRING([--disable-dbus],
@@ -9702,47 +9701,59 @@ dnl 
===
 AC_MSG_CHECKING([whether to build the OpenGL Transitions component])
 ENABLE_OPENGL=
 
-if test $_os = Darwin; then
+if test -z $enable_opengl; then
+if test $_os = WINNT; then
+# By default disable the OpenGL transitions for Windows (the code 
doesn't even compile
+# currently). In other words, use --enable-opengl only if you plan to 
hack on that code for
+# Windows.
+enable_opengl=no
+else
+enable_opengl=yes
+fi
+fi
+
+if test x$enable_opengl = xno; then
+AC_MSG_RESULT([no])
+elif test $_os = Darwin; then
 # We use frameworks on Mac OS X, no need for detail checks
 ENABLE_OPENGL=TRUE
 SYSTEM_MESA_HEADERS=YES
 AC_MSG_RESULT([yes])
+elif test $_os = WINNT; then
+# Experimental: try to use OpenGL on Windows
+ENABLE_OPENGL=TRUE
+# We need the internal Mesa headers.
+SYSTEM_MESA_HEADERS=NO
+BUILD_TYPE=$BUILD_TYPE MESA
+AC_MSG_RESULT([yes])
 else
-if test x$enable_opengl != xno; then
-if test $_os != WINNT; then
-save_LDFLAGS=$LDFLAGS
-

[Libreoffice-commits] .: 4 commits - bridges/Module_bridges.mk cross_tail_build/Makefile Makefile.in solenv/bin solenv/gbuild tail_build/Makefile

2013-01-10 Thread Libreoffice Gerrit user
 Makefile.in |   11 ++--
 bridges/Module_bridges.mk   |2 
 cross_tail_build/Makefile   |1 
 solenv/bin/build.pl |2 
 solenv/gbuild/Module.mk |   61 +---
 solenv/gbuild/extensions/post_SpeedUpTargets.mk |   38 +++---
 solenv/gbuild/gbuild.mk |   10 ---
 solenv/gbuild/partial_build.mk  |3 -
 tail_build/Makefile |1 
 9 files changed, 30 insertions(+), 99 deletions(-)

New commits:
commit ca6aefe0839286515c6874b917e980eaa6d03282
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Thu Jan 10 14:42:07 2013 +0100

these are phony targets

Change-Id: I6de7842d5f03c6df5bffefdf77a90b5efee2fc78

diff --git a/Makefile.in b/Makefile.in
index 3ed0cdd..d114d81 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -422,6 +422,7 @@ check: dev-install subsequentcheck
 subsequentcheck :| $(if $(filter-out 
subsequentcheck,$(MAKECMDGOALS)),dev-install)
$(GNUMAKE) -j $(CHECK_PARALLELISM) $(GMAKE_OPTIONS) -f 
$(SRC_ROOT)/Makefile.post $@
 
+.PHONY : unitcheck slowcheck debugrun translations
 unitcheck slowcheck debugrun translations:
$(GNUMAKE) -j $(PARALLELISM) $(GMAKE_OPTIONS) -f 
$(SRC_ROOT)/Makefile.post $@
 
commit 26637fe8f241c6ae00f8fdcd5d2a647c7c3cb411
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Wed Jan 9 21:37:10 2013 +0100

optimize for build targets in one place

Change-Id: Ib886a323c1ade6dcd455a44d3dfc6702086416e3

diff --git a/bridges/Module_bridges.mk b/bridges/Module_bridges.mk
index 14a7255..c1dbcef 100644
--- a/bridges/Module_bridges.mk
+++ b/bridges/Module_bridges.mk
@@ -96,7 +96,7 @@ $(eval $(call gb_Module_add_targets,bridges,\
) \
 ))
 
-ifeq ($(gb_FULL_BUILD),$(true))
+ifeq (,$(gb_STRIPPED_BUILD))
 ifeq ($(strip $(bridges_SELECTED_BRIDGE)),)
 $(call gb_Output_error,no bridge selected for build: bailing out)
 endif
diff --git a/solenv/gbuild/Module.mk b/solenv/gbuild/Module.mk
index 0f985bb..d20176a 100644
--- a/solenv/gbuild/Module.mk
+++ b/solenv/gbuild/Module.mk
@@ -89,7 +89,7 @@ $(call gb_Module_get_target,%) :
mkdir -p $(dir $@)  \
touch $@)
 
-.PHONY : build all clean unitcheck slowcheck subsequentcheck showmodules 
translations
+.PHONY : all build unitcheck slowcheck subsequentcheck clean check debugrun 
help showmodules translations
 .DEFAULT_GOAL := all
 
 all : build unitcheck
@@ -130,6 +130,9 @@ check : unitcheck slowcheck
 debugrun :
$(call gb_Module_DEBUGRUNCOMMAND)
 
+help :
+   @cat $(SRCDIR)/solenv/gbuild/gbuild.help.txt
+
 showmodules :
$(info $(strip $(gb_Module_ALLMODULES)))
@true 
@@ -184,9 +187,6 @@ endif
 
 endef
 
-gb_FULL_BUILD := $(if $(filter showmodules 
translations,$(MAKECMDGOALS)),$(false),$(true))
-
-ifeq ($(gb_FULL_BUILD),$(true))
 define gb_Module_add_target
 $(call gb_Module__read_targetfile,$(1),$(2),target)
 
@@ -220,7 +220,6 @@ $(call gb_Module_get_subsequentcheck_target,$(1)) : 
$$(gb_Module_CURRENTTARGET)
 $(call gb_Module_get_clean_target,$(1)) : $$(gb_Module_CURRENTCLEANTARGET)
 
 endef
-endif
 
 define gb_Module_add_moduledir
 include $(patsubst $(1):%,%,$(filter 
$(1):%,$(gb_Module_MODULELOCATIONS)))/$(2)/Module_$(2).mk
diff --git a/solenv/gbuild/extensions/post_SpeedUpTargets.mk 
b/solenv/gbuild/extensions/post_SpeedUpTargets.mk
index 6175cbc..406a69e 100644
--- a/solenv/gbuild/extensions/post_SpeedUpTargets.mk
+++ b/solenv/gbuild/extensions/post_SpeedUpTargets.mk
@@ -25,23 +25,31 @@
 #   in which case the provisions of the GPLv3+ or the LGPLv3+ are applicable
 #   instead of those above.
 
-
-# speed up if no target need to parse all modules
-gb_SpeedUpTargets_WRAPPEDBUILD:=
 ifneq ($(strip $(MAKECMDGOALS)),)
-ifeq ($(filter-out id tags docs distro-pack-install fetch help debugrun 
$(SRCDIR)/config_host.mk,$(MAKECMDGOALS)),)
-gb_SpeedUpTargets_WRAPPEDBUILD:=T
+# speed up depending on the target
+gb_SpeedUpTargets_LEVEL_3 := debugrun help translations
+gb_SpeedUpTargets_LEVEL_2 := showmodules $(gb_SpeedUpTargets_LEVEL_3)
+gb_SpeedUpTargets_LEVEL_1 := clean showdeliverables 
$(gb_SpeedUpTargets_LEVEL_2)
+
+ifeq (T,$(if $(filter-out $(gb_SpeedUpTargets_LEVEL_1),$(MAKECMDGOALS)),,T))
+gb_FULLDEPS :=
+
+ifeq (T,$(if $(filter-out $(gb_SpeedUpTargets_LEVEL_2),$(MAKECMDGOALS)),,T))
+gb_Module_add_target :=
+gb_Module_add_check_target :=
+gb_Module_add_slowcheck_target :=
+gb_Module_add_subsequentcheck_target :=
+
+# Turns of check in bridges/Module_bridges.mk:
+gb_STRIPPED_BUILD := $(true)
+
+ifeq (T,$(if $(filter-out $(gb_SpeedUpTargets_LEVEL_3),$(MAKECMDGOALS)),,T))
+gb_Module_add_moduledir :=
+
+endif
 endif
 endif
 
-ifneq ($(strip $(gb_SpeedUpTargets_WRAPPEDBUILD)),)
-gb_Module_add_target=
-gb_Module_add_check_target=
-gb_Module_add_slowcheck_target=
-gb_Module_add_subsequentcheck_target=
-gb_Module_add_moduledir=

gb_COMPILERNOOPTFLAGS does not even obey the user passed CFLAGS

2013-01-10 Thread Tomáš Chvátal
Hi guys,

As written in subject the variable is used on multiple targets and it
ignores the user defined cflags/cxxflags.
This should never happen and all our targets should take into effect the
user defined options too.

I am not sure which variable I should append there, so could anyone check
if the diff in attachment is right?

Cheers

Tom


cxxflags.patch
Description: Binary data
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: sc/source

2013-01-10 Thread Libreoffice Gerrit user
 sc/source/ui/dbgui/sortkeydlg.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 2c403c7f36561410418ebb9f86664e46a8c1c968
Author: Caolán McNamara caol...@redhat.com
Date:   Thu Jan 10 13:49:20 2013 +

Resolves: fdo#59112 re-scroll to current entry when dialog resized

Change-Id: I96486f6edad11520bf9bb43b700b1716d9b2f170

diff --git a/sc/source/ui/dbgui/sortkeydlg.cxx 
b/sc/source/ui/dbgui/sortkeydlg.cxx
index 76a8747..211afcc 100644
--- a/sc/source/ui/dbgui/sortkeydlg.cxx
+++ b/sc/source/ui/dbgui/sortkeydlg.cxx
@@ -126,6 +126,7 @@ void ScSortKeyCtrl::setScrollRange()
 sal_Int32 nVisibleItems = m_rScrolledWindow.getVisibleChildSize().Height() 
/ nScrollOffset;
 m_rVertScroll.SetPageSize( nVisibleItems - 1 );
 m_rVertScroll.SetVisibleSize( nVisibleItems );
+m_rVertScroll.Scroll();
 }
 
 // ---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] Change in core[libreoffice-4-0]: Resolves: fdo#59112 re-scroll to current entry when dialog r...

2013-01-10 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1628

To pull it, you can do:

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

Resolves: fdo#59112 re-scroll to current entry when dialog resized

Change-Id: I96486f6edad11520bf9bb43b700b1716d9b2f170
(cherry picked from commit 2c403c7f36561410418ebb9f86664e46a8c1c968)
---
M sc/source/ui/dbgui/sortkeydlg.cxx
1 file changed, 1 insertion(+), 0 deletions(-)



diff --git a/sc/source/ui/dbgui/sortkeydlg.cxx 
b/sc/source/ui/dbgui/sortkeydlg.cxx
index 45d65ca..e71a48f 100644
--- a/sc/source/ui/dbgui/sortkeydlg.cxx
+++ b/sc/source/ui/dbgui/sortkeydlg.cxx
@@ -127,6 +127,7 @@
 sal_Int32 nVisibleItems = m_rScrolledWindow.getVisibleChildSize().Height() 
/ nScrollOffset;
 m_rVertScroll.SetPageSize( nVisibleItems - 1 );
 m_rVertScroll.SetVisibleSize( nVisibleItems );
+m_rVertScroll.Scroll();
 }
 
 // ---

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I96486f6edad11520bf9bb43b700b1716d9b2f170
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


Re: replacing OUString::valueOf(static_castsal_Int32) ??

2013-01-10 Thread Lubos Lunak
On Thursday 10 of January 2013, Noel Grandin wrote:
 On 2013-01-09 19:58, Michael Meeks wrote:
  At least in my mind :-) but we're starting to bike-shed here... I
  didn't see anyone volunteering to do the actual batch cleanup there
  ;-) Regards, Michael.

 Created a proof-of-concept patch along with some unit tests and pushed
 to gerrit:

 https://gerrit.libreoffice.org/#/c/1625/

 Can we please keep the discussion still here? Gerrit may be fine for pointing 
out technical details in the code, but it's not very suitable for discussions 
about anything beyond that.

- There's no need for valueOfChar(). There is already OUString ctor from 
sal_Unicode, so the valueOf() overload for it is just making an obvious thing 
complicated. Code using it can be converted to use the ctor instead.

- It's a question if we really need 'OUString::valueOfBool( foo )' instead of 
simply 'foo ? OUString( true ) : OUString( false )' (such a pity the 
string literals handling doesn't allow foo ? true : false' ). I wonder 
how many places in the code really need to convert a boolean to the hardcoded 
english string representation.

- When more or less deprecating valueOf() this way, it has also float 
overloads, so something should be created for those too.

- I'm still not sold on the naming, OUString::valueInt() doesn't say much and 
OUString::valueOfInt() feels cryptic. Can we please use something obvious 
that doesn't need decyphering, such as OUString::number() or 
OUString::fromInt() (as much as I still don't like the idea of harcoding the 
irrelevant type information in the name)?

On Thursday 10 of January 2013, Noel Grandin wrote:
 On Wed, Jan 9, 2013 at 7:58 PM, Michael Meeks michael.me...@suse.comwrote:
  At least in my mind :-) but we're starting to bike-shed here... I
   didn't see anyone volunteering to do the actual batch cleanup there ;-)

 Doesn't sound that hard - some regular expression magic will do most of the
 work.

 I expect it won't, regular expressions can't tell what foo is in valueOf( 
foo ). Unless all you want to convert is only places which do the explicit 
cast, this will need a (fairly simple) Clang plugin.

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


Re: gb_COMPILERNOOPTFLAGS does not even obey the user passed CFLAGS

2013-01-10 Thread Lubos Lunak
On Thursday 10 of January 2013, Tomáš Chvátal wrote:
 Hi guys,

 As written in subject the variable is used on multiple targets and it
 ignores the user defined cflags/cxxflags.
 This should never happen and all our targets should take into effect the
 user defined options too.

 I am not sure which variable I should append there, so could anyone check
 if the diff in attachment is right?

 It doesn't look right. CXXFLAGS should always get used for building .cxx 
sources, so this should have no effect, moreover gb_COMPILERNOOPTFLAGS is 
simply the default noopt flag to use when doing debug build. And I don't 
quite understand why you'd want this change, what problem are you trying to 
solve?

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


Re: replacing OUString::valueOf(static_castsal_Int32) ??

2013-01-10 Thread Noel Grandin


On 2013-01-10 15:55, Lubos Lunak wrote:

On Thursday 10 of January 2013, Noel Grandin wrote:

Created a proof-of-concept patch along with some unit tests and pushed
to gerrit:

https://gerrit.libreoffice.org/#/c/1625/

  Can we please keep the discussion still here? Gerrit may be fine for pointing
out technical details in the code, but it's not very suitable for discussions
about anything beyond that.
That's why it said proof of concept. Mostly, I did it to point that 
someone was willing to do the work :-)



- There's no need for valueOfChar(). There is already OUString ctor from
sal_Unicode, so the valueOf() overload for it is just making an obvious thing
complicated. Code using it can be converted to use the ctor instead.

Which doesn't help with overload resolution problems.



- When more or less deprecating valueOf() this way, it has also float
overloads, so something should be created for those too.

If those also suffer from overload resolution problems, then sure.
But
 git grep String::valueOf.*static_cast.*double
doesn't find anything that looks like it needs help.


- I'm still not sold on the naming, OUString::valueInt() doesn't say much and
OUString::valueOfInt() feels cryptic. Can we please use something obvious
that doesn't need decyphering, such as OUString::number() or
OUString::fromInt() (as much as I still don't like the idea of harcoding the
irrelevant type information in the name)?

Which would be inconsistent with the existing method names.



  I expect it won't, regular expressions can't tell what foo is in valueOf(
foo ).

Actually, regex can. It's called capturing groups.
http://www.regular-expressions.info/brackets.html
Doesn't work very well with complex expressions, but it can handle the 
simple cases.



Unless all you want to convert is only places which do the explicit
cast, this will need a (fairly simple) Clang plugin.

Sure, if you feel like writing one.


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


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


Re: Need confirmation about non-clearing of Character Styles

2013-01-10 Thread Michael Stahl
On 10/01/13 14:25, Caolán McNamara wrote:
 On Thu, 2013-01-10 at 10:55 +0100, Fernand Vanrie wrote:
 Sinds all or editors use LO 3.6.2 (before OO3.3)

 We have complains about the fact that parts of the text, who are
 formated with a Character Style, can not been cleared to
 no-formating,
 not by Clear Direct Formating (cntrl+M)

 There is a dirty work-around by applying the Default character style

 Paragraph Styles can been cleared
 Is this intentional behavior or a regression open for a issue ?
 
 My understanding is
 a) that its intentional, see
 http://lists.freedesktop.org/archives/ooo-build/2010-February/000540.html
 and the commit
 http://cgit.freedesktop.org/libreoffice/core/commit/?id=6e690e0909459d46c53d17e7939891abea11e566

the corresponding issues are an interesting read:

https://issues.apache.org/ooo/show_bug.cgi?id=47893 - patch author
arguing for the change
https://issues.apache.org/ooo/show_bug.cgi?id=85464 - a certain Cor
reporting the change as regression :)

i definitely think that there should be some button to reset formatting
that leaves character styles intact, and that should be the most easily
accessible option.  but i wouldn't object to a second button to reset
all formatting, including styles.

resetting the character styles should be as easy as setting Default
char style on the whole document, so i'm not sure if another option is
worth the additional clutter.

but hyperlinks should not be touched by any reset formatting since
they are not formatting, but content entities (the reset feature also
doesn't convert fields to plain text).

also i'm thinking that ruby text should also not be touched by reset
formatting, and this has not yet been changed (i.e. it is reset
currently); the reset will actually throw away the ruby text.  i'm
already sitting on a patch to remove ruby from the reset list...


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


Re: replacing OUString::valueOf(static_castsal_Int32) ??

2013-01-10 Thread Stephan Bergmann

On 01/10/2013 02:55 PM, Lubos Lunak wrote:

On Thursday 10 of January 2013, Noel Grandin wrote:

https://gerrit.libreoffice.org/#/c/1625/


  Can we please keep the discussion still here? Gerrit may be fine for pointing
out technical details in the code, but it's not very suitable for discussions
about anything beyond that.

- There's no need for valueOfChar(). There is already OUString ctor from
sal_Unicode, so the valueOf() overload for it is just making an obvious thing
complicated. Code using it can be converted to use the ctor instead.

- It's a question if we really need 'OUString::valueOfBool( foo )' instead of
simply 'foo ? OUString( true ) : OUString( false )' (such a pity the
string literals handling doesn't allow foo ? true : false' ). I wonder
how many places in the code really need to convert a boolean to the hardcoded
english string representation.

- When more or less deprecating valueOf() this way, it has also float
overloads, so something should be created for those too.

- I'm still not sold on the naming, OUString::valueInt() doesn't say much and
OUString::valueOfInt() feels cryptic. Can we please use something obvious
that doesn't need decyphering, such as OUString::number() or
OUString::fromInt() (as much as I still don't like the idea of harcoding the
irrelevant type information in the name)?


Btw, the name valueOf and the idea of overloading it for all 
conceivable (sal) types comes from the -- somewhat misguided -- attempt 
at modelling rtll::OUString after java.lang.String.


The fromInt etc. names look OK to me, too.

Re dropping valueOfBool, I'm not sure how many places actually use it; 
for i in solver/*/lib/*.so*; do nm $i -D --undef | grep -F 
_valueOfBoolean; done does indicate its not completely unused.  Also, 
we have rtl_ustr_valueOfBoolean around anyway.


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


Re: [libreoffice-marketing] Latest Hardware and OS

2013-01-10 Thread Rainer Bielefeld

Tom Davies schrieb:

At the moment those devices are probably not seen as good things to use to edit 
documents.




Hi all,

of course (nearby) nobody will use an Android Handy to write a Novel or 
similar; nevertheless, yesterday in the omnibus, I watched a young woman 
and was fascinated concerning the speed she wrote on a handy only using 
her thumbs. I'm afraid she was faster than me here typing with 10 fingers.


But I think an android tablet or even a smartphone can be quite a nice 
ODF document viewer, may be with smaller edits in the documents or so, 
you can use LibO for creation of simple documents from Templates 
(timesheet, directly signed by the customer on the screen in a DRAW OLE 
object ...).


And LibO seems not to be the spear head there, please see 
http://softmaker.com/english/ofacomp_en.htm!


On my Android tablet I use a different ODF viewer (forgot the name, 
result is not satisfying), because Versions from 
daily/master/Android-... crashed during installation).


So we need to interest more new additional Hackers helping to create a 
LibO Android suite based on the current Master, including Documentation, 
Project info, ... .


Best regards


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


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - config_host.mk.in configure.ac solenv/gbuild

2013-01-10 Thread Libreoffice Gerrit user
 config_host.mk.in   |1 +
 configure.ac|   19 +++
 solenv/gbuild/gbuild.mk |4 
 3 files changed, 24 insertions(+)

New commits:
commit f0fd3bd8f8a13f6cbe2a27018b83eee24a694c09
Author: Tomas Chvatal tchva...@suse.cz
Date:   Mon Dec 10 10:41:35 2012 +0100

Add configure switch that allows hardlink deliver

This switch just makes possible to configure in ln delivery instead of
cp based one.

The feature was already in the place, we just make it visible as
configure option now.

Change-Id: I778ef3b03e6b9a57057e66ff6307e7da46bcd935

diff --git a/config_host.mk.in b/config_host.mk.in
index 6ffd0fe..d7ede5a 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -215,6 +215,7 @@ export GUI=@GUI@
 export GUIBASE=@GUIBASE@
 export GUIBASE_FOR_BUILD=@GUIBASE_FOR_BUILD@
 export GUI_FOR_BUILD=@GUI_FOR_BUILD@
+export HARDLINKDELIVER=@HARDLINKDELIVER@
 export HAVE_CXX0X=@HAVE_CXX0X@
 export HAVE_GCC_AVX=@HAVE_GCC_AVX@
 export HAVE_GCC_BUILTIN_ATOMIC=@HAVE_GCC_BUILTIN_ATOMIC@
diff --git a/configure.ac b/configure.ac
index b028e39..9da5b39 100644
--- a/configure.ac
+++ b/configure.ac
@@ -651,6 +651,12 @@ AC_ARG_ENABLE(ext-wiki-publisher,
 
 dnl -- *** --
 
+AC_ARG_ENABLE([hardlink-deliver],
+AS_HELP_STRING([--enable-hardlink-deliver],
+[Put files into deliver folder as hardlinks instead of copying them
+over. Saves space and speeds up build.])
+)
+
 AC_ARG_ENABLE(mergelibs,
 AS_HELP_STRING([--enable-mergelibs],
 [Enables linking of big, merged, library. Experimental feature tested
@@ -11986,6 +11992,19 @@ else
 fi
 AC_SUBST(MERGELIBS)
 
+# ===
+# Create hardlinks on deliver instead of copying for smaller size and speed up
+# ===
+AC_MSG_CHECKING([whether to create hardlinks for delivering files])
+HARDLINKDELIVER=
+if test $enable_hardlink_deliver = yes; then
+HARDLINKDELIVER=TRUE
+AC_MSG_RESULT([yes])
+else
+AC_MSG_RESULT([no])
+fi
+AC_SUBST(HARDLINKDELIVER)
+
 dnl ===
 dnl icerun is a wrapper that stops us spawning tens of processes
 dnl locally - for tools that can't be executed on the compile cluster
diff --git a/solenv/gbuild/gbuild.mk b/solenv/gbuild/gbuild.mk
index 321d42c..173222f 100644
--- a/solenv/gbuild/gbuild.mk
+++ b/solenv/gbuild/gbuild.mk
@@ -132,6 +132,10 @@ ENABLE_DEBUGINFO_FOR := all
 endif
 endif
 
+ifeq ($(HARDLINKDELIVER),TRUE)
+gb_Deliver_HARDLINK := $(true)
+endif
+
 ifeq ($(or $(ENABLE_SYMBOLS),$(enable_symbols)),FALSE)
 gb_SYMBOL := $(false)
 else
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: cppcanvas/source

2013-01-10 Thread Libreoffice Gerrit user
 cppcanvas/source/mtfrenderer/emfplus.cxx |   45 +++
 1 file changed, 45 insertions(+)

New commits:
commit da09115cffeca384ab91ffe4d7af4d8eab6cee82
Author: Radek Doulik r...@novell.com
Date:   Thu Jan 10 15:58:30 2013 +0100

implemented EmfPlusRecordTypeFillPie record

Change-Id: I15e7464a6a295bb4c1313c0a65ee33b6443c61c8

diff --git a/cppcanvas/source/mtfrenderer/emfplus.cxx 
b/cppcanvas/source/mtfrenderer/emfplus.cxx
index 3c791d4..f1b0eff 100644
--- a/cppcanvas/source/mtfrenderer/emfplus.cxx
+++ b/cppcanvas/source/mtfrenderer/emfplus.cxx
@@ -29,6 +29,7 @@
 #include basegfx/range/b2drange.hxx
 #include basegfx/range/b2drectangle.hxx
 #include basegfx/polygon/b2dpolygon.hxx
+#include basegfx/polygon/b2dpolygontools.hxx
 #include basegfx/polygon/b2dpolypolygon.hxx
 #include basegfx/polygon/b2dpolypolygontools.hxx
 #include vcl/canvastools.hxx
@@ -52,6 +53,7 @@
 #define EmfPlusRecordTypeFillRects 16394
 #define EmfPlusRecordTypeFillPolygon 16396
 #define EmfPlusRecordTypeDrawLines 16397
+#define EmfPlusRecordTypeFillPie 16400
 #define EmfPlusRecordTypeFillPath 16404
 #define EmfPlusRecordTypeDrawPath 16405
 #define EmfPlusRecordTypeDrawImage 16410
@@ -93,6 +95,7 @@
 #endif
 
 using namespace ::com::sun::star;
+using namespace ::basegfx;
 
 namespace cppcanvas
 {
@@ -1234,6 +1237,48 @@ namespace cppcanvas
 case EmfPlusRecordTypeObject:
 processObjectRecord (rMF, flags);
 break;
+case EmfPlusRecordTypeFillPie:
+{
+sal_uInt32 brushIndexOrColor;
+float startAngle, sweepAngle;
+
+rMF  brushIndexOrColor  startAngle  sweepAngle;
+
+EMFP_DEBUG (printf (EMF+ FillPie colorOrIndex: %x 
startAngle: %f sweepAngle: %f\n, (unsigned int)brushIndexOrColor, startAngle, 
sweepAngle));
+
+float dx, dy, dw, dh;
+
+ReadRectangle (rMF, dx, dy, dw, dh, flags  0x4000);
+
+EMFP_DEBUG (printf (EMF+ RectData: %f,%f %fx%f\n, 
dx, dy, dw, dh));
+
+startAngle = 2*M_PI*startAngle/360;
+sweepAngle = 2*M_PI*sweepAngle/360;
+
+B2DPoint mappedCenter (Map (dx + dw/2, dy + dh/2));
+B2DSize mappedSize( MapSize (dw/2, dh/2));
+
+double endAngle = startAngle + sweepAngle;
+if (endAngle  0)
+endAngle += M_PI*2;
+endAngle = fmod (endAngle, M_PI*2);
+
+if (sweepAngle  0) {
+double tmp = startAngle;
+startAngle = endAngle;
+endAngle = tmp;
+}
+
+EMFP_DEBUG (printf (EMF+ angles: %f,%f  --- 
%f,%f\n, startAngle, sweepAngle, startAngle, endAngle));
+
+B2DPolygon polygon = 
tools::createPolygonFromEllipseSegment (mappedCenter, mappedSize.getX (), 
mappedSize.getY (), startAngle, endAngle);
+polygon.append (mappedCenter);
+polygon.setClosed (true);
+
+B2DPolyPolygon polyPolygon (polygon);
+EMFPPlusFillPolygon (polyPolygon, rFactoryParms, 
rState, rCanvas, flags  0x8000, brushIndexOrColor);
+}
+break;
 case EmfPlusRecordTypeFillPath:
 {
 sal_uInt32 index = flags  0xff;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [libreoffice-l10n] Translation of the names of the keys on Windows

2013-01-10 Thread Adolfo Jayme Barrientos
On Tue, Jan 8, 2013 at 7:44 AM, Jesús Corrius je...@softcatala.org wrote:
 If you want to implement this, but you are not a developer, just send me
 the name of the keys with its translation and I will apply the changes
 myself.

Hi, I was wondering if you could add a Spanish translation:

static const struct KeysNameReplacement aImplReplacements_Spanish[] =
{
{ KEY_BACK, Retroceso },
{ KEY_ENTER, Intro },
{ KEY_SPACEBAR, Espacio },
{ KEY_HOME, Inicio },
{ KEY_UP, Arriba },
{ KEY_PAGEUP, Re Pág },
{ KEY_LEFT, Izquierda },
{ KEY_RIGHT, Derecha },
{ KEY_END, Fin },
{ KEY_DOWN, Abajo },
{ KEY_PAGEDOWN, Av Pág },
{ KEY_INSERT, Ins },
{ KEY_DELETE, Supr },
{ KEY_SHIFT, Mayús },
};

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


[Libreoffice-commits] .: sw/source

2013-01-10 Thread Libreoffice Gerrit user
 sw/source/core/crsr/pam.cxx |8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

New commits:
commit fd0da52d371ca13d66be3b04d89f9d9cf73baddd
Author: Miklos Vajna vmik...@suse.cz
Date:   Thu Jan 10 16:21:55 2013 +0100

fdo#57938 SwPaM::HasReadonlySel: commented text ranges are not read-only

diff --git a/sw/source/core/crsr/pam.cxx b/sw/source/core/crsr/pam.cxx
index e9b7802..852 100644
--- a/sw/source/core/crsr/pam.cxx
+++ b/sw/source/core/crsr/pam.cxx
@@ -686,6 +686,7 @@ sal_Bool SwPaM::HasReadonlySel( bool bFormView ) const
 sw::mark::IMark* pA = NULL;
 sw::mark::IMark* pB = NULL;
 bool bUnhandledMark = false;
+bool bCommentrangeMark = false;
 if ( pDoc )
 {
 const IDocumentMarkAccess* pMarksAccess = 
pDoc-getIDocumentMarkAccess( );
@@ -694,7 +695,11 @@ sal_Bool SwPaM::HasReadonlySel( bool bFormView ) const
 
 sw::mark::IFieldmark* pFieldmark = pMarksAccess-getFieldmarkFor( 
*GetPoint() );
 if ( pFieldmark )
+{
 bUnhandledMark = pFieldmark-GetFieldname( ) == ODF_UNHANDLED;
+if (!bUnhandledMark)
+bCommentrangeMark = pFieldmark-GetFieldname() == 
ODF_COMMENTRANGE;
+}
 }
 
 if (!bRet)
@@ -702,7 +707,8 @@ sal_Bool SwPaM::HasReadonlySel( bool bFormView ) const
 // Unhandled fieldmarks case shouldn't be edited manually to avoid 
breaking anything
 if ( ( pA == pB )  bUnhandledMark )
 bRet = sal_True;
-else
+// Allow editing of commented ranges.
+else if (!((pA == pB)  bCommentrangeMark))
 {
 // Form protection case
 bool bAtStartA = pA != NULL  pA-GetMarkStart() == *GetPoint();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: replacing OUString::valueOf(static_castsal_Int32) ??

2013-01-10 Thread Lubos Lunak
On Thursday 10 of January 2013, Noel Grandin wrote:
 On 2013-01-10 15:55, Lubos Lunak wrote:
  - There's no need for valueOfChar(). There is already OUString ctor from
  sal_Unicode, so the valueOf() overload for it is just making an obvious
  thing complicated. Code using it can be converted to use the ctor
  instead.

 Which doesn't help with overload resolution problems.

 It does. If you say it's wrong to use valueOfWhatever() for char-string 
conversion, there either will not be problems, or it will be wrong. If you 
want to fix an existing problem by introducing a new function, how does that 
make anything better than using an already existing function that does the 
job?

  - When more or less deprecating valueOf() this way, it has also float
  overloads, so something should be created for those too.

 If those also suffer from overload resolution problems, then sure.
 But
   git grep String::valueOf.*static_cast.*double
 doesn't find anything that looks like it needs help.

  - I'm still not sold on the naming, OUString::valueInt() doesn't say much
  and OUString::valueOfInt() feels cryptic. Can we please use something
  obvious that doesn't need decyphering, such as OUString::number() or
  OUString::fromInt() (as much as I still don't like the idea of harcoding
  the irrelevant type information in the name)?

 Which would be inconsistent with the existing method names.

 Oh. I think we may be talking past each others because we have different 
goals. Am I correct in the assumption that you merely want to add another set 
of functions that people can use whenever the originals don't work?

 I do not think it is a good idea to just do a quick hack to paper over a 
problem. The strings are one of the most basic classes in the LO code, and if 
we are going to do changes there, we may as well try to do them properly. 
Base classes are not something we should randomly throw changes at, and we 
can still spend ages fixing up all the design decisions in the string classes 
that already are there and could have been done better. So while I appreciate 
it that you're willing to write a kludge that'll help avoid a problem, I kind 
of consider it a waste of your time, when with just little more effort a 
proper solution could be created.

 If you just add another set of functions, it will be confusing which one to 
use. And people will still sometimes use the old one out of habbit or for 
whatever reason, and still the problem will occassionally show up, and then 
the code will need to be changed to the new function, which is not really 
that different to just adding a cast. And if you say that people should 
always directly use the new one, then you may as well remove the old one 
(except keep it for extensions backwards compatibility), and then all I've 
said applies (no need for char function, float function needed, use a good 
name, etc.).

 Or did I misunderstand the intent of your patch?

I expect it won't, regular expressions can't tell what foo is in
  valueOf( foo ).

 Actually, regex can. It's called capturing groups.

 Capturing groups may tell you that foo is foo, but it won't tell you it's a 
float. So how will you know to which of your proposed valueX() functions you 
should change it?

  Unless all you want to convert is only places which do the explicit
  cast, this will need a (fairly simple) Clang plugin.

 Sure, if you feel like writing one.

 Actually, I'd prefer to write a howto about that first, whenever I get to 
doing that, so that I don't have to write every single plugin. Such a plugin 
will be still much simpler than a regexp or any other way.

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


RE: [libreoffice-dev] - questions about calc extension

2013-01-10 Thread Rai, Neeraj
Hi Stephan,

I am unable to find XDesktop2. Your sample below looks really simple and right 
way to go.
I am working with release version 3.6.4.3 - built it from tar files.
I don't mind upgrading to latest tip, but I had trouble building the git 
checkout version.
Being fairly new to this large and comprehensive product, I decided to take the 
easy way to begin.

I would also be curious to know how to find out about such api - unfortunately, 
right now I am reading only specific emails on dev group and unable to follow 
conversations on other topics. Is reading docs the best way ? Is it possible to 
find some sample code ?

Kohei seems to have a working version with source below. I had trouble finding 
source for the other calc extensions listed on site.
I understand that situation is complicated with taking over working code and 
existing docs. It may take some time to get those things in order.
I am a user of LO api and don't want to look like demanding regime change (like 
LO spring driven by nabble instead of facebook :-))
If it is feasible to dedicate space for source of extensions, I have a feeling 
space requirements there might grow quickly.
That said, I know it may take time getting these one off things done with 
everyone having current deadlines and some of you are working on this part time.
The current examples that come bundled are all great. They focus on single 
feature and demonstrate it well. All the ones I tried, compiled.

Thanks
Neeraj

-Original Message-
From: libreoffice-bounces+neeraj.rai=citi@lists.freedesktop.org 
[mailto:libreoffice-bounces+neeraj.rai=citi@lists.freedesktop.org] On 
Behalf Of Stephan Bergmann
Sent: Thursday, January 10, 2013 3:45 AM
To: libreoffice@lists.freedesktop.org
Subject: Re: [libreoffice-dev] - questions about calc extension

On 01/09/2013 07:41 PM, Kohei Yoshida wrote:
 On 01/09/2013 01:12 PM, Rai, Neeraj wrote:
 What is the correct way to get a
 handle to spreadsheet in extension code.

 I don't want you to look too close to this (since I no longer maintain
 this), bug this code of mine

 http://code.google.com/p/scsolver/source/browse/source/ui/xcalc.cxx

 may give you some hint, especially the top 3 methods,
 getCurrentComponent(), updateCurrentComponent(), and getServiceManager()
 should give you how to get the current component object from the current
 context object.  As an extension, you are given access to the current
 context object, and you can reach the current component from that one
 way or another.

Some comments on the above code, for the benefit of fresh development:

Obtaining the com.sun.star.frame.Desktop service (variable xDesktop) in
updateCurrentComponent can be greatly simplified through the use of
so-called new-style service constructors (and Noel's ongoing work of
changing existing old-style services into new-style ones in UNOIDL):

   Reference frame::XDesktop2  xDesktop(
   frame::Desktop::create(m_xCC));

- It already returns the proper interface of the service (in this case
XDesktop2, derived from XDesktop), so no need for querying from
XInterface to the correct type.

- It directly uses the component context, no need for the service
manager.  (In general, demand for explicitly using the service manager
should go away more and more over time.)

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


[PATCH] Change in core[libreoffice-4-0]: fdo#57938 SwPaM::HasReadonlySel: commented text ranges are n...

2013-01-10 Thread Miklos Vajna (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1629

To pull it, you can do:

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

fdo#57938 SwPaM::HasReadonlySel: commented text ranges are not read-only

(cherry picked from commit fd0da52d371ca13d66be3b04d89f9d9cf73baddd)

Change-Id: I8191ed6383c4c636be67b8d42654dbba0a5c8b6f
---
M sw/source/core/crsr/pam.cxx
1 file changed, 7 insertions(+), 1 deletion(-)



diff --git a/sw/source/core/crsr/pam.cxx b/sw/source/core/crsr/pam.cxx
index baf62ad..6d7f07a 100644
--- a/sw/source/core/crsr/pam.cxx
+++ b/sw/source/core/crsr/pam.cxx
@@ -685,6 +685,7 @@
 sw::mark::IMark* pA = NULL;
 sw::mark::IMark* pB = NULL;
 bool bUnhandledMark = false;
+bool bCommentrangeMark = false;
 if ( pDoc )
 {
 const IDocumentMarkAccess* pMarksAccess = 
pDoc-getIDocumentMarkAccess( );
@@ -693,7 +694,11 @@
 
 sw::mark::IFieldmark* pFieldmark = pMarksAccess-getFieldmarkFor( 
*GetPoint() );
 if ( pFieldmark )
+{
 bUnhandledMark = pFieldmark-GetFieldname( ) == ODF_UNHANDLED;
+if (!bUnhandledMark)
+bCommentrangeMark = pFieldmark-GetFieldname() == 
ODF_COMMENTRANGE;
+}
 }
 
 if (!bRet)
@@ -701,7 +706,8 @@
 // Unhandled fieldmarks case shouldn't be edited manually to avoid 
breaking anything
 if ( ( pA == pB )  bUnhandledMark )
 bRet = sal_True;
-else
+// Allow editing of commented ranges.
+else if (!((pA == pB)  bCommentrangeMark))
 {
 // Form protection case
 bool bAtStartA = pA != NULL  pA-GetMarkStart() == *GetPoint();

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8191ed6383c4c636be67b8d42654dbba0a5c8b6f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Miklos Vajna vmik...@suse.cz

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


Re: gb_COMPILERNOOPTFLAGS does not even obey the user passed CFLAGS

2013-01-10 Thread Lubos Lunak
On Thursday 10 of January 2013, Tomáš Chvátal wrote:
 2013/1/10 Lubos Lunak l.lu...@suse.cz:
  As written in subject the variable is used on multiple targets and it
  ignores the user defined cflags/cxxflags.
  This should never happen and all our targets should take into effect the
  user defined options too.
 
  I am not sure which variable I should append there, so could anyone
  check if the diff in attachment is right?
 
   It doesn't look right. CXXFLAGS should always get used for building .cxx
  sources, so this should have no effect, moreover gb_COMPILERNOOPTFLAGS is
  simply the default noopt flag to use when doing debug build. And I don't
  quite understand why you'd want this change, what problem are you trying
  to solve?

 See how bridgetests are build,

 they fail on old systems because they just specify the -O0 and nothing
 else, not even march

 So Petr used this patch in build service [1] but i think we should
 make this respected everywhere.

 Reading this patch, your change looks to me like putting the CXXFLAGS in some 
random place where it will incidentally work, but the moment somebody doesn't 
include gb_COMPILERNOOPTFLAGS in the flags passed to add_cxxobject, the 
problem is back again. If add_cxxobject doesn't include CXXFLAGS, why not fix 
it to ensure that (which possibly may need others like add_noexception_object 
to delegate to add_cxxobject_internal instead of add_cxxobject)?

 [1]
 https://build.opensuse.org/package/view_file?file=bridges-missing-cxxflags.
diffpackage=libreofficeproject=LibreOffice%3AUnstablerev=157

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


Re: Extending Dictionaries

2013-01-10 Thread Petr Mladek
Samuel Mehrbrodt píše v Čt 03. 01. 2013 v 23:30 +0100:
 I just sent you a list of words directly.
 
 Can we document this somewhere in the Wiki, how to add new words to a 
 dictionary? I guess there are more people wanting to do this.

I have created
https://wiki.documentfoundation.org/Development/Dictionaries
Feel free to update it or ask if anything is unclear.

 Also, are there any guidelines which words are accepted in the dictionary?

The preferred way is to contact author of the dictionary and get the
words accepted by him. He is usually an expert for the given language
and the best person for decision making. Also it helps to make sure that
the new words are not lost within further updates.

By other words, LO git repository includes just a copy of the
dictionary. It is better to change in on the original location where it
come from.

Best Regards,
Petr


 Thanks
 Samuel
 
 Am 03.01.2013 22:30, schrieb Bjoern Jacke:
  On 2013-01-03 at 17:30 +0100 Samuel Mehrbrodt sent off:
  If there are no objections from the plugin authors, I'll create a
  patch against the dictionaries file.
  please jsut send me the list of words. Patching dictionaries doesn't make 
  much
  sense for getting that upstream. The dictionary is a compiled version of 
  the
  dictionaries of igerman98.
 
  Cheers
  Björn
 


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


[PATCH] Change in core[libreoffice-4-0]: Resolves: fdo#58301 make sure ScrollBar OptimalSize is up to...

2013-01-10 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1630

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/30/1630/1

Resolves: fdo#58301 make sure ScrollBar OptimalSize is up to date

if the scrollbar sizes haven't been calculated yet then the optimalsize will be
wrong, so it's necessary to enforce calculating them in that case.

(cherry picked from commit f41a2804dd4a23352b7af2dd28c03df02debfb13)

Change-Id: I12e95c3594ffbb508c7f79bd512e07aaf44a6eab
---
M vcl/inc/vcl/scrbar.hxx
M vcl/source/control/scrbar.cxx
2 files changed, 11 insertions(+), 3 deletions(-)



diff --git a/vcl/inc/vcl/scrbar.hxx b/vcl/inc/vcl/scrbar.hxx
index 8dbe270..7c62d94 100644
--- a/vcl/inc/vcl/scrbar.hxx
+++ b/vcl/inc/vcl/scrbar.hxx
@@ -83,8 +83,9 @@
 SAL_DLLPRIVATE long ImplDoAction( sal_Bool bCallEndScroll );
 SAL_DLLPRIVATE void ImplDoMouseAction( const Point rPos, sal_Bool 
bCallAction = sal_True );
 SAL_DLLPRIVATE void ImplInvert();
-SAL_DLLPRIVATE sal_Bool ImplDrawNative( sal_uInt16 nDrawFlags );
+SAL_DLLPRIVATE sal_Bool ImplDrawNative( sal_uInt16 nDrawFlags );
 SAL_DLLPRIVATE void ImplDragThumb( const Point rMousePos );
+SAL_DLLPRIVATE Size getCurrentCalcSize() const;
 DECL_DLLPRIVATE_LINK(   ImplTimerHdl, Timer* );
 DECL_DLLPRIVATE_LINK(   ImplAutoTimerHdl, void* );
 
diff --git a/vcl/source/control/scrbar.cxx b/vcl/source/control/scrbar.cxx
index 9e90362..b99ff26 100644
--- a/vcl/source/control/scrbar.cxx
+++ b/vcl/source/control/scrbar.cxx
@@ -323,7 +323,7 @@
 Rectangle maTrackRect = mpData-maTrackRect;  // TODO: remove when 
maTrackRect is no longer in mpData
 if ( mbCalcSize )
 {
-Size aOldSize = GetOptimalSize(WINDOWSIZE_PREFERRED);
+Size aOldSize = getCurrentCalcSize();
 
 const Rectangle aControlRegion( Point(0,0), aSize );
 Rectangle aBtn1Region, aBtn2Region, aTrackRegion, aBoundingRegion;
@@ -416,7 +416,7 @@
 
 mbCalcSize = sal_False;
 
-Size aNewSize = GetOptimalSize(WINDOWSIZE_PREFERRED);
+Size aNewSize = getCurrentCalcSize();
 if (aOldSize != aNewSize)
 {
 queue_resize();
@@ -1509,6 +1509,13 @@
 
 Size ScrollBar::GetOptimalSize(WindowSizeType) const
 {
+if (mbCalcSize)
+const_castScrollBar*(this)-ImplCalc(sal_False);
+return getCurrentCalcSize();
+}
+
+Size ScrollBar::getCurrentCalcSize() const
+{
 Rectangle aCtrlRegion;
 aCtrlRegion.Union(maBtn1Rect);
 aCtrlRegion.Union(maBtn2Rect);

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I12e95c3594ffbb508c7f79bd512e07aaf44a6eab
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


Re: [libreoffice-dev] - questions about calc extension

2013-01-10 Thread Kohei Yoshida

On 01/10/2013 10:50 AM, Rai, Neeraj wrote:

Hi Michael,

Appreciate your feedback.
I ran callgrind but no output on screen and 0 bytes files generated. I can run 
it on non-LO exe.
I'll compile LO with --enable-debug and see if situation improves. Please 
advise if other steps might be needed.

What's your hunch about how long it should take to update 60K cells with 
{getCellByPosition(ii,jj) and setValue() } ?


Two things you might try.

1) Try getDataArray() and setDataArray() to update multiple contiguous 
cells.  Doing it this way is much much faster than updating one cell at 
a time.  These methods are available for cell range object.


2) Try lockControllers() and unlockControllers() to disable/enable 
repainting while updating cell values.  The example of these methods 
should be in my xcalc.cxx file I referenced earlier.


HTH,

Kohei

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


  1   2   3   4   >