summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2016-01-13loplugin:unusedmethods unused return value in include/sotNoel Grandin
Change-Id: I8da6d5f60b6c65ff0da8659045da1cd7666fb1d9
2016-01-13loplugin:unusedmethods unused return value in include/sfx2Noel Grandin
Change-Id: Id4017bdfe18f8224d28a9f787865c7a880aeeed8
2016-01-13Bin unused fieldTor Lillqvist
Change-Id: I664e3815bcdaff82a03309ae387154914471a80b
2016-01-13vcl: Create accessor and mutator for full stop centered in FontMetricChris Sherlock
Accessor and mutator created for full stop centered flag, removed bit field. See commit description in 8bfccd3a71d911b6d ("vcl: Create accessor and mutator for font scaling in FontMetric") for reasoning behind patch. Unit tests - check to ensure that can set full stop centered flag - check equality operator on FontMetric after setting full stop centered flag Change-Id: I9cacb0fbf9ea65cfebcaebdc9f0481c0a796cbcf Reviewed-on: https://gerrit.libreoffice.org/21413 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-01-13vcl: format enum in ImplFontMetric to make it easier to rebaseChris Sherlock
Change-Id: Ic2e7213254b60cf3c384dd7ab4696092dbbc5108 Reviewed-on: https://gerrit.libreoffice.org/21412 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-01-13update emoji autocorrect files from po-filesChristian Lohmaier
Change-Id: I1844e4fc7fa08ee5526e7b9ff76224572cd4d089
2016-01-13update creditsChristian Lohmaier
Change-Id: I7d00e1eb3bb6d2da18ea2e6ae531aeb18e27c214
2016-01-13vcl: Create accessor and mutator for font scaling in FontMetricChris Sherlock
This is fragile code! There are actually *two* classes that do almost precisely the same thing, they are: - ImplFontMetric, and - ImplFontMetricData They both have much in common, including their class name, and even most of their functionality. In fact, they both have common accessor functions. When I look at the code, it looks like OutputDevice is actually given an ImplFontMetricData object, which it then uses to populate an ImplFontMetric object... Basically, I'm going to merge these classes. To do so, I'm going to do the following: Step 1: Implement accessor functions for ImplFontMetric and FontMetric (then remove the friendship of this class to OutputDevice!) Step 2: Write a unit test for each accessor function in ImplFontMetric Step 3: Ensure that ImplFontMetric and ImplFontMetricData use some sort of smart pointer (probably an intrusive_ptr like I did ages ago with FontCharMap) Step 4: Merge the two classes together once their class interfaces are the same and I am satisfied they do the same thing Step 5: Find all instances of inefficient usage - for instance, I can do away with the code that copies the ImplFontMetricData attributes into an ImplFontMetric object. Change-Id: I07c1cb848774b130fa2ca60b51da53e07754dd00 Reviewed-on: https://gerrit.libreoffice.org/21399 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-01-13Updated coreChristian Lohmaier
Project: translations 3b2b4114cc4585399aa5db74ce702b0382bc01d8 update translations for 5.1.0 rc2 and force-fix errors using pocheck Change-Id: I24437fdaa13a74a85f58314771245375fec94108
2016-01-12Fixup SAL_DEBUG_TRACE macro.Michael Meeks
Change-Id: I276ce9a356086001c208902ba94641863c2f2fdb Reviewed-on: https://gerrit.libreoffice.org/21408 Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
2016-01-13Modified state icons not packaged into zipMaxim Monastirsky
It seems that images must be mentioned in some src file. Unfortunately had to copy the save icon in place of the modified icon in galaxy, to satisfy the resource compiler. Change-Id: If3fa7a8a194a0289c79fa10987138dfb35eaeca2
2016-01-12xmloff: SvXMLExport::mbExtended is dead nowMichael Stahl
Change-Id: Ie679259deccce87949d7bf8767af707827b4134c
2016-01-12xmloff: there is no reason why office:version should be omittedMichael Stahl
... if a document contains foreign elements or attributes. In fact since ODF 1.2 the office:version attribute is mandatory and any document that omits it is therefore invalid, while "extended conforming" documents are allowed to contain foreign elements and attributes. This mysterious check was there since CVS initial import, but without any justification. Change-Id: Ifeafad2b7af41f06356461adb7ae65dbf7bae11d
2016-01-12tdf#96385 - opengl: dynamically adjust priority of swap buffers.Michael Meeks
Initially we start with a very low priority, so that the lame bits of code that do eg. focus, and cursor rendering before the document is visible do not cause a swap, flash. Then after we've processed a REPAINT priority idle (hopefully our first paint) we adjust the swap priority to highest. Essentially a fusion of Tor's approach and mine. Change-Id: Ib9b78a18fb9359ac8c3dee1cfeb30177c08fe162 Reviewed-on: https://gerrit.libreoffice.org/21405 Reviewed-by: Tor Lillqvist <tml@collabora.com> Tested-by: Tor Lillqvist <tml@collabora.com>
2016-01-12sc lokit: refactor setPartHenry Castro
Change-Id: I01786551aef0274ae11fe673cc5b0c250e8817cc Reviewed-on: https://gerrit.libreoffice.org/21403 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Henry Castro <hcastro@collabora.com>
2016-01-12tdf#97075 - don't render sdr overlays before we paint.Michael Meeks
Change-Id: I0b22af14c7dd684c25cfee26e8c12b95b40479ee Reviewed-on: https://gerrit.libreoffice.org/21400 Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
2016-01-12tdf#96412 Fix export text table cell UserDefinedAttributes to xml.aqcoder
It also export to different Fmt when only have the difference of UserDefinedAttributes for the text table cell. Reviewed-on: https://gerrit.libreoffice.org/20765 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Let's try to push it again, it's unclear what the tinderbox failures were that caused it to be reverted. Change-Id: I022b627926870b883eea363567a34b61c34ab353
2016-01-12connectivity: remove MSVC 2012 specific hackMichael Stahl
Change-Id: I78a25118d68ab7ad1db5240910f0974ba1e8381f
2016-01-12CppunitTest_xmlsecurity_signing: enable on OS XMiklos Vajna
Reviewed on: https://gerrit.libreoffice.org/21395 Change-Id: I4f96b322b0306a10d7d47d3b71db9b0716575fe4
2016-01-12Revert "Re-work focus handling to use an idle handler"Miklos Vajna
This reverts commit 0ba752201dca6129cc6b4a19ec93eb369eb70890. It breaks CppunitTest_desktop_lib on Linux.
2016-01-12Fix FastSaxSerializer::write() for non-BMP unicode characters.Mark Hung
Change-Id: I6ee9d028813c970b9ac5b9f0574af932e73111bd Reviewed-on: https://gerrit.libreoffice.org/21293 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-01-12RTF import, fix landscape flag & implement lndscpsxnGuillaume Smaha
Change-Id: Id2e8e9c1a9686fd674d712498acb7374a81077d7 Reviewed-on: https://gerrit.libreoffice.org/20163 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mark Hung <marklh9@gmail.com> Tested-by: Mark Hung <marklh9@gmail.com> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-01-12tdf#87118 LibreOffice should not claim to use version 1.01 of MathMLTakeshi Abe
Call it MathML 2.0 instead. Change-Id: Ifecd52cfcded680bbddf680f2600a42b1c7568d9 Reviewed-on: https://gerrit.libreoffice.org/21304 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-01-12tdf#65439: Export to PDF/A-1a gives transparency warningJulien Nabet
There was a use of transparency because Watermark was always defined Now we test if text of Watermark is empty or not Remark: it seems we don't cope not-text Watermark in pdf export (additional cleaning+renaming) Change-Id: I9d9c759a80c0a61e4f0a94c0fa295f2950caa3bc Reviewed-on: https://gerrit.libreoffice.org/20990 Tested-by: Jenkins <ci@libreoffice.org> Tested-by: Jan-Marek Glogowski <glogow@fbihome.de> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-01-12Re-work focus handling to use an idle handlerTor Lillqvist
Gets rid of some flashing when using OpenGL. Change-Id: I1273b960725bf868e7340eac3f2bad2f20e73d91 Reviewed-on: https://gerrit.libreoffice.org/21390 Reviewed-by: Tor Lillqvist <tml@collabora.com> Tested-by: Tor Lillqvist <tml@collabora.com>
2016-01-12Related tdf#96174: save again on Google Drive.Giuseppe Castagno
Files can no longer be saved on Google Drive. The bug was introduced in 0398e1e3967332c49b6451b5d41bcf0357052d0b. Google Drive may have user name in the form: name.surname@domain.com In some places the user name should be escaped before using it. Change-Id: Idee3e57c0ad1dcbbd517b60c387868bfc706f27e Reviewed-on: https://gerrit.libreoffice.org/21198 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-01-12Revert "Stick to exact-type check"Stephan Bergmann
This reverts commit 791ad3e170bcb5a2daedac77932dc9822f7f1092, as discussed on IRC: <sberg> os_cib, ^ or was that intentional? <os_cib> sberg: There was no reason for an exact type check. Afterwards the pointer is static_casted to an SdrRectObj anyway. <sberg> os_cib, but how is that "Afterwards..." part relevant for what the if() should actually check? <os_cib> sberg: The detective function puts some object types into a certain drawing layer (polygons, circles and rectangles). It won't find any other object type in that layer.
2016-01-12tdf#39468 translate OSL_ENSURE German textChris Sherlock
Change-Id: I41d66f7b19217d0ba7780c1a999bc1289a6d3234 Reviewed-on: https://gerrit.libreoffice.org/21387 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-01-12starmath: Assert that SmRootNode always has three childrenTakeshi Abe
Change-Id: I9d1b47fc7fc26254f331967dd58f60d88bd2112f Reviewed-on: https://gerrit.libreoffice.org/21036 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-01-12New loplugin:faileddyncastStephan Bergmann
Offline discussion about tdf#96067 "Crash on undo row inserts" brought up the idea to warn about cases where uses of dynamic_cast are statically knwon to always fail. Clang's clang::AST::CXXDynamicCastExpr::isAlwaysNull already implements such a check, reporting true if the casted-from class is final, but has two issues: For one, it does not work for template code, when one of the involved types is a template parameter type (so e.g., DestType->castAs<PointerType>() can crash). For another, it misses the opportunity to report true if the casted-to type is final and only derives from the casted-from type non-publicly. My hope was that this, after the "final" decorations in 548c43238d02b34cf73e7c2ca1a912ee4fe82544 "Mark some classes as final," might turn up the culprit of tdf#96067 (with a scenario similar to the failed dynamic_cast on private derivation in 63b67ab5cab8cf7576a68cabe5fb1a42c6ad800c "Use public derivation, and remove then-unnecessary downcasts")---but not so. Change-Id: I962ee19820758f9c601f4a292da7f37fa9dff5ce
2016-01-12Mark some classes as finalStephan Bergmann
...namely exactly those for which fa91dd31f39a24329d288d4e1cda28db3a16af0d "5th step to remove tools/rtti.hxx" (which is assumed the culprit in tdf#96067 "Crash on undo row inserts") had replaced exact-type checks of the form v->Type() == TYPE(X) with type-or-subtype-of checks of the form dynamic_cast<X *>(v) != nullptr which is only sound if X is a final class. (Except for one case already handled differently in 791ad3e170bcb5a2daedac77932dc9822f7f1092 "Stick to exact-type check.") Change-Id: Ia822c55c7c0e982d42c6a97589307e577721e324
2016-01-12Stick to exact-type checkStephan Bergmann
fa91dd31f39a24329d288d4e1cda28db3a16af0d "5th step to remove tools/rtti.hxx" had changed this from an exact-type check pObject->Type() == TYPE(SdrRectObj) to a type-or-subtype-of check dynamic_cast< const SdrRectObj* >(pObject) != nullptr but SdrRectObject is not a final class (SdrCaptionObj, SdrCircObj, ... derive from it). Change-Id: Ia820ee5475fbaf7a00ab0093d949c3cbd3406f83
2016-01-12Typo: 'It as' -> 'It is'Samuel Mehrbrodt
Change-Id: I6ce5dc2333429fa472e2d605a95385912175910e
2016-01-12sc: try to fix MSVC buildMichael Stahl
Guess we don't actually need the unique_ptr there. Change-Id: I9b041b084567be6d8d354558da4e6916fcf81639
2016-01-12Updated coreChristian Lohmaier
Project: help 8ecafdaa4e5b954941031e384b88c34309b7b27e <ahelp> should not be empty/self-closing element (<ahelp ../>) f5fa46ed8626910d710fd535518ddf9e756f9356 already fixed (tried to) the empty/self-closing ahelp tag, however <ahelp ..></ahelp> is merged to <ahelp .. /> by "make translations", so po files ended up with the self-closing form again. Surrounding a child/PCDATA with the tag solves the problem. Change-Id: I58725f6a7e5999609262abec2987a6d05445a7b2
2016-01-12remove nonsensical U2S void macroEike Rathke
help IDs are strings since years.. Change-Id: Ie3a939f43861b6a11a146db6df713dcf73098454
2016-01-12xmlsecurity: handle no META-INF in DocumentSignatureHelperMiklos Vajna
Change-Id: I7e0ac2d43e97fe9cb67a58c8dd8d83c1068f501d
2016-01-12SfxObjectShell::ImplAnalyzeSignature: handle SfxFilterFlags::SUPPORTSSIGNINGMiklos Vajna
Change-Id: I9d45d3a39c8332c01fcc23dcc5da8db7ed4c5cdc
2016-01-12xmlsecurity: gracefully handle lack of META-INF storage streamMiklos Vajna
Change-Id: Ic0b33f7b014ea7cfb16cc1babb69f2d2b6ba8dee
2016-01-12comphelper: initial SfxFilterFlags::SUPPORTSSIGNINGMiklos Vajna
Currently digital signatures can be made only on OWN formats. Add a new flag that can be used to mark filters (only DOCX as a start) which could also support signing and fix the first place that assumes that only OWN formats support signing. Change-Id: If9fbf956b765d75838986762e4d4063a9e8c0b5e
2016-01-12sc: replace boost::ptr_map with std::map<std::unique_ptr>Michael Stahl
Change-Id: I596cafb971d522b0f1414d0ebe0bea36d8f30e59
2016-01-12sc: replace boost::ptr_map with std::map<std::unique_ptr>Michael Stahl
Change-Id: I2fbce996afcd7d1dabd7c540272d8877d034cf14
2016-01-12sc: replace boost::ptr_map with std::map<std::unique_ptr>Michael Stahl
Change-Id: I319c7141749df11f7ec0b03b43f4f86cb743ada5
2016-01-12sc: replace boost::ptr_map with std::mapMichael Stahl
Change-Id: If254d2e6ae02cb1dfc4c967c1a8d0a1132bb6ad1
2016-01-12sc: replace boost::ptr_map with std::map<std::unique_ptr>Michael Stahl
Change-Id: I5571d9e4437a34df7ac567878a774ccd9b4496e4
2016-01-12sc: replace boost::ptr_map with std::map<std::unique_ptr>Michael Stahl
Change-Id: I8301ee6e52c9fc5ae62df662c452e27672612759
2016-01-12typosMichael Stahl
Change-Id: I970429c266cc1a8cf36cfba379b447899b5dbce9
2016-01-12tdf#90606 Crash when delete a large part of a column with shift cells leftArul
skipping splitFormulaCellGroup() and joinFormulaCellAbove() for nEndRow equal to MAXROW since nEndRow + 1 is not valid Change-Id: Iabfcb33bf58d20331841ca386cca4a91fd6598bf Reviewed-on: https://gerrit.libreoffice.org/21389 Reviewed-by: jan iversen <jani@documentfoundation.org> Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
2016-01-12loplugin:unusedmethods unused return value in include/ooxNoel Grandin
Change-Id: I22acb6b94774c5944d8276827c53a419f0803f83
2016-01-12loplugin:unusedmethods unused return value in include/helpcompilerNoel Grandin
Change-Id: Ieebbd896bf1ad66cd1950456ca1600f0dd8cda83