summaryrefslogtreecommitdiffstats
path: root/oox
Commit message (Collapse)AuthorAgeFilesLines
* Passing shaperect to addShape function is uselessTamás Zolnai2017-03-114-8/+6
| | | | | | | | | | | | Shaperect is ignored by createAndInsert() so it has no effect at all. Also when this parameter is set the size and position is set using the same shaperect, so don't need to pass it also as a parameter. Change-Id: I35a7953f3eda1bdd19af6f3e77c55b5c7e1f7161 Reviewed-on: https://gerrit.libreoffice.org/35073 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
* Use the converted shape rectanle hereTamás Zolnai2017-03-111-2/+2
| | | | Change-Id: I38092b9ca0de2c73a639588d441eff56eb88e33e
* tdf#106217: Wrong size and position of a shape inside chartTamás Zolnai2017-03-111-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | Same code which was added to the method DrawingFragment::onEndElement() in d178d7bef193565a7d6aacbc37a58dfc4bd7b316 A shape's size and position inside a chart object are primarily defined by <cdr:from> and <cdr:to> tags in drawing.xml, but in the same file we also have an <a:xfrm> tag which is used for the same thing (defining size and postion) in general. Testing with MSO it seems it ignores what values are set to <a:xfrm> and uses only the <cdr:from> and <cdr:to> values. The test document showing this problem must be created an earlier version of MSO, becuase <a:xfrm> is set to some random values. Editing the chart and saving it with MSO 2013 the <a:xfrm> values are set consistently with <cdr:from> and <cdr:to> pair. Change-Id: Icbaff78e5a6fcea36678c3b3755c97f57976b626 Reviewed-on: https://gerrit.libreoffice.org/35069 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
* loplugin:loopvartoosmallStephan Bergmann2017-03-072-17/+16
| | | | Change-Id: Id9ac335bcb365afdae66ec7836d6b3bfb6de8577
* tdf#106304: Don't unnecessarily use bogus default char size for a:endParaRPrTor Lillqvist2017-03-042-13/+33
| | | | | | | | Instead use the size last used for an a:rPr below the same WriteText() call. I am not exactly sure about how this hangs together, but this has the desired effect on the exported .pptx. Change-Id: Ie03dcd0b31c15cff8488b22d7423c9f9ad1e2d68
* tdf#106304: Output <a:br> instead of &#10; for hard newlinesTor Lillqvist2017-03-041-17/+25
| | | | | | | | | | This fixes the problem that the line before the hard newline in the .pptx export is not centred. The fix for the sd_export_ooxml1 test is a bit hacky, but then so is the code in question. Change-Id: I4f89e9d0177ab3b16e8ec624a99fce9e2f8ef730
* Remove redundant 'inline' keywordStephan Bergmann2017-03-039-16/+16
| | | | | | | | | | ...from function definitions occurring within class definitions. Done with a rewriting Clang plugin (to be pushed later). Change-Id: I9c6f2818a57ccdb361548895a7743107cbacdff8 Reviewed-on: https://gerrit.libreoffice.org/34874 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* drop SvxNumType and rename SvxExtNumType->SvxNumTypeNoel Grandin2017-03-031-3/+3
| | | | | | | | | | | | | | Both of them are abstractions over the css::style::NumberingType constants. SvxNumType serves no useful purpose, since it's just a typedef to sal_Int16. At least SvxExtNumType wraps up the css::style::NumberingType constants in a nice enum. Change-Id: I7b3f97cd42b0aa23aff7a4db4930da86149d4f4e Reviewed-on: https://gerrit.libreoffice.org/34816 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use SvxExtNumType in SvxNumberTypeNoel Grandin2017-03-031-4/+4
| | | | | | | | | and follow the rabbit trail, fixing various other types in the process Change-Id: I8fc7df3b585df21578ffb28fdd7e5e427b3bfd9d Reviewed-on: https://gerrit.libreoffice.org/34814 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Align columns in output harderTor Lillqvist2017-03-021-5/+5
| | | | | | | | If we are going to use printf anyway, why not use its field width and left-adjustment features and actually achieve the columnar output we presumably wanted? Change-Id: I95dbd2aaa01d06c69d23d74fa249fd3419afa8d6
* Fix typosAndrea Gelmini2017-03-021-5/+5
| | | | | | | Change-Id: Ic6c41fbcc36c11a7528cde0986593a39c2d6738b Reviewed-on: https://gerrit.libreoffice.org/34803 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Remove clearly outdated stuffTor Lillqvist2017-03-011-6/+0
| | | | Change-Id: I23ca8cc512a0bd433474c12d06461597d0bd35ee
* cid#1401592: bFlipX and bFlipY won't both be trueStephan Bergmann2017-03-011-9/+9
| | | | Change-Id: If5ab5d340166b5eedb97773a9907eee1b188027e
* Fix incorrect function name in SAL_INFOTor Lillqvist2017-02-281-1/+1
| | | | Change-Id: I75ee5f1a20fadef7c20103278bc03abd41b815cf
* new loplugin unoanyNoel Grandin2017-02-2827-154/+157
| | | | | | | Change-Id: I5d6c4a67cb2a09e7cd5bd620c6b262d188701b89 Reviewed-on: https://gerrit.libreoffice.org/34714 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Rename misleadingly named include file fillpropertiesgroupcontext.hxxTor Lillqvist2017-02-2719-20/+20
| | | | | | | | | There is nothing called FillPropertiesGroupContext. Rename the file to "misccontexts.hxx" as I could not come up with some more specific name that would describe the collection of classes defined in it. Rename the corresponding cxx file, too. Change-Id: I44db1db8c63c8e897774597b46a237a214aacedf
* There is nothing called FillPropertiesGroupContext since 2009Tor Lillqvist2017-02-272-2/+0
| | | | | | | So bin two pointless and misleading comments that consist of just that identifier. Change-Id: I14724b21d43283dbda1856ab28c56ae44512c3cc
* Clean up a few SAL_INFOs a bit and add useful note about the ==cscode== etcTor Lillqvist2017-02-273-18/+19
| | | | | | | I dislike OSL_THIS_FUNC, it is too verbose, at least with Clang on Linux. Just use the Class::Name without parameters and namespaces. Change-Id: Ic8722a131463762ab70ff35d64ae7040c67bf595
* Probably better to restrict the sequence sizeStephan Bergmann2017-02-271-1/+2
| | | | | | | | ...to any elements actually written. Was like that ever since the code's introduction in d3ffe3ed3fa1b80c7e54439673029e105940db80 "fdo#67737 : fix for flip not being imported & rendered correctly". Change-Id: I460c1c7c578bd7e53cc2e868da0b7aeb5f19194f
* cppcheck: unreadVariableJulien Nabet2017-02-261-1/+0
| | | | | | | Change-Id: Id9d2fbe9b2d4040cc73c401f5558999604acf259 Reviewed-on: https://gerrit.libreoffice.org/34657 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
* vector::data seems more natural than &vector::frontCaolán McNamara2017-02-243-4/+4
| | | | Change-Id: I0ce48075ad186cf1f9bd3e13fa76269fa9819af1
* Better make the zero-initialization more explicitStephan Bergmann2017-02-231-1/+6
| | | | | | | ...if initializing these members is necessary at all; a bit hard to tell from the code. Change-Id: If15fb11b601ba3b8b5d3fdbe5c3dac01b095fb05
* Bin pointless once used typedefTor Lillqvist2017-02-221-4/+1
| | | | Change-Id: I12f51748dd190f6d50ed7ae738d7eedeb32f0af8
* loplugin:unusedfieldsNoel Grandin2017-02-221-6/+4
| | | | | | | Change-Id: I8a114bcda99f123d230d61b0d0595bfe657cc3c0 Reviewed-on: https://gerrit.libreoffice.org/34513 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:subtlezeroinit: ooxStephan Bergmann2017-02-215-9/+9
| | | | Change-Id: I8a1b6baba6191b20264f32cbf73f5d75bc50235b
* tdf#90407: Export Text AutoFit to .pptxTor Lillqvist2017-02-171-2/+11
| | | | Change-Id: I2120f13fff58d4736ab55071236e0c2f80ba2eb0
* Try harder to fix Android tinderboxTor Lillqvist2017-02-161-11/+11
| | | | Change-Id: I4f3f112d4cd301d3ee1806cd2d26564ac3852723
* no std::lround on androidCaolán McNamara2017-02-151-0/+11
| | | | Change-Id: I910a0a81a925ce4578f3b9b3a51987fbbd91aa13
* tdf#106026: Export paragraph top and bottom margin to DrawingMLTor Lillqvist2017-02-151-0/+28
| | | | Change-Id: I8c60f86a1debe135c9172ff639aa5af93f03b99b
* Fix typosAndrea Gelmini2017-02-131-2/+2
| | | | | | | Change-Id: Iaf6bbe81f519d49b82f624282d786c8cc18d641c Reviewed-on: https://gerrit.libreoffice.org/34199 Reviewed-by: Aleksandar Stefanović <theonewithideas@gmail.com> Tested-by: Aleksandar Stefanović <theonewithideas@gmail.com>
* Related: tdf#105707 PPTX import: decode document name from URLAndras Timar2017-02-131-1/+1
| | | | | | | Change-Id: Ic4825549c6fef1a609a753a10f6224cbcd69b7fc Reviewed-on: https://gerrit.libreoffice.org/34135 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
* Related: tdf#105707 PPTX import: warn on empty SmartArt fallbackMiklos Vajna2017-02-101-1/+4
| | | | | | | | | | | | This second related situation is about when there is drawingML fallback for the SmartArt, but it's ampty shape tree, which is never created with MSO2010+ (those shape trees always have at least one element). This is as bad as the missing drawingML fallback, so warn on it. Change-Id: I539d05154a4d1fcd5871dfc29616f77eb9945454 Reviewed-on: https://gerrit.libreoffice.org/34131 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* Related: tdf#105707 PPTX import: warn on invalid SmartArt fallback relidMiklos Vajna2017-02-101-0/+12
| | | | | | | | | | | We already warn when the SmartArt doesn't have a drawingML fallback, but having a reference to a fallback that is not a valid one (so resolves to an empty fragment path) is also unusable. So warn in that case as well. Change-Id: I03b2325003b4eb38b1347a88899799c6d5c33606 Reviewed-on: https://gerrit.libreoffice.org/34129 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* coverity#736181 Out-of-bounds readCaolán McNamara2017-02-101-5/+4
| | | | | | | | | | | | | | | it should be fine, as we rely on there being at least one password char, this is the same as, commit 2d3ed7ebb592401de214eaeee68c656afbefbe05 Author: Michael Meeks <michael.meeks@collabora.com> Date: Tue Feb 25 22:48:48 2014 +0000 cid#736173 - increase default fill chars buffer size. which is another impl of this Change-Id: I5e1e07af37bea0398153ede300beed67d37d5e12
* oox, sfx2: can use std::move() hereMiklos Vajna2017-02-082-3/+3
| | | | | | | Change-Id: I0add196f79045e8cb7280b2b7d1d8620e4ec669e Reviewed-on: https://gerrit.libreoffice.org/34013 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* remove CREATE_OSTRING macroJochen Nitschke2017-02-071-2/+2
| | | | | | | Change-Id: I7d5e1570918adc3b0cf2063d9735414233011a81 Reviewed-on: https://gerrit.libreoffice.org/34009 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Add missing #includesStephan Bergmann2017-02-064-1/+5
| | | | | | | | | ...and remove some unncessary using directives/declarations, in preparation of removing now-unnecessary #includes from cppumaker-generated files, post e57ca02849c3d87142ff5ff9099a212e72b8139c "Remove dynamic exception specifications". Change-Id: Iaf1f268871e2ee1d1c76cf90f03557527ebc9067
* loplugin:unusedenumconstants in ooxNoel Grandin2017-02-061-4/+4
| | | | | | | | | | Convert FragmentHandler2::MCE_STATE to scoped enum and drop MCE_UNUSED constant Change-Id: Id0b3a81e61d77af5d3837527b008e196835f57cd Reviewed-on: https://gerrit.libreoffice.org/33954 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#31488: Background fill changes its color when saving a PPTX file to ODPTamás Zolnai2017-02-051-1/+4
| | | | | | | | | | | | | | For a gradient fill we need to generate a name because ODP export works with this name. In case of shapes it works because when fill attribute changes some internal name generation is triggered. The same thing doesn't work for slide background so generate this name explicitely in oox code. Change-Id: Ic6ebf37ef3d66a9c274747ca04653363b1fe6d02 Reviewed-on: https://gerrit.libreoffice.org/33937 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
* tdf#105707 PPTX import: warn when SmartArt is missing DrawingML fallbackMiklos Vajna2017-02-033-2/+39
| | | | | | | | | | | | | | By the time DiagramGraphicDataContext::onCreateContext() completes, we know if there will be a DrawingML fallback for the SmartArt (called ExtDrawing in the code) or not. Warn about this case once at the end of the import when in interactive mode. Headless mode silently ignores the warning as expected. Change-Id: I2bfeeedcaa244f08d8a0c208167e79d7bb697e6d Reviewed-on: https://gerrit.libreoffice.org/33866 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* loplugin:stringconstantStephan Bergmann2017-02-021-1/+1
| | | | Change-Id: I14ac842fa7b8a8ec8c6eb7554d136a68fe727ed7
* convert method names in tools::SvRef to be more like our other..Noel Grandin2017-02-021-2/+2
| | | | | | | | | | | reference classes, uno::Reference and rtl::Reference. Specifically rename Is()->is() and Clear()->clear(). Change-Id: Icb7e05e2d09cb9977121508b837ba0961dabb4ae Reviewed-on: https://gerrit.libreoffice.org/33576 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* oox: clean-up crypto classes, use c++11 featuresTomaž Vajngerl2017-01-305-245/+211
| | | | | | | | | | | - remove "using namespace std;" - &vector[0] to vector.data() - use nullptr in OPENSSL Change-Id: Ib4067b0256801f94d448bc8d3faf5a2902d694e5 Reviewed-on: https://gerrit.libreoffice.org/33629 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
* loplugin:stringconstant check for unnecessary OUString constructor..Noel Grandin2017-01-281-1/+1
| | | | | | | | | ..calls when creating exceptions Change-Id: I3bc58a5aa4dc6f0508ecb88b3a843b96b8c7ebfe Reviewed-on: https://gerrit.libreoffice.org/33617 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Remove dynamic exception specificationsStephan Bergmann2017-01-2636-384/+233
| | | | | | | | | | | | | | | | | | | | | | | | | ...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at <https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html> "Dynamic Exception Specifications" for details. Most changes have been done automatically by the rewriting loplugin:dynexcspec (after enabling the rewriting mode, to be committed shortly). The way it only removes exception specs from declarations if it also sees a definition, it identified some dead declarations-w/o-definitions (that have been removed manually) and some cases where a definition appeared in multiple include files (which have also been cleaned up manually). There's also been cases of macro paramters (that were used to abstract over exception specs) that have become unused now (and been removed). Furthermore, some code needed to be cleaned up manually (avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no configurations available that would actually build that code. Missing @throws documentation has not been applied in such manual clean-up. Change-Id: I3408691256c9b0c12bc5332de976743626e13960 Reviewed-on: https://gerrit.libreoffice.org/33574 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin: unnecessary destructor oox..saxNoel Grandin2017-01-254-19/+0
| | | | | | | Change-Id: Ie12bfabc92bce04d702f3e77aa5896366e49245e Reviewed-on: https://gerrit.libreoffice.org/33509 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Move two include files to oox/inc as they are used only in ooxTor Lillqvist2017-01-2435-42/+296
| | | | Change-Id: Ie13614c1977f45aa8086f4db65ca86b7d9212735
* Drop incorrect "artistic" prefix from some token namesTor Lillqvist2017-01-242-18/+18
| | | | | | | | | | | | | Apparently copy/paste mistakes in the 642a252cf1a2f1d08c4bbfcae15527bb82c7664d and 21d4cfe19e2796ebf89c408e292c4473924b2bc4 commits from 2014; most of the picture effect type tokens added in that commit do have names that start with "artistic" but not all. Does not break any unit test. (Does not fix the bug I am investigating either, though.) Change-Id: I2c3bb7243e1feaa3fa949b58d3acf34825d6987b
* use rtl::Reference in FastParserNoel Grandin2017-01-244-11/+8
| | | | | | instead of storing both a raw pointer and a uno::Reference Change-Id: I6b67a6098a3ccdce7e29ee8d601c743897032eaf
* New loplugin:dynexcspec: Add @throws documentation, ooxStephan Bergmann2017-01-198-15/+59
| | | | Change-Id: I83c6995e4f15d6a042e57bac4d71874124402547
generated by cgit (git 2.34.1) at 2024-08-05 01:55:27 +0000