summaryrefslogtreecommitdiff
path: root/filter
AgeCommit message (Collapse)Author
2017-11-07convert nDrawingContainerId to sal_uInt16Noel Grandin
which is all that can be stored anyhow Change-Id: Iaec28a97ed25004715accafc0cbd4bd620bfbba9 Reviewed-on: https://gerrit.libreoffice.org/44360 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-05tdf#113037 create Watermark with correct ratioSzymon Kłos
Change-Id: I17ab8d98a618cff6e38dad344f4096dc520cce43 Reviewed-on: https://gerrit.libreoffice.org/44228 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
2017-11-05ofz#4058 Integer-overflowCaolán McNamara
Change-Id: I76050f392f04cff54e52eb913ceb451bc5973a72 Reviewed-on: https://gerrit.libreoffice.org/44323 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-04loplugin:constparams in filterNoel Grandin
Change-Id: I9fc76693da88d74b6fe7bbb920b8e731b82be298 Reviewed-on: https://gerrit.libreoffice.org/44260 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-03vcl: remove sgf/sgv ancient gfx filtersThorsten Behrens
Change-Id: Ib02986e5b3d54d65982836e22ebcb70eb7a0e7b3 Reviewed-on: https://gerrit.libreoffice.org/44216 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2017-11-02Divide-by-zeroCaolán McNamara
Change-Id: Ieb94b1ff71fb70f38d8bf8a1b9f62edbc3cfb867 Reviewed-on: https://gerrit.libreoffice.org/44198 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-02Integer-overflowCaolán McNamara
Change-Id: Ic99219ff65cea0f316831696231ed6ba8dd10b60 Reviewed-on: https://gerrit.libreoffice.org/44199 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-02loplugin:finalclasses in filter,extensionsNoel Grandin
Change-Id: I2822b8c5a91feaf47466356c01728d32a85bbae9 Reviewed-on: https://gerrit.libreoffice.org/44190 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-11-02coverity#1420538 Copy-paste errorCaolán McNamara
Change-Id: If85024b144f9d13da35b1f548100cf5860f3c75f
2017-11-01de-sal_uLong egifCaolán McNamara
Change-Id: I4d18c60f618d2b3c04ad256324ee2620cc583775 Reviewed-on: https://gerrit.libreoffice.org/44162 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-01ofz#3948 Integer-overflowCaolán McNamara
Change-Id: I4bb7d6012b18221706b05c4c8a4e09e6aac1dac6 Reviewed-on: https://gerrit.libreoffice.org/44119 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-11-01ofz: Divide-by-zeroCaolán McNamara
Change-Id: Id6153afbfd04ac2eafc8a8670144e157cb527538 Reviewed-on: https://gerrit.libreoffice.org/44126 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-31ofz#3932 Divide-by-zeroCaolán McNamara
Change-Id: I912e082826f8433254edafe146bf5e746fdf914a Reviewed-on: https://gerrit.libreoffice.org/44100 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-31loplugin:constantparam in f*Noel Grandin
Change-Id: I87145db3af6c3eb180cea6b4244f98b00205a306 Reviewed-on: https://gerrit.libreoffice.org/44095 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-10-31ofz#3931 Integer-overflowCaolán McNamara
Change-Id: Iab9e4c300e0b860d7fac5274b7ca6345c56343f8 Reviewed-on: https://gerrit.libreoffice.org/44072 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-30tdf#96505 Get rid of cargo cult long integer literalsudareechk
Removing unnecessary 1L occurrences Change-Id: I85d4949eacde0dfc0ff7a513f4bf42324e6d34c1 Reviewed-on: https://gerrit.libreoffice.org/42840 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2017-10-30ofz#3901 Integer-overflowCaolán McNamara
Change-Id: Iebb9c82aea61a255b1a5ef4d9f1b66bc0fb17c59 Reviewed-on: https://gerrit.libreoffice.org/44053 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-30ErrorCode can be a simple intCaolán McNamara
Change-Id: Ia0e9cc193be6f49a5a58c2f5ea5c7a5897e8400e Reviewed-on: https://gerrit.libreoffice.org/44035 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-30de-sal_uLong met filterCaolán McNamara
Change-Id: Ia9c0051f42801e8017469c77bf3d89672a748115 Reviewed-on: https://gerrit.libreoffice.org/44034 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-29ofz: Divide-by-zeroCaolán McNamara
Change-Id: I1bf3e17950bda3386514dd5956b89df89565b780 Reviewed-on: https://gerrit.libreoffice.org/44024 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-29ofz#3872 Divide-by-zeroCaolán McNamara
Change-Id: I851fdb905e0ca981cba6699fcbe20c5d8fabf7ae Reviewed-on: https://gerrit.libreoffice.org/44009 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-27ofz#3837 Integer-overflowCaolán McNamara
Change-Id: Iaf9573ba7a8c08ba8c1b8dcf62340d0ea213e1a9 Reviewed-on: https://gerrit.libreoffice.org/43944 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-27tdf#113162 corect the size only for watermark 2Szymon Kłos
Missing part for: 91d909ac8ec53ea135401babe8227f9c41b779b8 Change-Id: Id35d61084e034c7381c8e8016e50dd66aaf41d59 Reviewed-on: https://gerrit.libreoffice.org/43915 Reviewed-by: Szymon Kłos <szymon.klos@collabora.com> Tested-by: Szymon Kłos <szymon.klos@collabora.com>
2017-10-27ofz#3834 Divide-by-zeroCaolán McNamara
Change-Id: I654152b964c6228efeb8b7cc7203064338fd1dd6 Reviewed-on: https://gerrit.libreoffice.org/43934 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-27ofz#3811 Integer-overflowCaolán McNamara
Change-Id: Ia08ac1ae46d4af6df7b0a590752e17a6d9a6836e Reviewed-on: https://gerrit.libreoffice.org/43869 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-27Fix typosAndrea Gelmini
Change-Id: I96f64b7c259f04a67684bb699ea3cc43b68e40fa Reviewed-on: https://gerrit.libreoffice.org/43913 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
2017-10-26vcl: make MapMode constructor explicitMichael Stahl
Insert constructor everywhere, except a couple places that apparently want to compare GetMapUnit(). Change-Id: I1910deb60562e5e949203435e827057f70a3f988
2017-10-26ofz#3820 Integer-overflowCaolán McNamara
Change-Id: I620b0278273e285100fe44d252389e6393a2022c Reviewed-on: https://gerrit.libreoffice.org/43877 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-26ofz#3793 Divide-by-zeroCaolán McNamara
Change-Id: Ica00209bc6679edaa0c83fc7d31bd5d4653b5e96 Reviewed-on: https://gerrit.libreoffice.org/43879 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-26More loplugin:unnecessaryparenStephan Bergmann
81892b2037453108b9bde1512a500cf3b2ce438a "loplugin:unnecessaryparen when compiling as C++17, so the ParenExpr is no longer hidden behind ExprWithCleanups/CXXConstructExpr/MaterializedTemporaryExpr wrappers" gave me the idea to generally look though IgnoreImplicit instead of IngoreImpCasts in loplugin:unnecessaryparen. However, that would still not look through implicit CXXConstructExpr, so would still not have found the occurrences in 81892b2037453108b9bde1512a500cf3b2ce438a when compiling in pre-C++17 mode. Therefore, let ignoreAllImplicit also look through CXXConstructExpr. (I am not entirely sure in which situations non-implicit CXXConstructExpr---that should thus not be ignored---would occur, but assume they would be underneath something like a CXXFunctionalCastExpr, which is not ignored.) Change-Id: I947d08742e1809150ecc34a7abe84cca5e0ce843
2017-10-26Resolved a redundant assignmentKristóf Umann
Change-Id: I9d65f4b2a444a12c03f0ec009baeadc131045e98 Reviewed-on: https://gerrit.libreoffice.org/43832 Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com> Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2017-10-25no matching overload foundCaolán McNamara
Change-Id: I6d494a8a93ff2997aac5cb7ea57f3132dac469b7
2017-10-25ofz#3803 Integer-overflowCaolán McNamara
Change-Id: Id10ab04340449c990d384041dd6631ba84c5296f Reviewed-on: https://gerrit.libreoffice.org/43836 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-25ofz#3796 Integer-overflowCaolán McNamara
Change-Id: I9603106dfccf8acbe7f37c549bab66a6aab94fe5 Reviewed-on: https://gerrit.libreoffice.org/43826 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-25ofz#3790 Divide-by-zeroCaolán McNamara
Change-Id: Id6c1eb3dcf95e9cdc68c0898dc472aaac5d430d2 Reviewed-on: https://gerrit.libreoffice.org/43807 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-25ofz#3793 Divide-by-zeroCaolán McNamara
Change-Id: Idc18b194840c8d2646426a0346b49348707be5ad Reviewed-on: https://gerrit.libreoffice.org/43806 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-25ofz#3761 Undefined-shiftCaolán McNamara
Change-Id: Ibf45da2dd5aa351973b7227371e4f6c35d54021d Reviewed-on: https://gerrit.libreoffice.org/43764 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-25ofz#3755 Integer-overflowCaolán McNamara
Change-Id: If21e45ce711d483cdd27e96877ee7f3ca35c2f14 Reviewed-on: https://gerrit.libreoffice.org/43753 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-25ofz#3775 Divide-by-zeroCaolán McNamara
Change-Id: Idfbd2bdf10b5fcf54e1fc2a61dbfecabf7e75a6d Reviewed-on: https://gerrit.libreoffice.org/43784 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-25ofz#3759 check for valid starting dash indexCaolán McNamara
Change-Id: I09e117e14eda2565c9b25d407cc4328d4f2ee97a Reviewed-on: https://gerrit.libreoffice.org/43751 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-24ofz#3754 Divide-by-zeroCaolán McNamara
Change-Id: I2e8504dd67d2a7ad1e83a95d7be5a1d1086de5d5 Reviewed-on: https://gerrit.libreoffice.org/43758 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-24ofz#3744 Undefined-shiftCaolán McNamara
Change-Id: I4b3445c0ee50b9b50edba464da7ad61cda625d3e Reviewed-on: https://gerrit.libreoffice.org/43755 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-24ofz#3742 Integer-overflowCaolán McNamara
Change-Id: Ibb8441df04643dd52111342e6d2c2659033838bb Reviewed-on: https://gerrit.libreoffice.org/43741 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-24ofz#3741 Integer-overflowCaolán McNamara
Change-Id: I9d3743291c88ab52543077ac7e62dc3c5925bafd Reviewed-on: https://gerrit.libreoffice.org/43744 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-24ofz#3741 Integer-overflowCaolán McNamara
Change-Id: I5059eca94cde52696affbf40753b9db1061a8c2b Reviewed-on: https://gerrit.libreoffice.org/43740 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-23loplugin:includeform: filterStephan Bergmann
Change-Id: I1720c2ec2649577a9f552cb49edd93a82c8282a0
2017-10-23overload std::hash for OUString and OStringNoel Grandin
no need to explicitly specify it anymore Change-Id: I6ad9259cce77201fdd75152533f5151aae83e9ec Reviewed-on: https://gerrit.libreoffice.org/43567 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-10-21check seek succeededCaolán McNamara
Change-Id: I2bbee3adde8f36be7ebc6b4555d9417c3add00a3 Reviewed-on: https://gerrit.libreoffice.org/43669 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-21check for sucessful seekCaolán McNamara
Change-Id: I6da634ddd38f57eaef0eefca30f6d2c3867869c3 Reviewed-on: https://gerrit.libreoffice.org/43670 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2017-10-21check that seeks succeededCaolán McNamara
Change-Id: I4d27bf8287f9ba2945081537a6c839232df75c42 Reviewed-on: https://gerrit.libreoffice.org/43668 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>