summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2014-10-03loplugin: cstylecastNoel Grandin
Change-Id: Ie478d1fcfd3b64d7a2128887c216b5e0b4501ab9
2014-10-03loplugin: cstylecastNoel Grandin
Change-Id: I011a0eae37d01606d28c00f8ab3839f20bf309ce
2014-10-03loplugin: cstylecastNoel Grandin
Change-Id: I114924d5e0407883b2e77016da3c2e3e55657b21
2014-10-03cid#1194925 Resource leakNoel Grandin
reorganise the code to keep coverity happy Change-Id: I18fc2dfd17f97edf1d17c57caaed4cc9d2fedd13
2014-10-03GLSL files should go into $(LIBO_ETC_FOLDER)/openglTor Lillqvist
(And not $(LIBO_BIN_FOLDER)/opengl. Note that on Linux and Windows, LIBO_ETC_FOLDER and LIBO_BIN_FOLDER are the same, "program".) That's where getShaderFolder() in vcl/source/opengl/OpenGLHelper.cxx looks for them. Also, as they are data and not executable code, to conform to strict OS X code signing and sandboxing requirements, they definitely should not be in $(LIBO_BIN_FOLDER) Change-Id: I902e08f7a1f142697df827d50e0bda74be918eb4
2014-10-03docx redline import test (bnc#821804)Luboš Luňák
Change-Id: I09749037deb44c0cd74b28af226dd4b3e34ee0d1
2014-10-03report ParagraphFormat redline type in UNO tooLuboš Luňák
Change-Id: I0cf7586a154ca73ce64a2328ff8df1858e2290ac
2014-10-03handle scope of w:pPrChange and w:rPrChange properly (bnc#821804)Luboš Luňák
Redlines changing formatting of runs and paragraphs are valid for the entire run/paragraph, not just their existence in the XML. So store them in the matching contexts, which will care of it, instead of the endtrackchange stuff. Change-Id: Ie583e4be14e8df95829852bfbbbe25aa0684f02e
2014-10-03display paragraph format redline also as 'formats' rather than empty stringLuboš Luňák
Change-Id: I5ae5ce80fdb453fdf5de54c691de4cd0b88ad7f6
2014-10-03proper scoping for docx redline import (bnc#821804)Luboš Luňák
propagateCharacterPropertiesAsSet sends the properties only when ending a text run (or maybe starting another one, I'm not quite sure), so it breaks ordering by sending them later then expected (although it worked in many cases). It's a question if propagateCharacterPropertiesAsSet is to be used by anything actually, since it seems rather broken to use it in the ooxml frontend. Using sendPropertiesWithId sends the properties properly at the right time, as one would expect. I don't know why dmapper can't simply handle this on its own, as I think it does handle entering and leaving other elements, but spending more time on it with this overdesigned abomination, oh well. Change-Id: Ie36c5f933ea3e6d789ea8f9e4ee3b60a5d1c920c
2014-10-03fix buildCaolán McNamara
Change-Id: I997dfa3b1fe651db56ad2a6a24ec0cb6d5044550
2014-10-03CppunitTest_sw_mailmerge: enable on OS XMiklos Vajna
Reviewed on: https://gerrit.libreoffice.org/11773 Change-Id: I5dc63abea660f097a63f7e61fc2b3b4cd48fa949
2014-10-03fix clang 3.5 buildNoel Grandin
Change-Id: I86fe850b4b7bc8fc4c44717ca813772114ca70ff
2014-10-03WaE: private fields 'mnTab' and 'mnCol' are not usedTor Lillqvist
Change-Id: I449b85b00aeaf690644e17be0a6b2b9398a3cd3f
2014-10-03coverity#1241327 Dereference after null checkCaolán McNamara
Change-Id: I214884e9ee1d49dcc4db71f8744513468b912691
2014-10-03coverity#982268 Copy-paste errorCaolán McNamara
Change-Id: I68e8e6a564356bae620647b385e16279ff212d32
2014-10-03coverity#1078526 Logically dead codeCaolán McNamara
Change-Id: I203511fcb1e740d7d371821d49a7c13a72e6fc3b
2014-10-03coverity#1093225 Dereference after null checkCaolán McNamara
Change-Id: I34a7b79b4a7b5d86c3d39fd995ed3bd998eeeab8
2014-10-03coverity#1241080 Dereference after null checkCaolán McNamara
Change-Id: I82113fcd4182f9557ba0e7258498b5d4e798bf03
2014-10-03coverity#704994 Dereference after null checkCaolán McNamara
Change-Id: I19f98152b5bd8c9b2cdfacb560d13fc0c1f8c4be
2014-10-03coverity#704318 Logically dead codeCaolán McNamara
Change-Id: I591188e0323bb75c55cad9a078eea188f79b3a03
2014-10-03coverity#736943 try swapping arguments around as an experimentCaolán McNamara
Change-Id: I89f0cfc9049503318955d58dde3f9a3e1197c45d
2014-10-03coverity#1242449 Identical code for different branchesCaolán McNamara
Change-Id: I0cffc9df6d971393ae874060a8c306e12966370c
2014-10-03coverity#1242521 silence Arguments in wrong orderCaolán McNamara
Change-Id: I30b61fa17a941b785a4f148e2ce6c912dbd3c645
2014-10-03coverity#1242529 silence Arguments in wrong orderCaolán McNamara
Change-Id: I662c89e6e3e0917b51e8084bc50a9ea38d6c0838
2014-10-03coverity#1242530 silence Arguments in wrong orderCaolán McNamara
Change-Id: I809e771e4f052ad64300b09174a1bdbc769e15d3
2014-10-03coverity#1242533 silence Arguments in wrong orderCaolán McNamara
Change-Id: I7031e7408f5bcd9ae85fc50952253e14d2fb2edf
2014-10-03coverity#1242532 Arguments in wrong orderCaolán McNamara
I don't think this actually matters Change-Id: Ia7190f436eeb47888b1aace1391cf0b5aef9f9f0
2014-10-03Speed up SwAttrIter::GetNextAttr()Matthew J. Francis
The inner loop which iterates over the characters of m_pTxtNode->GetTxt() is already bounded to the length of the string, so there's no need to pay the price of checking its length for each array position Change-Id: I7674ea2b46db75fea30dd016b96ec932068fd73b Reviewed-on: https://gerrit.libreoffice.org/11784 Reviewed-by: David Tardon <dtardon@redhat.com> Tested-by: David Tardon <dtardon@redhat.com>
2014-10-03Fix unbalanced new/free() -> new/deleteMatthew J. Francis
(Spotted while valgrinding an unrelated issue) Change-Id: I2eab4c08e251d79f427fd01442c4dce20d7d89f0 Reviewed-on: https://gerrit.libreoffice.org/11785 Reviewed-by: David Tardon <dtardon@redhat.com> Tested-by: David Tardon <dtardon@redhat.com>
2014-10-03WaE: -WshadowTor Lillqvist
Change-Id: I3a9e01237d8cade5d99955f02c01c25618ccd9de
2014-10-03Use constants for ST_SectionMark valuesMiklos Vajna
Change-Id: I3ac385b8f21409b5083b1224652283fec8bb2fa4
2014-10-03fdo#84617: Guard the destructor with SolarMutex.Jan Holesovsky
The same pattern as introduced elsewhere recently. Change-Id: Ie7b4fd1ec26d417f63bee26dd336dfc54f4847dc
2014-10-03fdo#82144 fix unpacking of additional tarballsDavid Tardon
Change-Id: I74a61005fa9534b3f96cd392c6e38c54c5fd35e1
2014-10-03rename SvRef::AddRef to AddFirstRefNoel Grandin
to make it's intended purpose clearly distinguishable from AddNextRef Change-Id: I5da780b48b19fd873667b648031bc394113f953b Reviewed-on: https://gerrit.libreoffice.org/11763 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-03scripting: organize importsRobert Antoni Buj i Gelonch
Change-Id: I26a1164d44f464a44ed7deb93ac39465feb0541c Reviewed-on: https://gerrit.libreoffice.org/11768 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-03styles dropdown: Show only the triangle for the dropdown.Jan Holesovsky
The entire button appears on mouse over. Change-Id: I27e713f75a30ec4a2b271d813fcbcbe8c91a4180
2014-10-02Another test case for this feature.Kohei Yoshida
Change-Id: Ia406baec2867605bf515dac0ba090500e2bbd1ce
2014-10-02Write the first test for this formula-to-value feature.Kohei Yoshida
Change-Id: I545328a1c126b3bfeee8c1f662c5eb9a471d33ca
2014-10-02New feature to allow converting formula cells to static values.Kohei Yoshida
You used to have to convert formula cells to static values by copying them and pasting them as values onto the original place. Why not just allow converting them in place? This is something I've always wanted to implement.
2014-10-02Update mdds to 0.11.1.Kohei Yoshida
Change-Id: Ie1c80029a762fcf92dc20ffe74260ea54c61da28
2014-10-03gbuild-to-ide: Add support for vs2013David Ostrovsky
Change-Id: I385e756109ab0a47feeeb4407ba4a2a4b68548dd Reviewed-on: https://gerrit.libreoffice.org/11728 Reviewed-by: Peter Foley <pefoley2@verizon.net> Tested-by: Peter Foley <pefoley2@verizon.net>
2014-10-03fdo#84592 Fix wrong background color in listsSamuel Mehrbrodt
Change-Id: Ie154bfb15a19ae98b2bc4eb6626240a00cf182d4
2014-10-02scripting: The if statement is redundantRobert Antoni Buj i Gelonch
Change-Id: Ia6186be4ed48fd75d1063eed233a88e09b675bf1 Reviewed-on: https://gerrit.libreoffice.org/11774 Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com> Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2014-10-02writerfilter: that fallback appears necessary after all...Michael Stahl
Change-Id: Id4bc638c81d695745fc44bde16252e3040f0a03c
2014-10-02Scope reduction by early bail-out.Kohei Yoshida
Change-Id: Iacbf9f46a1b5683b6de5dd93002a69078af46951
2014-10-02Make the coding style consistent with the rest of Calc code.Kohei Yoshida
I just ran SlickEdit's 'beautify' command with some follow-up manual editing. Change-Id: I402e782e7d147c4c95ebaa8ffcc40b50a0c565a7
2014-10-02Bin EE_CNTRL_URLSFXEXECUTE which was only set, never testedTor Lillqvist
Change-Id: I18a9355bb902b19f0eeecb26d33eba4fb25db8cb
2014-10-02Bin commented-out #definesTor Lillqvist
Change-Id: I01f473658708a75c3eac6219efd38cd8ec5aa44f
2014-10-02bnc#657899: Paragraph vertical spacing differs from PowerPointTor Lillqvist
We need to set the "paragraph summation" flag also for .pptx import. (As is already done for .ppt import.) This is what in the Options (Preferences) UI is called "Add spacing between paragraphs and tables (in current document)", on the LibreOffice Impress:General page. This is a setting that when set becomes local to the document. I actually don't really understand why this needs to be a user-visible option, wouldn't it be enough for it to be an internal flag that is set for documents imported from .ppt and .pptx, and is clear otherwise? Change-Id: I525178957276e7d587b6ac4be8ae7919fc245a27