summaryrefslogtreecommitdiff
path: root/sw/source/core/graphic
AgeCommit message (Collapse)AuthorFilesLines
2016-04-22Avoid reserved identifiersStephan Bergmann1-5/+5
Change-Id: I27ff0f4f0eb395d7e0a60dd604758c220a3134c4
2016-04-12clang-tidy performance-unnecessary-value-param in swNoel Grandin1-1/+1
Change-Id: I3e030743b640d5a5f6b3bf4a4e9ba50ccd001287
2016-02-29loplugin:unuseddefaultparam in sw (part3)Noel Grandin1-3/+2
Change-Id: Ibff7ac9718929349ee7daa3febb0f8fe4a9fa4db
2015-12-11tdf#69977: uno::Sequence is expensiveNoel Grandin1-2/+2
when used as a mutable data-structure. Plain std::vector halves the time taken to display the chart dialog Create a class to represent the std::vector we are going to be passing around, and move some of the utility methods into it to make the code prettier. Also create an optimised append(&&) method for the common case of appending small temporaries. Change-Id: I7f5b43fb4a8a84e40e6a52fcb7e9f974091b4485
2015-11-25bin/rename-sw-abbreviations.shlibreoffice-5-1-branch-pointRobinson Tryon1-12/+12
This commit renames the most annoying abbreviations in Writer (and partially in the shared code too). Change-Id: I77e5134f42f25e3786afa36b7a505c7e3237a9e8
2015-11-115th step to remove tools/rtti.hxxOliver Specht1-2/+0
tools/rtti.hxx removed completed the interface of some Sdr.* Items and removed pseudo items Change-Id: I0cdcd01494be35b97a27d5985aa908affa96048a Reviewed-on: https://gerrit.libreoffice.org/19837 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann1-19/+19
Change-Id: I01e11fa956a249974e77dce9deebe79311f098d0
2015-11-06com::sun::star->css in sw/source/coreNoel Grandin1-1/+1
Change-Id: I30016977f1be8fb53dd239367d043de92a9467d9
2015-10-20loplugin:defaultparamsStephan Bergmann1-2/+2
Change-Id: I11ff551955c1ac291ab576f2f18c2dd410427eda
2015-10-06 tdf#94559: 4th step to remove rtti.hxxOliver Specht2-3/+3
replaced use of PTR_CAST, IS_TYPE, ISA in idl, editeng, sc, sd, sw, sfx2, sot, starmath Change-Id: I4a5bba4fdc4829099618c09b690c83f876a3d653 Reviewed-on: https://gerrit.libreoffice.org/19132 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
2015-08-26convert Link to TypedNoel Grandin1-2/+2
and make it statically allocated, no point in dynamically allocating such a small object Change-Id: If476bf5dc1e0d535383d16bc49c8d567776f16cd
2015-08-20loplugin: defaultparamsNoel Grandin1-1/+1
Change-Id: I6019dea21b58ac8ba0dba5c7f6ca837716e19869
2015-07-06improve the returnbyref lopluginNoel Grandin1-9/+9
Change-Id: I1b510a6194282dfa4a9001d473127c5ebc8b44eb Reviewed-on: https://gerrit.libreoffice.org/16731 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-04Fix typosAndrea Gelmini1-1/+1
Change-Id: Id9296115f30858e7fd470a199e59343a96d7deec Reviewed-on: https://gerrit.libreoffice.org/16712 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2015-06-08loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann1-2/+2
Change-Id: I8d27ae80d209159690182ab91e272cf00e9f863d
2015-05-20bin/rename-sw-abbreviations.shlibreoffice-5-0-branch-pointChristian Lohmaier1-20/+20
renames the most annoying abbreviations in Writer (and partially in the shared code too). Change-Id: I9a62759138126c1537cc5c985ba05cf54d6132d9
2015-05-06convert BMP_MIRROR constants to scoped enumNoel Grandin1-8/+8
and fix bug in svx/source/xoutdev/_xoutbmp.cxx,XOutBitmap::MirrorGraphic where it was not correctly translating between different flags types. Change-Id: I78b4965544da2aa54b67ec18307fa54c8ffc3aee
2015-05-05loplugin:staticmethodsNoel Grandin1-1/+1
Change-Id: I90dd921077bbfc57200e398e7959306f26c65cfe
2015-04-23loplugin:staticmethodsNoel Grandin1-7/+6
Change-Id: Ib0c0841e48814d1a6e13bcd4575725d86ffa5e5e
2015-04-02Change INetProtocol enumerators to CamelCaseStephan Bergmann1-1/+1
...to avoid clashes with macros and poor warnings about hiding global FILE Change-Id: I444e8b958f029201b228937e088efabc4b6d3050
2015-04-02Change INetProtocol to scoped enumerationStephan Bergmann1-1/+1
...and fix o3tl::enumarray::operator [] const overload Change-Id: I749b1b9d68686b03a97074253478d9d2d9d32b0b
2015-04-01convert BOX_LINE and BOXINFO_LINE to enum classNoel Grandin1-4/+4
since their usage is intertwined. Also introduce new o3tl utilities enumrange and enumarray to make working with scoped enums a little simpler. Change-Id: I2e1cc65dd7c638e59f17d96dfae504747cad6533
2015-03-28Clean up C-style casts from pointers to voidStephan Bergmann1-1/+1
Change-Id: I2ec58d0bfe13fae4fc2141f9b017b5a72e0ba2c2
2015-03-26const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann1-1/+1
Change-Id: Icb14a036ea9d7636359b6bc5e0af17568c0d54cb
2015-03-24rename GetDepends->HasWriterListener and make it a boolBjoern Michaelsen1-1/+1
Change-Id: Idc090c9e105f24ef91ef123db03db6d6e67ed1d4
2015-03-24convert sfxlink to enum classNoel Grandin1-1/+1
Change-Id: I4466af8d40e7860b20a26c5ccf2265ee40c5a9ab
2015-03-19loplugin:constantfunction: svtoolsNoel Grandin1-25/+0
Change-Id: I42479fb10878a0a1c2071f7b35ae375061781d40
2015-03-18create new 'enum class' SotClipboardFormatId to unify typesNoel Grandin1-2/+2
of which there are several. There are some issues here I am unsure of - the SW and SC and CHART2 modules essentially ignore the enum values and assign their own ids Perhaps I should change them to use the common values and create new enum values where necessary? - the sc/qa/ and sq/qa/ and starmath/qa/ code was doing some dodgy stuff. I translated the code to pass down the stuff numeric values to the underlying code, but perhaps further fixing is necessary? Change-Id: Ic06d723e404481e3f1bca67c43b70321b764d923
2015-03-03Typo: instert(ed)->insert(ed)Julien Nabet1-1/+1
Change-Id: Id4262fa42c9c13722ceb2f91f01876614cbf1ad2
2015-02-22ChgTwipSizeFromPixel flag is uselessZolnai Tamás1-3/+3
It is used only to make the same conversion on a different way. GetGraphicSizeTwip() also returns the size in twips. Change-Id: Ic532341ab37c86a63792c82bf3a5fb2e3d8d0e67
2015-01-19fdo#39440 sw: reduce scope of local variablesMichael Weghorn1-1/+2
This addresses some cppcheck warnings. Change-Id: Iccdd4bebd46f274697597b13746c820aaadc99a4 Reviewed-on: https://gerrit.libreoffice.org/13960 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2014-12-25sal_uInt16 to size_tMatteo Casalin1-1/+1
Change-Id: Ide8913930f9f951039cefecf71609c5a2bcc5e82
2014-12-10Use Any specializations for bool, instead of SetValueMatteo Casalin1-2/+2
Change-Id: I7a5babe494bdb09a71164ea74d7be3f6bf985ff6 Reviewed-on: https://gerrit.libreoffice.org/13215 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2014-12-04-Werror=unused-variableMiklos Vajna1-1/+0
Change-Id: I7445c699bf3fa6419411d4800db1b49f92b77a55
2014-12-04use C++11 iterationBjoern Michaelsen1-3/+3
Change-Id: Iff18fa8d3860437f9d9a14ba52d06b1aa4d38076
2014-11-19loplugin: cstylecastNoel Grandin2-15/+15
Change-Id: If25e3b030d484f03078bfba265bed82b315e2742
2014-11-16Logically dead code related to embedded stream name in SwGrfNode\SdrGrafObjZolnai Tamás1-55/+2
See also: 286e2f5c6ec829bc0987b1be7016699f7ef03e5e Since embedded URL exists until the first swap in these lines are not needed anymore. Change-Id: Ie6bf8efe7808cf42f20f7b4b3f8cb927555c0ea8
2014-11-16Set back these lines, later it can be usefulZolnai Tamás1-0/+4
Removed in: 9dc3b49c891fb9fe45c24de4b7e1e88fe400afe0 Change-Id: Id8cee4e17d214ca0eaa5cd11dc25849e5f68851e
2014-11-16Avoid DelStreamName because it can lead to image loss.Zolnai Tamás1-41/+0
See also: f811e628411bda29a76ebb1f72eb107ce67d27f0 The problem is that more images can have the same stream name so we can't decide here when to remove one stream name. Better to leak in the storage as to loose images (actually we already leak here, so) Change-Id: I2c2afe87e024c2521fe22d62126b567931604101
2014-11-16Related fdo#82953: Forget package URL of image after it is loadedZolnai Tamás1-0/+4
It causes problems if we handle those imported images differently which are identified by a package URL, so after the first load remove this URL and handle images on the same way as inserted images. Some related bugs: * #i44367# * #i124946# * #i114361# * fdo#73270 The image in the test document has a special ID which is different from that one which is generated by LO internally so after ODF export the new generated image URL is different from the imported one. Change-Id: I4e7d3490674c5f86bec5c7c6e1c975dcafd7c265
2014-11-14vcl: move the Impl class of MapMode ...uhm... out of the public header?Michael Stahl1-0/+1
And remove the tools/fract.hxx include. grep -l -r fract.hxx workdir/Dep/*Object* |wc -l before: 4569 after: 1851 Shrinks some libraries, example from --enable-dbgutil Fedora gcc 4.8.3: libvcllo.so by 473k (0.35%) libswlo.so by ~1Mb (sadly that is just 0.2%) Change-Id: I09bd025d551a5d2c5528b938a68c6aa5f8f114a0
2014-11-07Remove some useless swapping call inside SwGrfNodeZolnai Tamás1-13/+1
Change-Id: I4cd2677197c7a6cff71e2966c2b2dd2285032c07
2014-11-07Be explicit here when to swap out as a linkZolnai Tamás1-9/+8
Change-Id: I70dd00d5f82c5f4f622805e1d6ee1dfc30900b31
2014-11-07More replacements of manual swapping with SwGrfNode::GetGrf()/GetGrfObj()Zolnai Tamás1-5/+5
Change-Id: Ie56584c03af8a6d3ea8f8d4294f5492a841933b7
2014-11-07Make GetTwipSize() call SwapIn() by itself in case of invalid sizeZolnai Tamás1-0/+4
Change-Id: I5bc6cf097e61d65007dde531af4a213b19e8ca5b
2014-11-07Swap in graphic by SwGrfNode and not manuallyZolnai Tamás1-0/+12
It's a Writer specific problem, that images lost during export because of not swapped in graphic data. Other components (Impress, Calc...) use SdrGrafObj to get the graphic and SdrGrafObj calls swap in before retrun with the graphic. Change-Id: I7398d8e3f6535199b10de048acd58543bdb42531
2014-11-07Make SetSwapState() an internal methodZolnai Tamás1-7/+0
So we can be sure it is always called when user data changed. Change-Id: If107907afffb85a7a57817f5807847a5c028416c
2014-11-07Avoid using null pointer as a special indicatorZolnai Tamás1-1/+1
When ImpGraphic::ImplSwapOut() is called with null pointer it was assumed that it is becase the graphic is a link and so we don't need to swap out it actually (we can load it anytime using the link), only clear the graphic's internal data. The problem with that it can happen that ImplSwapOut() is called with null pointer accidentally on a non-link graphic object which leads to that we loose the graphic. Seems more robust to use an explicit indicator (GRFMGR_AUTOSWAPSTREAM_LINK) for links swapout. indicator Change-Id: Icf31524a192c7866278ba6a13eb85648aa69f554
2014-10-23Fraction: Revert "fdo#81356: convert Fraction to boost::rational<long> - wip"Jan Holesovsky1-4/+4
This reverts commit 47a2d7642d249d70b5da0c330a73f3a0032e4bba. Conflicts: cui/source/tabpages/transfrm.cxx svx/source/svdraw/svdedtv1.cxx svx/source/svdraw/svdibrow.cxx sw/source/filter/ww1/w1filter.cxx tools/source/generic/rational.cxx Change-Id: I4849916f5f277a4afef0e279b0135c76b36b9d15
2014-10-23Fraction: Revert "fdo#84854 it seems long is not enough on 32 bit"Jan Holesovsky1-4/+4
This reverts commit 582ef22d3e8e30ffd58f092d37ffda30bd07bd9e. Conflicts: svx/source/svdraw/svdedtv1.cxx svx/source/svdraw/svdibrow.cxx sw/source/filter/ww1/w1filter.cxx Change-Id: I80abc7abdeddc267eaabc9f8ab49611bb3f8ae83