summaryrefslogtreecommitdiffstats
path: root/xmloff
Commit message (Collapse)AuthorAgeFilesLines
* Add hack to optionally get stable ODF output from the same inputTor Lillqvist2013-09-304-29/+235
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To be used in regression testing and similar scenarios, where the output ODF is *not* intended to be further manipulated in LibreOffice. An environment variable LIBO_ONEWAY_STABLE_ODF_EXPORT is used to toggle this behaviour. I am not 100% sure whether the generated ODF with the hack toggled on is even fully correct, but correctness is not the purpose of the hack anyway. Two classes of issues handled: 1) Automatic style names and 2) use of randomness. For class 1), when the hack toggle is in effect, we generate the names at first as strings based on all the properties of the style, and sort them based on those, and then rename them (for brevity in the output) to the "normal" form of a short prefix plus a number (like "P12"). Sure, it would have been better to just figure out *why* the automatic style naming currently is not stable in the first place, but outputs the styles in different order (with some styles being assigned different numbers) in separate invokations of LibreOffice), but I was unable to understand that. Possibly this code could be used in all cases, except that it does break some unit test (can't recall which right now). I don't know whether that is simply because the unit test assumes too much knowledge of the internal workings of the automatic style name generation, or whether the generated ODF is actually invalid. For 2), I found a handful of places where randomness was used to generated various kinds of identifiers in ODF output. I changed those to just use large (64-bit) non-overlapping integers instead. I assume there *is* a point in the original code in each case that explains why randomness is needed, so the hack definitely needs to be optional and used only for the above mentioned scenarios. Change-Id: I17b657197e38bcf24abdfe61ad4a277f4339eeae
* Mark as const / staticTakeshi Abe2013-09-297-12/+12
| | | | Change-Id: Ib88b9e5533e56bb8b0b05c4e668299b033f17ffb
* typo fixes in commentsAndras Timar2013-09-266-8/+8
| | | | Change-Id: Iaadec33715f8e0e0c6595c5e684606905274fdab
* sal_Bool to boolTakeshi Abe2013-09-253-13/+13
| | | | Change-Id: I390b6b0d8e1c143a0d2e8f4f0a1ec80c2924942c
* Mark as const / staticTakeshi Abe2013-09-256-64/+64
| | | | Change-Id: I4a70ed8b527096e35fa717b87efc4b6990f29121
* pXML_DrawAspect_Enum is no longer usedTakeshi Abe2013-09-251-13/+0
| | | | | | since bc1b9d2dc0428f6a23f9284ea4a058ee55f6c3e4 Change-Id: I7e6bccc451f086093086578775e1446afe0babb6
* Try to fix cross-compilationTor Lillqvist2013-09-231-1/+1
| | | | | | | | Add more FOO_FOR_BUILD variables and some gb_Foo_for_build functions. Get rid of gb_INSTROOT and gb_DEVINSTALLROOT, just use INSTROOT. Change-Id: Iee531b02d14fae41edb68ad589a5dec829a60255
* Still WaE: unsafe mix of type 'sal_Bool' and type 'const bool'Tor Lillqvist2013-09-181-1/+1
| | | | Change-Id: I16cc5dc16f16d7718dcbda8cc279ae98a89cb751
* WaE: unsafe mix of type 'sal_Bool' and type 'const bool'Tor Lillqvist2013-09-181-1/+1
| | | | Change-Id: I220868988af814c80c47fad9f8d43c30601d397b
* Resolves: fdo#68431 style:font-name unknown in non autostyles in impress/drawCaolán McNamara2013-09-1710-23/+42
| | | | | | | | | Change-Id: Ib904af4fac5659821ba3b5dbbf62370df8243001 Reviewed-on: https://gerrit.libreoffice.org/5810 Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric@lanedo.com> Tested-by: Pierre-Eric Pelloux-Prayer <pierre-eric@lanedo.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* If it is a set, don't call it a list, that just confuses the readerTor Lillqvist2013-09-172-43/+43
| | | | Change-Id: I1d80c2ee463a0253e7e3a84331106006316c1ea6
* sal_Bool to boolTakeshi Abe2013-09-178-55/+55
| | | | Change-Id: I81767fda35b52e73bc78c07b660244a6808a0409
* Insert impastp4.cxx into impastpl.cxxTor Lillqvist2013-09-163-406/+380
| | | | Change-Id: Iccd059e2d09666775f8391a1ac2a074518a5c111
* Insert impastp3.cxx into impastpl.cxxTor Lillqvist2013-09-163-145/+121
| | | | Change-Id: I8038a724d79a81ac5502f926f8b8f19e2a277e74
* Insert impastp2.cxx into impastpl.cxxTor Lillqvist2013-09-163-51/+29
| | | | | | | I guess "impastp" could stand for "IMPort AutoSTyle Pool" but no idea what the "l" means, I chose the name "impastpl.cxx" just to match "impastpl.hxx" Change-Id: I6e03df47f4782bfe5bd959d9b52f1fb666a3086f
* Start of collecting the impastp*.cxx files into oneTor Lillqvist2013-09-162-1/+1
| | | | Change-Id: Ibb4e2475efc7f70c08035184cddedfaf40b7ec25
* sal_Bool to boolEike Rathke2013-09-1685-887/+886
| | | | Change-Id: Id32780e5cb565e85f0366a91282ce085ed7d2f3c
* CharBrd 10: Some documentationZolnai Tamás2013-09-152-0/+11
| | | | Change-Id: I4c6c2cd4acf8ae4a759a662f92066ad63df74b3c
* conditional SAL_WARNEike Rathke2013-09-131-1/+4
| | | | | | | | An attempt to import a language from fo:language is fine even if *:rfc-language-tag was already imported, just ignore this case instead of warning, warn only if Language!="qlt". Change-Id: I0f7ebc6964fe079c9f86ca19e98cfa56ec56398a
* Bin unused MAX_CACHE_SIZETor Lillqvist2013-09-131-2/+0
| | | | Change-Id: I00e04ba603da8b07d53a9e23134fa01ea1ab3f0b
* Resolves: fdo#34987 skip autoheight reset if it will be set to the same valueCaolán McNamara2013-09-121-2/+43
| | | | | | | | | | | | | | | | | Triggered by aa9af08b389a106fcfb53842ac7669b208a27205 which explicitly sets rSet.Put( SdrTextAutoGrowHeightItem(FALSE) ); so there is something set on the style which is being overwritten. The code here resets the style to the default of "true" before going on to set it to the explicit "false" again. In that window of time the master shapes listen to the property change, on being set to autoheight they resize and on being unset, they remain stuck on their autoheight calculated size. Change-Id: I567a791b2bbbcb3a1a111633fabf509142984645 Reviewed-on: https://gerrit.libreoffice.org/5887 Reviewed-by: mhofmann <borim7@web.de> Reviewed-by: Thorsten Behrens <thb@documentfoundation.org> Tested-by: Thorsten Behrens <thb@documentfoundation.org>
* xmloff: fix Apple g++ 4.2.1 warningMichael Stahl2013-09-121-1/+2
| | | | Change-Id: I7ded25661e7d3343afc783e804ab5f3cf0207641
* sal_Bool to boolTakeshi Abe2013-09-123-25/+25
| | | | Change-Id: I826911bdf06b8047ca8e3bd256687e10fb51962d
* Mark as constTakeshi Abe2013-09-1210-19/+19
| | | | Change-Id: I1d62c66bd89ec5a813eb7b211042e392903c8baf
* Towards a working instdir for Mac OS XStephan Bergmann2013-09-111-1/+1
| | | | | | | | | | | | | | | | | Introduced gb_INSTROOT, which is the same as $(INSTDIR) except for Mac OS X, where it is $(INSTDIR)/LibreOffice.app/Contents. Most stuff ends up there (so most occurrences of $(INSTDIR) have been replaced with $(gb_INSTROOT)), but SDK- related stuff goes to $(INSTDIR)/$(gb_Package_SDKDIRNAME). (And GeneratedPackage needed to be made more flexible, to allow for packages that go into either of those two places.) For Android and iOS, gb_INSTROOT probably still needs to be set. The most obvious missing thing yet to make instdir work for Mac OS X is the instdir/*/LibreOffice.app/Contents/ure/ vs. instdir/*/LibreOffice.app/Contents/ure-link/ split. Change-Id: I4478edd27b14c92c96d92d5169bdca3ec50d78f5
* sal_Bool to boolTakeshi Abe2013-09-107-51/+51
| | | | Change-Id: If6d39259c29e9abcb693c9b0bdfb89b94001f928
* Mark as const / staticTakeshi Abe2013-09-1012-27/+27
| | | | Change-Id: I9bcf02b270200fd146d2bfee6f15ff1f9ad8888b
* update pchThomas Arnhold2013-09-051-1/+2
| | | | Change-Id: I6d5e8d3fd69189935a3338fee19b60022f70a7eb
* fdo#68839: ODF import: fix name of multi-image frameMichael Stahl2013-09-033-6/+33
| | | | | | | | | | | | | | When multiple image child elements are inside a frame, each one is imported and gets unique name via SwDoc::SetFlyName(). But the retained one is not necessarily the first one, which is the only one that may have the original name. Also the solveMultipleImages needs to return a smart pointer, as nothing else keeps the image contexts alive. (regression from 44cfc7cb6533d827fd2d6e586d92c61d7d7f7a70) Change-Id: I28a8a752f3eed176cc2ebb4c9af11a0dd4d18ea6
* fix indentationPhilipp Riemer2013-08-311-1/+1
| | | | | | | | This is a follow up commit to - 22d1beb78a475e4846af945afde1c4d6c263b5d6 - 1c7af455ab9345304a7ac48ce2e0310de2ac8a75 Change-Id: I55ff666c357c89ad355a1a5bc0d0347fcc188476
* fdo#62475 - remove visual noisePhilipp Riemer2013-08-311-142/+2
| | | | | | | | This is a follow up commit to - 22d1beb78a475e4846af945afde1c4d6c263b5d6 - 1c7af455ab9345304a7ac48ce2e0310de2ac8a75 Change-Id: I102685391125f3b4f7bdf838f8bd17a2283d558d
* cppcheck: fix redundant assignmentJulien Nabet2013-08-311-4/+1
| | | | Change-Id: If9768751a50f302b8ad036f345ddcbcc29366b7c
* -Werror=unused-but-set-variable second tryJulien Nabet2013-08-301-8/+1
| | | | Change-Id: Ia8a9f60831d089ec04127213bb9ded31aa754bf7
* Revert "-Werror=unused-but-set-variable bCategoriesApplied"Julien Nabet2013-08-301-0/+3
| | | | This reverts commit 72e019c4e301ef93bd1be985edd037d1e2b10e28.
* -Werror=unused-but-set-variable bCategoriesAppliedJulien Nabet2013-08-301-3/+0
| | | | Change-Id: I43c1f866847e2029bc1547dff3be3c1ed454d281
* fdo#68663 don't blindly force categories when there are noneLionel Elie Mamane2013-08-301-10/+0
| | | | Change-Id: If52ea471ee3ca10e2f69d2fcd7b3ae0d5772bfc3
* ODF filter: handle draw:shadow-opacity for Writer framesMiklos Vajna2013-08-241-0/+1
| | | | Change-Id: I3ad39e5de5b6999d6c2182c1ec622ae7f873cafe
* ODF export/import of fine dashed border styleZolnai Tamás2013-08-232-0/+6
| | | | Change-Id: Ia81385dd9952e90cd5ebb3d1bba38aff6f84d079
* CharBrd 7: Border shadowZolnai Tamás2013-08-231-0/+4
| | | | | | | | | | | | | | | | | | | | | 1, New character attribute - Add the three shadow members to font (width, color, location) 2, Painting - Extract the shadow painting to a local function and use that for character shadow (paintfrm.cxx) - Reduce the painting rectangle of background (shadow doesn't inherites the background of content) 3, Border merge - Use text portion join members to skip shadow when neccessary. 4, Vertical text - Get shadow space dependant from oritentation and border merge. 5, UNO API and ODF filter Paragraph: ParaShadowFormat -> Character: CharShadowFormat Change-Id: Ief4669990d238016c3ffc18a2609a3df7d9dddbf
* fdo#60740: Populate auto styles from edit cells without using UNO API.Kohei Yoshida2013-08-211-1/+1
| | | | | | Not using UNO API here alone made this code fly. Change-Id: Ib3eff168bbe49e49fb413909b7a14057f808cdf9
* finish deprecation of O(U)String::valueOf()Luboš Luňák2013-08-2133-80/+80
| | | | | | | Compiler plugin to replace with matching number(), boolean() or OUString ctor, ran it, few manual tweaks, mark as really deprecated. Change-Id: I4a79bdbcf4c460d21e73b635d2bd3725c22876b2
* Re-work 8bit characters in source code, or remove them.Michael Meeks2013-08-201-1/+1
| | | | | | | | Change-Id: I93e14d4936c0ffbe03425d4a54bb0e09bc62b3e3 Reviewed-on: https://gerrit.libreoffice.org/5550 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@suse.com> Tested-by: Michael Meeks <michael.meeks@suse.com>
* Resolves: fdo#67665 font names filtered out on odp/odg exportCaolán McNamara2013-08-201-18/+41
| | | | Change-Id: Ia529992d15152db981379a1e0a4dec63b2dad40c
* These can be file local.Kohei Yoshida2013-08-191-20/+25
| | | | Change-Id: I71193951ce7a1d1a63fc6931a12267f22cdb72c3
* fdo#57950: Remove some chained appends in xmloffJelle van der Waa2013-08-1912-67/+21
| | | | | | | Change-Id: Iff0c43d190e6afa26c23416db4da9843cf7f41a5 Reviewed-on: https://gerrit.libreoffice.org/5327 Reviewed-by: Petr Mladek <pmladek@suse.cz> Tested-by: Petr Mladek <pmladek@suse.cz>
* Further work on the "Mac-like app structure" optionTor Lillqvist2013-08-181-1/+1
| | | | | | | Lots of stuff still either ended up in the wrong place, or was looked up from the wrong place, or both. Fix most cases. Change-Id: I06ebbce207c219f3cd82b4387dd9b3fdb83420d4
* fdo#43807: Fix import of drop caps character styleZolnai Tamás2013-08-181-1/+1
| | | | | | Export works and import has just a little typo. Change-Id: I570d70423b5a626f21117971fb7eff030eae20af
* CharBrd 8.1: ODF filtersZolnai Tamás2013-08-183-167/+350
| | | | Change-Id: Ib1a79678ffce7764638378b4002f5e87ae749d00
* No need for elaborate type name for this.Kohei Yoshida2013-08-175-20/+9
| | | | Change-Id: Ia29a5956813c5d71f042f774d00de3bc82b7f1aa
* Move static struct to where it's used, and use std::vector for this.Kohei Yoshida2013-08-172-14/+12
| | | | Change-Id: Id68fa488731d3b34213db23bf4d50f73b04c6fe4