summaryrefslogtreecommitdiffstats
path: root/editeng/source/items/numitem.cxx
Commit message (Collapse)AuthorAgeFilesLines
* tdf#42949 Remove unnecessary localization headers of editengGabor Kelemen2018-04-181-1/+0
| | | | | | | | | | Found by searching for the header names and the localization function: git grep -l -e \<editeng/eerdll.hxx\> -e \<editeng/editrids.hrc\> | xargs grep -c EditResId | grep :0$ | grep -v /pch Change-Id: Ic542ceee09352cb52dcdc9780d4cb1890e02f383 Reviewed-on: https://gerrit.libreoffice.org/53062 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* loplugin:constantparam in desktop..i18npoolNoel Grandin2018-03-081-1/+1
| | | | | | | Change-Id: Ie2e1004c1ccc03777a8da9cb1144e89eb28ff313 Reviewed-on: https://gerrit.libreoffice.org/50928 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more Color in editeng..formsNoel Grandin2018-03-071-1/+1
| | | | | | | Change-Id: If6c862e7bb61cd78c3379afde11b528a74162900 Reviewed-on: https://gerrit.libreoffice.org/50860 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* drop old tools/gen methods in accessibility..formulaNoel Grandin2018-03-011-2/+6
| | | | | | | Change-Id: I075e29173945200854f2ef8e420867871659766a Reviewed-on: https://gerrit.libreoffice.org/50446 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Revert "loplugin:changetoolsgen in editeng..extensions" and reapply pluginNoel Grandin2018-02-161-2/+2
| | | | | | | | | | | | | because I (a) forgot to insert parentheses which changes the meaning of some expressions and (b) I now use the AdjustFoo calls when changing unary operations, which reads much better This reverts commit 2096aac8b958db66b3ddce16b06dca87edc8ba0a. Change-Id: I4b5941c4119b95aaefb9180a0ca95a1dbb4ec317 Reviewed-on: https://gerrit.libreoffice.org/49844 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:changetoolsgen in editeng..extensionsNoel Grandin2018-02-141-2/+2
| | | | | | | Change-Id: I476fd8b988216a300c57fcf184ea3742139363fe Reviewed-on: https://gerrit.libreoffice.org/49656 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:useuniqueptr in SvxNumRuleNoel Grandin2018-02-051-16/+11
| | | | | | | Change-Id: I778f62edbddfa2ca10eefa396272e7b011025513 Reviewed-on: https://gerrit.libreoffice.org/49172 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:useuniqueptr in SvxNumberFormatNoel Grandin2018-02-051-17/+11
| | | | | | | Change-Id: I0433349e6f4108297e0f4cab65c1e859424bd282 Reviewed-on: https://gerrit.libreoffice.org/49148 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* new loplugin: pointerboolNoel Grandin2018-01-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | look for possibly bogus implicit conversions to bool when passing (normally pointer) args to bool params. this plugin comes in the wake of a couple of bugs caused by refactoring, where some of the call sites were not currently updated. Of the changes, the following are real bugs: desktop/../dp_persmap.cxx StartInputFieldDlg in sw/../fldmgr.cxx which occurred as a result of commit 39d719a80d8c87856c84e3ecd569d45fa6f8a30e Date: Tue May 3 11:39:37 2016 +0200 tdf#99529 sw: don't pop up input field dialog before inserting field CSerializationURLEncoded::encode_and_append in forms/../serialization_urlencoded.cxx XclExpCFImpl::XclExpCFImpl in sc/../xecontent.cxx I have no idea how to properly fix this, just made a guess. SwDocTest::test64kPageDescs in sw/qa/core/uwriter.cxx which looks like a simple copy/paste error. Change-Id: I795ebd5ef485a1d36863dc27fe13832989f5a441 Reviewed-on: https://gerrit.libreoffice.org/48291 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* More loplugin:cstylecast: editengStephan Bergmann2018-01-151-8/+8
| | | | Change-Id: I58978e5376ff1b91fa6dddbd308c923b3f3ab0ec
* More loplugin:cstylecast: editengStephan Bergmann2018-01-121-3/+3
| | | | | | | | auto-rewrite with <https://gerrit.libreoffice.org/#/c/47798/> "Enable loplugin:cstylecast for some more cases" plus solenv/clang-format/reformat-formatted-files Change-Id: I09a4d688e6f3c1ecbe05a7d27ebd955e8ba1eb65
* tools: remove unused #include from solar.hMichael Stahl2017-12-131-0/+1
| | | | Change-Id: I214e13add371380701ae39403d90a574a63e495d
* Translate German variable nameJohnny_M2017-11-211-5/+5
| | | | | | | Change-Id: I70f9355bc7f219e8d6aaf9fb8ecc356e1a86d9da Reviewed-on: https://gerrit.libreoffice.org/44921 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jens Carl <j.carl43@gmx.de>
* rename AvoidConfig to FuzzingCaolán McNamara2017-11-091-1/+1
| | | | | | | | | | | | cause that's what its really used for and a couple of cases are not specifically about avoiding config but avoiding uninteresting disk acccess and what not Change-Id: I4c6454f98388579fcd0bf9798321d30408ab65ee Reviewed-on: https://gerrit.libreoffice.org/44491 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* convert GetNumStr from sal_uLong->sal_Int32Noel Grandin2017-11-061-3/+3
| | | | | | | | | the code was truncating the param to sal_Int32 anyway Change-Id: I743b60097df937c16c995e2e6deacc12cf12e5e9 Reviewed-on: https://gerrit.libreoffice.org/44341 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* remove unused aLocale from SvxNumRuleNoel Grandin2017-09-181-2/+4
| | | | | | | | | | and fix usage site, regression from commit 41956321ceefb5836feb2c119b66de7bb557740f Date: Sat Nov 24 18:54:09 2012 +0100 get rid of Svx...Locale...() double conversion nonsense Change-Id: Ic70abf5f52dce1db9cffd9c819e57f1a6c2a3df3
* Removing unused SfxItemPool serialisation from editengVarun Dhall2017-08-071-12/+0
| | | | | | | Change-Id: I051201c272b7acdd48d54cc0c230f8b432196188 Reviewed-on: https://gerrit.libreoffice.org/40429 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Jenkins <ci@libreoffice.org>
* loplugin:constparams in editengNoel Grandin2017-07-281-2/+2
| | | | | | | Change-Id: Ice32a16d14d0132a19e8209d311bb799fd7ee0c8 Reviewed-on: https://gerrit.libreoffice.org/40526 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* teach unnecessaryparen loplugin about identifiersNoel Grandin2017-07-101-1/+1
| | | | | | | Change-Id: I5710b51e53779c222cec0bf08cd34bda330fec4b Reviewed-on: https://gerrit.libreoffice.org/39737 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* new loplugin unnecessaryparenNoel Grandin2017-07-051-1/+1
| | | | | | | Change-Id: Ic883a07b30069ca6342d7521c8ad890f4326f0ec Reviewed-on: https://gerrit.libreoffice.org/39549 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:constantparam part2Noel Grandin2017-04-051-3/+2
| | | | | | | Change-Id: I7ec4c946be52a6b56aee908426f95ecacc7b0746 Reviewed-on: https://gerrit.libreoffice.org/36072 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* convert SvxAdjust to scoped enumNoel Grandin2017-03-081-2/+2
| | | | | | | Change-Id: I0df7a5313d6d62d0c657e62ef6014a3ddbab4639 Reviewed-on: https://gerrit.libreoffice.org/34941 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* 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-5/+5
| | | | | | | | | 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>
* new loplugin: useuniqueptr: editengNoel Grandin2017-01-131-5/+3
| | | | | | | Change-Id: I6df65eab882780d996ee996b5fef8020186b6d98 Reviewed-on: https://gerrit.libreoffice.org/32958 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* make the element names in dumpAsXml match the class namesNoel Grandin2016-11-161-2/+2
| | | | | | | Change-Id: I955facfe3e901fcb76798dab342f96a67d5ac63f Reviewed-on: https://gerrit.libreoffice.org/30894 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unusedmethodsNoel Grandin2016-10-181-5/+0
| | | | | | | Change-Id: Icf773925c37dde8b7404edac9864e7b10fe113b4 Reviewed-on: https://gerrit.libreoffice.org/29968 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:constantfunctionNoel Grandin2016-10-111-2/+0
| | | | | | | | | | | | | update the plugin similarly to commit 3ee3b36ae0c064fb5c81268d8d63444309d1b970 Author: Stephan Bergmann <sbergman@redhat.com> Date: Fri Oct 7 12:05:49 2016 +0200 loplugin:staticmethods: Don't be fooled by decls starting with macros Change-Id: I98ac3216d5acf89a49a26feb089ae2fd34e6e510 Reviewed-on: https://gerrit.libreoffice.org/29665 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* convert MapUnit to scoped enumNoel Grandin2016-10-051-2/+2
| | | | | | | | | | | | I left a prefix on the names "Map" so that I would not have to re-arrange each name too much, since I can't start identifiers with digits like "100thMM" And remove RSC_EXTRAMAPUNIT, which doesn't seem to be doing anything anymore. Change-Id: I5187824aa87e30caf5357b51b5384b5ab919d224 Reviewed-on: https://gerrit.libreoffice.org/29096 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:implicitboolconversionNoel Grandin2016-09-211-1/+1
| | | | Change-Id: Id7dafb926a0eb7fae5e481555eae3d35fe795900
* add dumpAsXml() to more pool itemsNoel Grandin2016-09-211-1/+30
| | | | Change-Id: I276109148bab8180ce2442f5ee69722357362453
* Resolves: tdf#97404 outline space wrong on second levelCaolán McNamara2016-07-191-24/+1
| | | | | | | | | | | | | | | | | | | | | Revert "tdf#42788: FORMATTING - Numbering/ordered list" This reverts commit 6517141b6233c5f9667031bc92f66109fddf5b76. https://gerrit.libreoffice.org/#/c/23006/ was apparently intended to revert this, but it never was Then there was https://gerrit.libreoffice.org/#/c/23427/ which was an attempt to improve it but it was abandoned Then there was https://gerrit.libreoffice.org/#/c/23542/ whichw as another attempt to improve it, but it was too abandoned. all of which leaves us worse than when we started with a pile of unhappy abandoned efforts and cruddy indent Change-Id: I8b3666a957339030d5a946a2b387f452e7fb0099
* Add SAL_FALLTHROUGH, where apparently appropriateStephan Bergmann2016-05-101-2/+2
| | | | Change-Id: I005943a7c7bc4f8bfc647b2cf44f143a66a7e997
* clang-tidy modernize-loop-convert in e*Noel Grandin2016-04-261-2/+2
| | | | | | | Change-Id: If56abefa81b41479e3ea9890dee1c43f006086de Reviewed-on: https://gerrit.libreoffice.org/24384 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unuseddefaultparams in editengNoel Grandin2016-03-031-5/+2
| | | | Change-Id: I1dc0ba262c06bd69cf92aae20b344fe23f460f55
* Further uninitialized SvxNumberFormat::mbNumAdjustChanged fixingStephan Bergmann2016-01-221-1/+1
| | | | | | | | | | | | | The uninitialized read during CppunitTest_sd_html_export_tests mentioned in d90a20833a815362c344b82fc16b4bbcff0ebf07 "Missing initialization of SvxNumberFormat::mbNumAdjustChanged" is rather caused by the SvxNumberFormat copy ctor callling the SvxNumberFormat copy assignment op, which calls SvxNumberFormat::SetNumberingType before assigning mbNumAdjustChanged. What is probably wanted here is to simply copy the nNumType member without doing the eNumAdjust adjustments anyway. But a rather brittle design, that. Change-Id: Ic587687038f1ea71173461412350e6b2dee13aa4
* TypoKhaled Hosny2016-01-221-1/+2
| | | | Change-Id: Ib4c474cab050cdef003dc5e611e8a9bc1fb46e86
* Missing initialization of SvxNumberFormat::mbNumAdjustChangedStephan Bergmann2016-01-221-1/+2
| | | | | | | | introduced in 6517141b6233c5f9667031bc92f66109fddf5b76 "tdf#42788: FORMATTING - Numbering/ordered list," causing a read of the uninitialized value in CppunitTest_sd_html_export_tests. Change-Id: I7955593999e3a2bedca815ee447d96c1b997a358
* tdf#42788: FORMATTING - Numbering/ordered listNusaiba Al-Kindi2016-01-211-0/+21
| | | | | | | | | | | | I changed the default numbering alignment of the Roman(upper/ lower) numbers list to the right instead of left with out any changes in the default alignment of other numbring type lists. In addition, numbering alignment can be changed by the user and any new list created will not be affected by the user choice (using the default settings for the new list). Change-Id: I7a6b7fb31bf40e26910e90538cd336e767f4c056 Reviewed-on: https://gerrit.libreoffice.org/21439 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
* vcl: change Font::SetName() to Font::SetFamilyName()Chris Sherlock2016-01-181-1/+1
| | | | | | | Change-Id: I9b7bd1ec25955981ebcb9085579cbb9fbaa802d5 Reviewed-on: https://gerrit.libreoffice.org/21560 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* vcl: rename Font::GetName to Font::GetFamilyNameChris Sherlock2016-01-171-1/+1
| | | | | | | Change-Id: Ie20871a3078bf875c1782b7761d60591a9c9704f Reviewed-on: https://gerrit.libreoffice.org/21529 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* revert vcl patch series that brok Mac and WindowsNorbert Thiebaud2016-01-161-2/+2
| | | | | | | | | | | | | | | | | | revert: 9bc2f3de8672e812f3a67541c6d7069b434a7e42 vcl: add comment about ImplFontMetric::{Get|Set}LineHeight() 26371f105bc44e04469ec03fc5bb12505e651c6b vcl: FontAttributes::GetSlantType() -> FontAttributes::GetItalic() 2dd0b4317372b8022efe3911b38b4fa02956d8b9 vcl: font.hxx and metric.hxx cleanup, make ctors explicit and the dtor virtual 5ab13bf3ead3539e4ad847656da81e7eb6029652 vcl: tabify font.hxx f99550dae55e40e49bf9c9875053fe2abb4c71ca vcl: change Font::SetName() to Font::SetFamilyName() 2b297116cb6bb1061c43e5714e2609c8ee9f57d2 vcl: rename Font::GetName to Font::GetFamilyName Change-Id: I23f38921187d66b970ca815eb30dda4222a2da62
* vcl: change Font::SetName() to Font::SetFamilyName()Chris Sherlock2016-01-161-1/+1
| | | | | | | Change-Id: I54a4036544c680c4a49607677af776aa7a433fbc Reviewed-on: https://gerrit.libreoffice.org/21510 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
* vcl: rename Font::GetName to Font::GetFamilyNameChris Sherlock2016-01-161-1/+1
| | | | | | | Change-Id: I83927e0992dfe0a2a79d139818a9f45d3761aae5 Reviewed-on: https://gerrit.libreoffice.org/21509 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
* loplugin:unusedmethods unused return value in include/editengNoel Grandin2016-01-121-10/+4
| | | | | | | Change-Id: I1314480950b0d3a3e5ed066d71c175604dd41970 Reviewed-on: https://gerrit.libreoffice.org/21361 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-101-23/+23
| | | | Change-Id: I9fe00eef7ddcd4a3c87e497a8d62f98e71a0d6d8
* use uno::Reference::set method instead of assignmentNoel Grandin2015-10-311-1/+1
| | | | | | | Change-Id: I150f213ab033f5fc1b50abe4e1529292206c9feb Reviewed-on: https://gerrit.libreoffice.org/19689 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* CppCheck : reduce variables scopeMichaël Lefèvre2015-09-231-1/+1
| | | | | | | | Change-Id: Ief402017b693a4337f330fb07bb7a6dc6e749f72 Reviewed-on: https://gerrit.libreoffice.org/18753 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* editeng: com::sun::star->cssNoel Grandin2015-09-131-4/+4
| | | | | | | Change-Id: Ied65d2de394ce54cb2c35abb152cf58ac3139395 Reviewed-on: https://gerrit.libreoffice.org/18503 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* for testing allow disabling configmgr for time critical pathsCaolán McNamara2015-08-191-2/+5
| | | | | | | | Change-Id: I08021f18d53e1748927f8847649994f95252bbc2 Reviewed-on: https://gerrit.libreoffice.org/17844 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>