Re: [Libreoffice] [RFC] npapi-sdk instead of xulrunner
On 09/29/2011 06:01 PM, Tomáš Chvátal wrote: Hi, I was noticed that there is package called npapi-sdk [1] that can be used as firefox/mozilla headers instead of requiring whole beast to build the plugins. Do you think it would be possible to use this in libreoffice, or is it impossible? I don't understand the nsplugin part much, and it was just reported by users at gentoo [2]. There are only 4 C header files that are needed to build a NPAPI plugin. They may as well be shipped inside LibO and not bother with external deps for this - it would mean adding another build/configure test for npapi-sdk along with the ones looked for now. The NPAPI API is stable so an internal copy should be fine. But I think mozilla build-dep was needed anyway for LDAP, I don't know if that dependency is obsolete or not. Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] reviewboard: a solution or just creating new problems?
On 04/21/2011 07:17 PM, Bjoern Michaelsen wrote: Hi all, I just want to bring up an idea of which I am not really sure if it is a good one. Currently we have new contributors writing patches to the dev-ML and grant them direct repository access later. Commiting directly on the master might be quite scary at first, so I wondered if something like: http://www.reviewboard.org/ might help the migration. It has a nifty interface (look at their page to get a sales pitch) and all that jazz, but: - it would be another system to setup and maintain - it would require its own logins (or integration in the existing infra) - lots of other things I have not considered yet. So -- I am not sure, if it is a good idea. I just wanted to get this of my chest, so that I wont be sorry later for not speaking up about it. I think using code review helper apps is a great idea, and the lack of their use in any non-trivial FOSS project with many contributors will be in 5 years as perplexing as seeing one now going without the use version control software :) I had used Gerrit a bit and I found it very nice. Easy to deploy (Java) I did not use Reviewboard so I cannot compare. Gerrit was made to work with git specifically and is used by the Android developer ecosystem. There is some work to set up and to maintain, but I think commiters would be happy to be able to just Ack a patch in a web ui and have the commit happen automatically. One drawback is the some of the feedback that now happens inline on the mailing list for all to learn from, would move to the webapp in merge request comments. To a certain degree that would be similar to feedback that happens in bugzilla so maybe not a terrible loss. Very specific code related comments vs higher level discussion on the devel list. I think having such a system set up in demo/trial mode would have a better chance of generating feedback. Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] [PUSHED] remove unused declaration - an unlikely fix for armel segfault in regcomp
On 03/09/2011 06:01 PM, Caolán McNamara wrote: On Wed, 2011-03-09 at 10:46 +0200, Jani Monoses wrote: On 01/29/2011 03:04 PM, Caolán McNamara wrote: On Thu, 2011-01-27 at 20:13 +0200, Jani Monoses wrote: What about using GCC atomic builtins like in the attached patch? Excellent, the pthread fallback always niggled me. Sorry for not thinking about this earlier, can this be cherry-picked into the 3.3 branch? This doesn't affect a lot of people in practice, only the Linux non-x86/x86_64 people IIRC, +1 from me if you want to stick it into 3-3. Great. Who can do the actual sticking into 3.3 (I can't) ? This was the master commit: 788072cefdce8cb61d46549a7aede4c754d9fae3 thanks Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] [PUSHED] remove unused declaration - an unlikely fix for armel segfault in regcomp
On 01/29/2011 03:04 PM, Caolán McNamara wrote: On Thu, 2011-01-27 at 20:13 +0200, Jani Monoses wrote: What about using GCC atomic builtins like in the attached patch? Excellent, the pthread fallback always niggled me. Sorry for not thinking about this earlier, can this be cherry-picked into the 3.3 branch? Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] [PATCH] remove unused declaration - an unlikely fix for armel segfault in regcomp
On 01/27/2011 05:31 PM, Caolán McNamara wrote: > On Thu, 2011-01-27 at 15:42 +0200, Jani Monoses wrote: >> armel > > sal/osl/unx/interlck.c probably should have a specific impl for arm btw, > probably a low hanging fruit for someone who knows a little bit of arm > assembly. What about using GCC atomic builtins like in the attached patch? If correct (seems so, but I did no test it beyond building it - are there tests for this in LibO?) it has the advantage of covering more than ARM, and for ARM letting gcc emit optimal (I hope) code by taking care of differences between atomic exchange primitives on ARM pre v6 (SWP) and newer (STREX/LDREX) and memory barrier instructions (DMB vs MCR ). Debian still builds for armv5 while Ubuntu for armv7 so there would have been at least some ifdefs if assembly were used. IMHO it is better to stick to portable C/C++ if it does not constitute a drawback otherwise :) Jani >From 6570fea4a5851db6dce6f2c9e69133115026ff63 Mon Sep 17 00:00:00 2001 From: Jani Monoses Date: Thu, 27 Jan 2011 20:02:10 +0200 Subject: [PATCH] Add oslInterlockedCount inc/dec functions using GCC atomic builtins --- osl/unx/interlck.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/osl/unx/interlck.c b/osl/unx/interlck.c index f164371..51c1c9e 100644 --- a/osl/unx/interlck.c +++ b/osl/unx/interlck.c @@ -135,6 +135,16 @@ oslInterlockedCount SAL_CALL osl_decrementInterlockedCount(oslInterlockedCount* return nCount; } +#elif defined ( GCC ) +oslInterlockedCount SAL_CALL osl_incrementInterlockedCount(oslInterlockedCount* pCount) +{ + return __sync_add_and_fetch(pCount, 1); +} + +oslInterlockedCount SAL_CALL osl_decrementInterlockedCount(oslInterlockedCount* pCount) +{ + return __sync_sub_and_fetch(pCount, 1); +} #else /* use only if nothing else works, expensive due to single mutex for all reference counts */ -- 1.7.2.3 ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] [PUSHED] remove unused declaration - an unlikely fix for armel segfault in regcomp
Hi Michael, thanks for pushing to 3.3 When there's a relase with the patch included, it should close this issue I opened a few days ago https://bugs.freedesktop.org//show_bug.cgi?id=33402 > > Having said that, it is already fixed in master, so I just merged your > patch to libreoffice-3-3 - so it will be in 3.3.1. > ah, I did not know this was already fixed in master, I worked on the rc4 release. > Great to have you contributing, and looking forward to any other funky > ARM pieces you have. Out of interest - did the unit tests execute > correctly during the build ? The smoke-tests (if that is what you are referring to) are currently disabled in the Ubuntu package build. I have very little knowledge of LibO and packaging internals, this debugging adventure was 'a vertical' :) Other than that everything up to the install phase executed correctly, and I did not get to test yet what is after installing - a full rebuild takes the better part of the day even if this is among the faster arm machines (pandaboard) Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Libreoffice] [PATCH] remove unused declaration - an unlikely fix for armel segfault in regcomp
Hello, this declaration seems to not be used. A similar one exists in the file token.cxx in formula/ where this was copied from according to the comments. This, in combination with some toolchain issue I guess caused regcomp crashing on Ubuntu armel at the install phase. >From what I gathered so far the FixedMemPool destructor for the first instance (in formula's token.cxx) was overwritten by the destructor in sc's token.cxx. When running these destructors in regcomp's atexit/fini, the formula one was never called, while the sc one was called twice resulting in the segfault. This only occured if the registry file was not already there, in which case there was no crash but arena tracing printed out bad memory anyway. I don't know if this was the case for other arch's as well and they were just lucky in avoiding the segfault, or it's an armel toolchain issue at play as well. Jani >From 7305b53aad89f4c0362e6126edc8f1930eed22f4 Mon Sep 17 00:00:00 2001 From: Jani Monoses Date: Thu, 27 Jan 2011 15:33:06 +0200 Subject: [PATCH] Remove unused declaration --- sc/source/core/tool/token.cxx |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/sc/source/core/tool/token.cxx b/sc/source/core/tool/token.cxx index 3298517..48021b1 100644 --- a/sc/source/core/tool/token.cxx +++ b/sc/source/core/tool/token.cxx @@ -110,9 +110,6 @@ namespace // } // namespace // -// ImpTokenIterator wird je Interpreter angelegt, mehrfache auch durch -// SubCode via FormulaTokenIterator Push/Pop moeglich -IMPL_FIXEDMEMPOOL_NEWDEL( ImpTokenIterator, 32, 16 ) // Align MemPools on 4k boundaries - 64 bytes (4k is a MUST for OS/2) -- 1.7.2.3 ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] LibreOffice and Wayland support
On 01/25/2011 11:13 AM, Alexander Thurgood wrote: > Le 25/01/11 08:24, Jonathan Aquilina a écrit : > > Hi Jonathan, > >> Correct me if I am wrong but wont LO be a little screwed since LO only >> supports X from what I have seen codewise? > > That was precisely my point : applications have to be rewritten to work > with Wayland. At least LibreOffice won't be the only application > concerned, which is some consolation !! Applications using more popular toolkits will have an easier transition though, as the porting will mostly be done in those toolkits' code. Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] getting LO into debian
The OPs mail simply shows Ubuntus attitude and cluelessless, both reasons for why they have to die. http://www.youtube.com/watch?v=zrzMhU_4m-g#t=1m20s ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] getting LO into debian
On 11/27/2010 05:36 PM, Rene Engelhard wrote: Hi, On Sat, Nov 27, 2010 at 04:22:06PM +0100, Rene Engelhard wrote: On Sat, Nov 27, 2010 at 04:08:37PM +0100, Jonathan Aquilina wrote: hey guys what is being done to get LO into debian so that downstream It already for almost a month. to make that more precise: first upload to Debian on *Oct, 13*. Accepted (because NEW package) etc.: Nov, 8. That was beta2. Beta3 got accepted on Nov, 18. http://packages.qa.debian.org/libr/libreoffice.html http://packages.debian.org/libreoffice ubuntu can get it pulled from the upstream debian repository? Ask them, not me. If they can't do a simple pull in one month, though they are paid for doing it and /me who did already the major work (also at nights!) for it, well.. There may be exceptions but I think Ubuntu pulls from sid not from experimental as a rule. When the package appears in sid it will probably be available in Ubuntu devel version in a short time. Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] patches as attachments or mails? (Poll included)
On 11/08/2010 10:37 AM, Sebastian Spaeth wrote: I think, I'll add a "how to prepare and send patches" section to the wiki, as that causes lots of confusion. Is there consensus by committers whether we want to have patches attached to a mail or sent inline with "git send-email"? Depending on the outcome, I'll write up the recommendation accordingly. Could pulling from repos/branches be considered as a way of contributing? Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] LibreOffice and Java
On 11/03/2010 04:06 PM, Caolán McNamara wrote: On Tue, 2010-11-02 at 14:30 +0100, Cedric Bosdonnat wrote: On Tue, 2010-11-02 at 12:47 +, Michael Meeks wrote: Quite; cf. such uncertainty - it probably makes considerable sense to look into a migration strategy from Java to (insert anything else). Some candidates might be python for the more scripty pieces (though I hate non-typed languages), and/or native C++. What about the existing Java extensions? Quite a lot of people are using Java to either writer extensions or use UNO Java bridge from an external application. Its not, at least that's my understanding, about removing the java-uno bridge, or removing the *ability* to support Java. Its the thought to consider not defaulting to *require* Java for the built-in bits, which probably mostly boils down to defaulting base to a different backend for file based databases and a redo an occasional wizard here and there. Out of the over 3400 java files in the repos, over half seem to be tests (some in directories called test/ others in qa/ - is there a difference?), and over 200 are wizard related, so it seems a lot of UI work to port these if they are all indeed part of the default install. Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Libreoffice] supported compilers?
Is there a list of which compilers are supported and what versions recommeneded for building LO? I was thinking of the feasibility of using gcc's __builtin_constant_p in a macro to differentiate fast and not so fast paths for createFromAscii cases to keep the code uniform but still get the optimizations, but then figured that would not work with Visual Studio. Jani Ex: #include #define CONSTACTION(str) printf("%s is const\n", str) #define VARACTION(str) printf("%s is not known to be const\n", str) #define ACTION(str) if (__builtin_constant_p(str)) {\ CONSTACTION(str); \ } else {\ VARACTION(str);\ } int main(int argc, char *argv[]) { char *s = "ss"; ACTION("cc"); ACTION(s); return 0; } ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Libreoffice] [PATCH] Include header file
Just like the other .cxx files in starmath, make this one include unomodel.hxx and get the definition of the A2OU macro from there instead of redefining it. I hope ommitting this header inclusion was not intentional, it builds at least with it included :) Jani >From 753c0a3ff6704045d3837581a2299f4c20b76452 Mon Sep 17 00:00:00 2001 From: Jani Monoses Date: Sat, 30 Oct 2010 01:55:47 +0300 Subject: [PATCH 1/2] Include header file instead of redefining a macro. --- starmath/source/accessibility.cxx |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/starmath/source/accessibility.cxx b/starmath/source/accessibility.cxx index 6b8f72b..75311fc 100644 --- a/starmath/source/accessibility.cxx +++ b/starmath/source/accessibility.cxx @@ -61,6 +61,7 @@ #include "accessibility.hxx" #include +#include #include #include @@ -70,8 +71,6 @@ using namespace com::sun::star::lang; using namespace com::sun::star::uno; using namespace com::sun::star::accessibility; -#define A2OU(cChar) rtl::OUString::createFromAscii(cChar) - // static awt::Rectangle lcl_GetBounds( Window *pWin ) -- 1.7.0.4 ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Libreoffice] [PATCH] EasyHack RTL_CONSTASCII_USTRINGPARAM in starmath
Hello, This changes the only occurrence in writer/starmath. The wiki entry does not mention it, but should UNOOUSTRING::createFromAscii (not OUString) be replaced as well? There are two of that in starmath Jani >From da30daf2c9cf9d1f19da7b674a8896fc5fa6e453 Mon Sep 17 00:00:00 2001 From: Jani Monoses Date: Sat, 30 Oct 2010 02:12:24 +0300 Subject: [PATCH 2/2] use RTL_CONSTASCII_USTRINGPARAM --- starmath/source/mathmlexport.cxx |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/starmath/source/mathmlexport.cxx b/starmath/source/mathmlexport.cxx index 97088bb..c5fdd81 100644 --- a/starmath/source/mathmlexport.cxx +++ b/starmath/source/mathmlexport.cxx @@ -621,7 +621,7 @@ sal_uInt32 SmXMLExport::exportDoc(enum XMLTokenEnum eClass) // make use of a default namespace ResetNamespaceMap();// Math doesn't need namespaces from xmloff, since it now uses default namespaces (because that is common with current MathML usage in the web) -_GetNamespaceMap().Add( OUString::createFromAscii(""), GetXMLToken(XML_N_MATH), XML_NAMESPACE_MATH ); +_GetNamespaceMap().Add( OUString(RTL_CONSTASCII_USTRINGPARAM("")), GetXMLToken(XML_N_MATH), XML_NAMESPACE_MATH ); rList.AddAttribute(GetNamespaceMap().GetAttrNameByKey(XML_NAMESPACE_MATH_IDX), GetNamespaceMap().GetNameByKey( XML_NAMESPACE_MATH_IDX)); -- 1.7.0.4 ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
[Libreoffice] build or rawbuild dir?
This page describes building in the topmost dir http://www.documentfoundation.org/develop/ whereas the wiki-page example changes to rawbuild before. Which one is recommended? I have been building from topmost dir so far. thanks Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: [Libreoffice] Wiki developper page
On 10/12/2010 04:43 AM, Norbert Thiebaud wrote: Ok so I don't have a blog... but We have a brand new wiki... So I started a page, intended to help prospective developper to get acquainted with the build and with git... This is quite fresh in my mind... since I've learn that in the past few days. But I may be wrong, so a reviewed by more senior developpers (and anyone else for that matter) is welcome. http://wiki.documentfoundation.org/Developper_Corner Nice! Is it possible/easy to set up OpenID login on this wiki, or do we all make yet another account :) ? Jani ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice