Age | Commit message (Collapse) | Author |
|
Change-Id: I0066b2de769a412cb04bcec7cdd8401f42b9dcf3
Reviewed-on: https://gerrit.libreoffice.org/48972
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: I7f98fa855ff521db42bf9b379bfac3d430791ae0
Reviewed-on: https://gerrit.libreoffice.org/48968
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: I63e12a22a46de27a8bd010e4d21bf36d3a7e9178
Reviewed-on: https://gerrit.libreoffice.org/48967
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: I2cf424ecfa2748cfca0f85bab8da820c6286d331
Reviewed-on: https://gerrit.libreoffice.org/49061
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>
|
|
Change-Id: I877e1c9b33ae240a2e8ac9972a6bc45e0848ee28
Reviewed-on: https://gerrit.libreoffice.org/49027
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
|
|
Refactor the copy-pasted code in StartRuby of WW8AttributeOutput,
RtfAttributeOutput, so that it can be used it in DocxAttributeOutput
later.
Change-Id: I63631be7604a62899c5630f3e57985993f128728
Reviewed-on: https://gerrit.libreoffice.org/48764
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mark Hung <marklh9@gmail.com>
|
|
Allow RubyPosition property to be set via UNO
interface.
Change-Id: I506312b1b5b27f016d0ee1c66231ff90747e5538
Reviewed-on: https://gerrit.libreoffice.org/48327
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mark Hung <marklh9@gmail.com>
|
|
Change-Id: I9f7170023dd31c11ef6e02a6901bb5ea18f5fa81
|
|
commit 5ceea51947b99f27eb3401bd3c9b8a7cf6ee9c96
Author: Caolán McNamara <caolanm@redhat.com>
Date: Fri Dec 8 14:13:22 2017 +0000
ofz: fix mem leak
Change-Id: I295e52723297b257dcfe532108471702631ca5f5
Reviewed-on: https://gerrit.libreoffice.org/46093
Change-Id: I98c2dc7e32c6dfc725a5763077bceb527cb31db2
|
|
Change-Id: I22553d7cbbae7cfaac3f39a17db54dfe06e2e5bc
Reviewed-on: https://gerrit.libreoffice.org/48989
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: Ib13f198e5941a33a9a8d1091c8ceb48e273f00cc
Reviewed-on: https://gerrit.libreoffice.org/48949
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: Id575db58199f438e95d9c73778cc520c452e1a42
Reviewed-on: https://gerrit.libreoffice.org/48948
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: I9e157f7b27eab831c5c15f6c22b9ffb20207b593
Reviewed-on: https://gerrit.libreoffice.org/48947
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: I6274d563f9bb4f1f173d0d5d25239e5617ee064c
Reviewed-on: https://gerrit.libreoffice.org/48946
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
|
|
Change-Id: I50f37e09df6c4c7cbe697e3419731e637c980807
Reviewed-on: https://gerrit.libreoffice.org/49000
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
The original code introduced with 1d10fb1197ea93f3532726c55ddd5a0eaa9ed944 "New:
Text adjustment in multiportions" read
+ if( nMultiSpace < KSHRT_MAX )
+ {
+ pCurr->GetpSpaceAdd()->Insert(KSHORT(nMultiSpace),0);
+ bRet = sal_True;
+ }
so needed nMulitSpace to fit into KSHORT (an alias for USHORT back then),
whereas today nMultiSpace is inserted into a std::vector<long>.
Change-Id: I24cafa7e295b87bf93f4e0ef5fbda5f99834286a
Reviewed-on: https://gerrit.libreoffice.org/49034
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: Id95c3d528ac3f50653c69778ca0b497662669f8e
Reviewed-on: https://gerrit.libreoffice.org/48938
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>
|
|
Change-Id: Ic72762c030878158d505306cbdd9c50553feecfd
Reviewed-on: https://gerrit.libreoffice.org/49032
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>
|
|
Same as in sc/source/filter/lotus/lotform.cxx LotusToSc::DoFunc()
Change-Id: I4972e065ab96abdea42d64481d4e30674230ab99
|
|
Change-Id: I994653dc4bf858bdea7ba60f40da0ee3c8d028e4
|
|
Change-Id: I829e0454a954e9870ebf2b86b43dc088ea74551c
|
|
Change-Id: Ibd370623f89341560d5d3a6c3bd073cc7365fab9
|
|
Complete commit 8310642c6c404ea501b8be5ae4d12860e1562ac9
It passed "make check"
Change-Id: If599bf7b66c972f225f88802d10c70bea4783a38
Reviewed-on: https://gerrit.libreoffice.org/48998
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: I1e78c05cc78c03c1b5578342f7480405f6992c68
Reviewed-on: https://gerrit.libreoffice.org/48999
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: I7dddc597b1a5c3f4572be4d3f91d9ebd83181a4d
Reviewed-on: https://gerrit.libreoffice.org/49030
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
|
|
by using for range loops
Change-Id: I52d6e6c9e1c2c321dc81d8258943a1a9a611441c
Reviewed-on: https://gerrit.libreoffice.org/48987
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Project: help f7e82f12585648e59c308607d8754616c5d930b0
Fix typo
Change-Id: I8fd351833236fe594a0480d9e09b0ef072ec7c46
Reviewed-on: https://gerrit.libreoffice.org/48997
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
|
|
Project: help 11adc4492b11253c5d24d217354581956da1f52b
Fixes for MS browsers and tweaks to fuzzy search
Some experimental JS tech had slipped into the mix.
Search settings should also be a bit more performant now.
Change-Id: Ifae9986efc02d20fefa9480ae463107fe4743944
Reviewed-on: https://gerrit.libreoffice.org/48988
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
Tested-by: Olivier Hallot <olivier.hallot@libreoffice.org>
|
|
Change-Id: I45d933ef308fbad31695cfc9f668eced4cd8e1f1
Reviewed-on: https://gerrit.libreoffice.org/48872
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
|
|
PPTNumberFOrmatCreator's nTextOfs and nBulletOfs are both sal_uInt32, while
SvxNumberFormat's SetAbsLSpace and SetFirstLineOffset both take short parameters
Change-Id: I2512fad4387bc8393be694b67be03ac8ea543cce
Reviewed-on: https://gerrit.libreoffice.org/48954
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I73c9ee7aef7062e3cb025c6ae0f3bb6a91485330
Reviewed-on: https://gerrit.libreoffice.org/48983
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
|
|
OBookmarkContainer actually re-uses the reference count of
ODatabaseSource, so converting ODatabaseSource::m_aBookmark to Reference
created a cycle, which somehow causes the dataloss, because evidently
something as important as storing the data must be done in the
destructor.
(regression from 2660d24a07866e083c5135ea263030f3e3a2e729)
Change-Id: I4cad01dc9cdaf405c1eb31d6c0e161eb6712b78f
|
|
Change-Id: I903362c158190a9f1ee68c2e580a0c695a4c86d8
Reviewed-on: https://gerrit.libreoffice.org/48970
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Look for a whole column at once so we don't have to iterate all draw objects
for every cell, but only once per column.
Follow-up for 3a2a430ae8e2c1647c18d8904477949f6e2e7941
Change-Id: Ic8740fca7d595528785b432c1cedf4fad4f13ba1
Reviewed-on: https://gerrit.libreoffice.org/48416
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
|
|
...so there appears to be no good reason to cast from sal_Int32 to sal_uInt16 first
Change-Id: I67384bc736685269acb6c2033b2c4edffd27a510
Reviewed-on: https://gerrit.libreoffice.org/48953
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...implicitly casting it to smaller types where necessary, instead of the other
way around
Change-Id: Ib7673fbae016b0c91bf37d6768692fa99ccae46c
Reviewed-on: https://gerrit.libreoffice.org/48955
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...so there appears to be no good reason to cast from long etc. to sal_uInt16 first
Change-Id: I870e63404bed09cd42673692e08fc12c1c1e2fdf
Reviewed-on: https://gerrit.libreoffice.org/48958
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...and none of the places that call SvxNumberFormat::GetAbsLSpace (also
returning short) look like they deliberately cast back to sal_uInt16 (i.e., to
tunnel a genuinely unsigned value through SvxNumberFormat), so it looks more
plausible to restrict values here to short and SHRT_MAX, instead of sal_uInt16
and USHRT_MAX.
Change-Id: I6807c1b29d30c3f9bfe1ad8b98d0b3ef64dd6a5a
Reviewed-on: https://gerrit.libreoffice.org/48957
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
...(and both calls of CorrectTabStopInSet pass in the result of a GetAbsLSpace
call), and SvxTabStop::GetTabPos() is sal_Int32& (and that's what nAbsLeft is
combined with inside CorrectTabStopInSet), so why use an intermediary parameter
of type sal_uInt16?
Change-Id: I1cd1869ead58c419eecb3e2d1fbfb87f96bca86f
Reviewed-on: https://gerrit.libreoffice.org/48956
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Width reduced to 75px
Change-Id: I269eed9054e75d3885cb53d6a773460d30c67a14
Reviewed-on: https://gerrit.libreoffice.org/48896
Reviewed-by: Heiko Tietze <tietze.heiko@gmail.com>
Tested-by: Heiko Tietze <tietze.heiko@gmail.com>
|
|
Change-Id: I26461962c52c7b77308dbf3a7734b6ca9fba6e82
Reviewed-on: https://gerrit.libreoffice.org/48951
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I363ad44545adae11fb4d74048d906eb6fdd8e819
|
|
...so there appears to be no good reason to cast from long to sal_uInt16 first
Change-Id: I5baf699fbf043eae6a20e2055b53aef10103f591
Reviewed-on: https://gerrit.libreoffice.org/48952
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I3f3ebaea1fe3a3b3919fe204bd8769287e84deaf
|
|
Change-Id: Ie44698565865b3aacc1ebd0a3e41d5001bf8e09a
|
|
Change-Id: Iebcf12eec5cf5282e798ff5d4fe6649e3a8eea3f
|
|
Change-Id: I8bd18130c5fe87b2b352d285d58dd6a4f4499768
|
|
Change-Id: I85a096505b0de98fd1de6179b9239318622fc34a
Reviewed-on: https://gerrit.libreoffice.org/48933
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
|
|
by using for range loops
Remark: some iterators were used outside loops,
these have been replaced by the counterpart origin variable
Change-Id: I0b1e69811f16752b0962f5c5e662ea20064e26b4
Reviewed-on: https://gerrit.libreoffice.org/48829
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
|
|
Change-Id: If47716c6dbc353059c42a2bcd09f7c11ed53796f
Reviewed-on: https://gerrit.libreoffice.org/48937
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jens Carl <j.carl43@gmx.de>
|