summaryrefslogtreecommitdiff
path: root/svtools
AgeCommit message (Collapse)Author
2016-07-29loplugin:countusersofdefaultparams in store..svtoolsNoel Grandin
Change-Id: I15b4400bddc5a4d0e3de5dfffe18b7e493f97df6 Reviewed-on: https://gerrit.libreoffice.org/27580 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-28Resolves: tdf#94940 don't load font history if option is disabledCaolán McNamara
Change-Id: I70348b4e4bb85850331044e901c03aabb0d22ef8
2016-07-27improve passstuffbyref return analysisNoel Grandin
Change-Id: I4258bcc97273d8bb7a8c4879fac02a427f76e18c Reviewed-on: https://gerrit.libreoffice.org/27317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-26masses of MessBoxes not being disposed promptlyCaolán McNamara
since... commit ba81e5c6bd420b41a84ade6ccd774011a8089f7f Date: Thu May 28 21:35:43 2015 +0100 tdf#91702 - fix stack-based MessBox allocation. There is no special ScopedVclPtr<X>::Create or ScopedVclPtrInstance<X>::Create just VclPtr<X>::Create and a raw VclPtr<X>::Create()->foo doesn't call dispose on the owned X Change-Id: Ifacc8d5e742820701307c3c37b9b86487667d84f
2016-07-25new loplugin overrideparamNoel Grandin
verify that parameters on override methods have the same set of default values for their params as their parent/super-methods do. Change-Id: Ibdbc1c6e417fbaa680ea025a6bbf5ba9c2e5bcd2 Reviewed-on: https://gerrit.libreoffice.org/27437 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-22Fix --disable-pdfimport buildMiklos Vajna
Change-Id: Ida06b892aea92cfe26fd62372e21ca611d8838d3 Reviewed-on: https://gerrit.libreoffice.org/27447 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2016-07-18Resolves: tdf#100823 crash during PopupMenu dtorCaolán McNamara
Change-Id: I6d9aed05a76311a13470c9072543609f5a87501d
2016-07-15new loplugin unnecessary overrideNoel Grandin
Change-Id: I88d3e33823d68745b98625050a8a274f9ef04bcb Reviewed-on: https://gerrit.libreoffice.org/27135 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2016-07-13Resolves: tdf#100340 make compress graphic dialog less wideCaolán McNamara
'regression' seems harsh, its not so wide it didn't fit on a reasonable sized screen or look particularly horrific. Change-Id: I20d55b8aac609ee0d683eb9a1c2b173aa0c9d8da
2016-07-13Resolves: tdf#59570 scrollbar used instead of sliderCaolán McNamara
Change-Id: I14241c8a4418e8356a590a32b661f283b606f9fa
2016-07-10ditch the font/screen-scaling thingCaolán McNamara
its of limited utility wrt the vcl layout work and optimal size widgets, and its confuses people looking for hidpi settings Change-Id: I37d21b480446f9fe2845b3a2d9fd59cdeddf1a93
2016-07-08Resolves: tdf#100452 class Date full (BCE,CE) proleptic Gregorian calendarEike Rathke
... implementing signed years with year 0 gap. Date(31,12,-1) last day BCE Date(1,1,1) first day CE New class Date member functions: * AddYears(sal_Int16) to be used instead of aDate.SetYear(aDate.GetYear()+sal_Int16) to handle year 0 gap. * convenience GetNextYear() to be used insted of GetYear()+1 * convenience GetPrevYear() to be used insted of GetYear()-1 * AddMonths(sal_Int32) * operator=(const css::util::Date&) New class DateTime member functions: * operator=(const css::util::DateTime&) Made some conversion ctors explicit, specifically Date(sal_Int32) Adapted hopefully all places that used a sal_uInt16 year to use sal_Int16 where appropriate. Eliminated some quirks in date handling found on the fly. Added era handling to i18npool icu calendar setting interface, which missing was responsible for 0001-01-01 entered in Calc being set as -0001-01-01, hence subtracting one day resulted in -0002-12-31. Change-Id: I77b39fba9599ebd5067d7864f6c9ebe01f6f578f Reviewed-on: https://gerrit.libreoffice.org/27049 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Jenkins <ci@libreoffice.org>
2016-07-07loplugin:passstuffbyref also for {css::uno,rtl}::ReferenceStephan Bergmann
Change-Id: I3b52f6a392b6f6dcc8e3d528dc61bf49c510737d
2016-07-07FloatWinPopupFlags::NoFocusClose is only set, never readCaolán McNamara
Change-Id: Ide67b9279b626d3427c93891d27378ffd1f004cc
2016-07-06restore loplugin:vclwidget checking for calling clear() on VclPtr fieldsNoel Grandin
Change-Id: I85eda1c33016c1461d897fc0a3b70457209a7405 Reviewed-on: https://gerrit.libreoffice.org/26806 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-05when disposing popup menus, don't leave dangling pointersNoel Grandin
Change-Id: Ia1b49937ba0693a567097367b99b75a3011666bf Reviewed-on: https://gerrit.libreoffice.org/26960 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
2016-07-05merge svt::StoreString with svt::StoreContentURLNoel Grandin
Change-Id: I6f2a7102c697cecc96e8b950b499bc9b0f91240e Reviewed-on: https://gerrit.libreoffice.org/26904 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-04remove comphelper::string::removeNoel Grandin
and replace it with OUString/OString::replaceAll Change-Id: I37b1c3b51251dfd9d749d6f1060c75b3a93d7f1a Reviewed-on: https://gerrit.libreoffice.org/26850 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-04comphelper::OBaseMutex -> cppu::BaseMutexNoel Grandin
convert usage of deprecated class, and remove the old class Change-Id: I19fb9271090d19e5531622b58492e1a848d306e2 Reviewed-on: https://gerrit.libreoffice.org/26843 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-07-01fix mis-mergeStephan Bergmann
Change-Id: I529b7abff98e38e503ffbcb39946e30d12a053cc
2016-07-01GSoC possibility to change icon size in the SidebarToolBoxSzymon Kłos
+ Added registy entry to store icon size + Added UI to change settings: Tools -> Options... -> View + Loading last settings in the SidebarToolBox + Settings update listener Update icon code from: framework/source/uielement/toolbarmanager.cxx Change-Id: I1d713c50fccfc19e1c8ea82eba68556ddb76cd3c Reviewed-on: https://gerrit.libreoffice.org/26362 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2016-06-29vcl: handle PDF data in Read/WriteImpGraphic()Miklos Vajna
Initially I wrote the test to cover f7d6f3e4e3fda3cd4936880918e2831246634a3e (svtools: survive swapout/in of graphic with embedded pdf, 2016-06-28), but as it turns out the actual swapin/out code is in vcl, so extend that as well to support pdf data. Change-Id: I320ea729dce9440b84a5089607807322ea7b0b04 Reviewed-on: https://gerrit.libreoffice.org/26755 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2016-06-28loplugin:singlevalfields in variousNoel Grandin
Change-Id: Ia0d8f463a4dba9ec63aa0159441e3e607dd3bf5e Reviewed-on: https://gerrit.libreoffice.org/26738 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-28svtools: survive swapout/in of graphic with embedded pdfMiklos Vajna
Use case is to insert a pdf file in Impress, then choose Save from the context menu of the image. On one hand, for some reason only bitmap images can be saved, so add an exception for the graphic-with-pdf case. SVG is not affected, as it counts as bitmap with its PNG replacement image, it seems. On the other hand, copying the GraphicObject around triggers swapout/swapin of the graphic, and there the original pdf data was lost, fix that. Change-Id: I8b4b67cd951e39b0250a101766f487ddb1088cab Reviewed-on: https://gerrit.libreoffice.org/26739 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
2016-06-27Clean up uses of Any::getValue() in svtoolsStephan Bergmann
Change-Id: Id11dc4724fed4e43981534ffac5ec5358f9387b2
2016-06-24clang-analyzer-deadcode.DeadStoresStephan Bergmann
Change-Id: I1660e3ed5eb5fac276f3715fe1ffa2308408f1b5
2016-06-24loplugin:singlevalfields in vcl(part1)Noel Grandin
Change-Id: I0031199937cc95793951a070c4b3d8910933e69f Reviewed-on: https://gerrit.libreoffice.org/26595 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-24loplugin:singlevalfields in svtoolsNoel Grandin
Change-Id: I80fe1d4646af2b8d8e6362a25f6cda4b7ac29eab Reviewed-on: https://gerrit.libreoffice.org/26603 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-23VclPtr: svtools error message dialog doesn't disposeCaolán McNamara
Change-Id: I83640cc60d5e0239d228c749c32f896df5e2d9f2
2016-06-23tdf#46279 Show restart message after extension installationirem
Change-Id: I91feab63b77055b8f73d0fb481a76586b249160c Reviewed-on: https://gerrit.libreoffice.org/24595 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-06-23tdf#97527 - vcl: reference-count MenuNoel Grandin
some places are marked with "dodgy"- need to check those to see what is going on, because they are leaving dangling pointers behind in the Menu class Change-Id: I41d5c7c0fec2f70ce9e3ffdc48cd03d26c0a869b Reviewed-on: https://gerrit.libreoffice.org/26516 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-22loplugin:stringconstantStephan Bergmann
Change-Id: I98c383106c8d5874bc8eb3daf05ca3f172d9f41a
2016-06-22svtools: fix -Werror=unused-parameterMiklos Vajna
Change-Id: Ic8887571e376483e84d7c2eb270d3daaa37c63f3
2016-06-22uitest: support tree listsMarkus Mohrhard
Change-Id: Iaa1a49d7e38c8d87bb3cfd749af94a0b92feee0d
2016-06-22SAL_FINAL to finalXisco Fauli
Change-Id: I58e9c533819218482f89239f6d9b07e3497d4438 Reviewed-on: https://gerrit.libreoffice.org/26558 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-21tdf#89329: use shared_ptr for pImpl in slidesorterbaroptXisco Fauli
Change-Id: If5b7fd23448a584d1faebaf0a540e61f091471fa Reviewed-on: https://gerrit.libreoffice.org/26331 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-21loplugin:overrideStephan Bergmann
Change-Id: I8ce79a9c4398657e2a6b4c94fecd21ade71e4ed0
2016-06-21tdf#89329: use shared_ptr for pImpl in optionsdrawinglayerXisco Fauli
Change-Id: Ic9da196c1dfe138ff6408a37fa8ff70492d00bc2 Reviewed-on: https://gerrit.libreoffice.org/26282 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-20Convert PART to scoped enumNoel Grandin
Change-Id: If4c2849beb207593d3d450ae3846ed24eaf66ca4 Reviewed-on: https://gerrit.libreoffice.org/26173 Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-19loplugin:salbool: Implicit conversions from non-Boolean fundamental typesStephan Bergmann
Change-Id: I67eac95686678e6f5a2d60798535b2c65a9ba5d7
2016-06-18uitest: set ids for a few UI objects created in codeMarkus Mohrhard
Change-Id: I421b97448bb596b134908ca6ddd60d7b4f60b186
2016-06-18uitest: an enum is a bad idea for the typeMarkus Mohrhard
Change-Id: I1efd59b2f664f0aa9be733466fa7c1057e92ef27
2016-06-18uitest: add initial version of wrapper for tree list boxMarkus Mohrhard
The tree list will also be the first case where we need non-vcl::Window based children. We should create one child for each entry to make it possible to interact with them correctly. Change-Id: I49e1ddf7b271946fd595ebfe2f4f2d0c8a535fdc
2016-06-17loplugin:overrideStephan Bergmann
Change-Id: I2c33f5f91f16c465cb329204529b76e1c6165535
2016-06-17tdf#89329: use shared_ptr for pImpl in helpoptXisco Fauli
Change-Id: I9d3fa64405f70cfa942935eaae6cc520f172d70d Reviewed-on: https://gerrit.libreoffice.org/26281 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-16properly prefix various global variables and hide themMichael Stahl
Change-Id: I6b97e6a4c6ac4c0e26bcbb2b132b333920c16f19
2016-06-16Local linkage for static var (and drop misleading "m_" prefix)Stephan Bergmann
Change-Id: Ide9d2262192899f7a1444760c0df806d30514919
2016-06-16tdf#89329: use shared_ptr for pImpl in menuoptionsXisco Fauli
Change-Id: I93ece349dc15ea9af00c661ac34fed80a57ea3d2 Reviewed-on: https://gerrit.libreoffice.org/26318 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-16tdf#89329: use shared_ptr for pImpl in toolpaneloptXisco Fauli
Change-Id: I2035971f6633aed1389ffae5815c7000699b9735 Reviewed-on: https://gerrit.libreoffice.org/26279 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-06-16tdf#89329: use shared_ptr for pImpl in miscoptXisco Fauli
Change-Id: I7c60116d4f14e1451e7b96568104baada01dfaca Reviewed-on: https://gerrit.libreoffice.org/26330 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>