- 09 Tem, 2017 1 kayıt (commit)
-
-
Sean Stangl yazdı
Change-Id: I766fedb34737a1a7815693bf496fa08c08f492b1 Reviewed-on: https://gerrit.libreoffice.org/39719Reviewed-by:
Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by:
Markus Mohrhard <markus.mohrhard@googlemail.com>
-
- 08 Tem, 2017 8 kayıt (commit)
-
-
Arnaud Versini yazdı
Also simplify this class. Change-Id: I7abccd19287101918a6d9b4636ce2faecf169123 Reviewed-on: https://gerrit.libreoffice.org/39446Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Noel Grandin <noel.grandin@collabora.co.uk>
-
Jaskaran Singh yazdı
Change-Id: I6e0e1a7f58873c012e0e8015ed69da38a32a44f4 Reviewed-on: https://gerrit.libreoffice.org/39323Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Markus Mohrhard <markus.mohrhard@googlemail.com>
-
Jens Carl yazdı
Change-Id: I5a3d64fd255fb8fd1f1caba572bc00a5399a6519 Reviewed-on: https://gerrit.libreoffice.org/39550Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Markus Mohrhard <markus.mohrhard@googlemail.com>
-
Justin Luth yazdı
Word appears to use negative z-indexes to indicate behind-text zordering and positive numbers for in-front placement. This was added for shapes in LO4.3, now applying to textboxes also. Change-Id: I3b06fb231329f151ca978f3a68b4d4e89bc28515 Reviewed-on: https://gerrit.libreoffice.org/39671Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Justin Luth <justin_luth@sil.org>
-
Tamás Zolnai yazdı
Change-Id: Ieb89a2e69ae44fe1cfbe99e843239cf25e97463a Reviewed-on: https://gerrit.libreoffice.org/39712Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Tamás Zolnai <tamas.zolnai@collabora.com>
-
Szymon Kłos yazdı
Change-Id: If1dd46643dc2ae9cc74ba94038609ae3445a416c Reviewed-on: https://gerrit.libreoffice.org/39706Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Szymon Kłos <szymon.klos@collabora.com>
-
Szymon Kłos yazdı
If DrawAspect is equal "Icon", show an icon not document preview Document is opened in the separate window, not in-place. Change-Id: I3a8d81e7340b29d247f8ac440c06b0420bb65644 Reviewed-on: https://gerrit.libreoffice.org/39440Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Szymon Kłos <szymon.klos@collabora.com>
-
Chris Sherlock yazdı
As pointed out by sberg, either the issue is a programming error that needs an assert, or the issue is unusual and should just generated a warning, but never both. Change-Id: I07ae1e99597cd6c342791aa268b2240db987ae91 Reviewed-on: https://gerrit.libreoffice.org/39514Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Chris Sherlock <chris.sherlock79@gmail.com>
-
- 07 Tem, 2017 31 kayıt (commit)
-
-
Julien Nabet yazdı
See http://dbase.free.fr/tlcharge/structure%20tables.pdf Change-Id: I68df30f73d2556a217f30d80d1d55ec93bca1564 Reviewed-on: https://gerrit.libreoffice.org/39710Reviewed-by:
Julien Nabet <serval2412@yahoo.fr> Tested-by:
Julien Nabet <serval2412@yahoo.fr>
-
Arnaud Versini yazdı
Change-Id: Iabc7293370c7a8b3fdccb00d6e1cdbfac4c46992 Reviewed-on: https://gerrit.libreoffice.org/39445Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Noel Grandin <noel.grandin@collabora.co.uk>
-
Jochen Nitschke yazdı
results are in the range of first_iterator to last_iterator. If one of those is nullptr the algorithm would fail anyway. This removes some impossible checks in sw/source/core/unocore/unochart.cxx: SwChartDataProvider::detectArguments. A sorted range still holds the same values and has the same length as the original range. Replacing raw pointers eases reading this code. Change-Id: If96bd11e9167488346a57e9e08507ac42338d3cd Reviewed-on: https://gerrit.libreoffice.org/39683Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Noel Grandin <noel.grandin@collabora.co.uk>
-
Chris Sherlock yazdı
Change-Id: I99e3f1e1fe2f51c6c3891a6e508b468ec068cae0 Reviewed-on: https://gerrit.libreoffice.org/39539Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Michael Meeks <michael.meeks@collabora.com>
-
Chris Sherlock yazdı
Whitespacing made consistent in function, and switch to more conventional nullptr testing style. Change-Id: I70921e9f79e8e0ebf4a74b84f73313aed033ec5b Reviewed-on: https://gerrit.libreoffice.org/39538Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Michael Stahl <mstahl@redhat.com>
-
Chris Sherlock yazdı
I could not easily tell on first reading this function if Directory was a pointer or not (it is), so changed name to pDirectory). The signature was too long, so fixed this, whitespacing was inconsistent so small change there. Also converted to more contentional testing of nullptr. Change-Id: Ia8b1d33a4a4fbe3e050d63116997ef57a28d73b3 Reviewed-on: https://gerrit.libreoffice.org/39537Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Michael Stahl <mstahl@redhat.com>
-
Chris Sherlock yazdı
When I was reading this code, it wasn't entirely clear to me without looking at the typedef of oslDirectory whether I was dealing with a variable or a pointer, so I have changed this to pDirectory. I also made a small tweak to the whitespace to help readability. I also changed from explicit comparison to nullptr to the more conventional style. Change-Id: I4e9a69575733ab71a175d14a30c1976e6771ed5b Reviewed-on: https://gerrit.libreoffice.org/39536Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Michael Stahl <mstahl@redhat.com>
-
Mohammed Abdul Azeem yazdı
This should fix the issue, as well as restrict available to return only non-negative values. Change-Id: I198e226e945b9bd79dec32b1686c20e2a8dfaf3e Reviewed-on: https://gerrit.libreoffice.org/39665Reviewed-by:
Michael Meeks <michael.meeks@collabora.com> Tested-by:
Jenkins <ci@libreoffice.org>
-
Tamás Zolnai yazdı
Add a constraint for bullet relative size to avoid too small bullets which are hardly recognizable. Use the same 25% lower limit what MSO uses. Change-Id: Id956ecf3ec831c569188b944be58da03bf30a55e Reviewed-on: https://gerrit.libreoffice.org/39696Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Tamás Zolnai <tamas.zolnai@collabora.com>
-
Varun Dhall yazdı
Change-Id: If81d55aef96af4b764b62dc53a1986b5f0739d1e Reviewed-on: https://gerrit.libreoffice.org/39698Reviewed-by:
Michael Stahl <mstahl@redhat.com> Tested-by:
Jenkins <ci@libreoffice.org>
-
Miklos Vajna yazdı
Change-Id: Ia022f6cb1e9c91c9aa49025e54699a5c7718fc75
-
Miklos Vajna yazdı
Commit b5e0a143 (tdf#108524 sw: attempt to split section frames inside table cells, take two, 2017-07-03) checked for tables in SwFrame::GetNextSctLeaf() when it considered looking up the next "follow" cell frame. But this is too general, in practice it is only necessary to look for follow cell frames in case the frame in question is in a table, but not in a table-in-section. This at the same time avoids a crash with tables inside nested sections, as it happens in the bugdoc. Change-Id: If648cb477be5492c7158f89934435ca7021a6a63 Reviewed-on: https://gerrit.libreoffice.org/39692Reviewed-by:
Miklos Vajna <vmiklos@collabora.co.uk> Tested-by:
Jenkins <ci@libreoffice.org>
-
Michael Stahl yazdı
No longer need to compile with dbglevel=3 since 4dfa3161 Change-Id: I2dbc7ccd07abd191701c93534241ffd4dfef9df1
-
Michael Stahl yazdı
Half of these were around SAL_INFO which does its own checking of OSL_DEBUG_LEVEL anyway, and the other half can check at runtime the g_bDebugDisableCompression variable like PDFWriterImpl::MARK() already does since commit 49968942. Change-Id: Ic688eb643bde9253375e14e2e2233ece8dc7c18b
-
Caolán McNamara yazdı
Change-Id: Ib5bdd594efd41eb881dfc4e2454b72e4739ffd56 Reviewed-on: https://gerrit.libreoffice.org/39693Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com>
-
Michael Stahl yazdı
Change-Id: Ica69cd4527f36edd7480d897a28ff814ecbe97a9
-
Michael Stahl yazdı
SwFltControlStack::SetAttrInDoc() will insert a RES_FLTR_ANNOTATIONMARK without checking that there is actually a SwPostItField at that position, which triggers the assertion "<AnnotationMark::InitDoc(..)> - annotation field missing!" in AnnotationMark::InitDoc(). This happens when loading "crash-7.doc" which has a different kind of field - a SwPageNameField - at that position. In the WW8 filter, the SwPostItField is inserted in Read_And(), the AnnotationMark in Read_AtnBook(), which look quite orthogonal. Since it doesn't look like a valid document model to have the AnnotationMark without its field, avoid inserting it in the filter. Change-Id: I960d346601522b1a4d8644e538cde174763e99ff
-
Noel Grandin yazdı
After commit c6902761 "Revert "use std::vector in BigPtrArray"" <mst_> noelgrandin: you reverted both gdb changes but only one sw change? <noelgrandin> mst_, I thought your gdb change only appliled to the std::vector? <mst_> noelgrandin: there was another std::something or other change there, one was mvInfo and the other mvData <noelgrandin> mst_, sigh, will fix that Change-Id: I3059d29b1a559eb3077f5d713be011e15e887ee0
-
Noel Grandin yazdı
no need to do c_str() everywhere, when we can just work with OString's Change-Id: I4c9e80200c7c4d36abc541ab47b3f3cf1b42e2fb
-
Noel Grandin yazdı
which is causing crashes in the crashtesting in ooo119635-3.docx and ooo119568-2.docx It is definitely some kind of use-after-free error, but the compress and delete logic for BigPtrArray is too hairy for me to debug right now. This reverts commit 1eee0abd. Also revert commit 4f743419 "gdb pretty-printers: fix BigPtrArrayPrinter after recent std::isation" Change-Id: Id870876432a060f9347aafb43bf0df692ea24464 Reviewed-on: https://gerrit.libreoffice.org/39684Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Noel Grandin <noel.grandin@collabora.co.uk>
-
Caolán McNamara yazdı
Change-Id: I25d4efe816bf6ae4ba456bb42a827b9a70f503c7 Reviewed-on: https://gerrit.libreoffice.org/39689Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com>
-
Caolán McNamara yazdı
Change-Id: I2525123dfe2c2dedcbf808f5d2f8e054e4c4bdf1 Reviewed-on: https://gerrit.libreoffice.org/39690Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com>
-
Michael Stahl yazdı
Change-Id: I86df59e4b77d2ddc479144f32342113a6e29a725
-
Troy yazdı
Required by dpkg --build on Debian squeeze. Change-Id: I5e0e9c8f7851555a277d21aa14f9d695bad12198 Reviewed-on: https://gerrit.libreoffice.org/38765Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Michael Stahl <mstahl@redhat.com>
-
Mike Kaganski yazdı
See paragraph 2.10 of XML 1.0 specification and 17.3.3.31 of ECMA-376-1:2016 Change-Id: I7f19d3b9cf2ccce88a5fa03022beeb99facc04fe Reviewed-on: https://gerrit.libreoffice.org/39682Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Mike Kaganski <mike.kaganski@collabora.com>
-
Marco A.G.Pinto yazdı
Change-Id: I05e1e3b87478767a233caf38640b5e85f0fe751f Reviewed-on: https://gerrit.libreoffice.org/39583Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Adolfo Jayme Barrientos <fitojb@ubuntu.com>
-
Takeshi Abe yazdı
Change-Id: Ic8515d04980ee627f964821547fb25d52240e933 Reviewed-on: https://gerrit.libreoffice.org/39681Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Takeshi Abe <tabe@fixedpoint.jp>
-
Dennis Francis yazdı
Change-Id: I087c47084347c28f1af7881129de1a7b01960715 Reviewed-on: https://gerrit.libreoffice.org/39678Reviewed-by:
Michael Stahl <mstahl@redhat.com> Tested-by:
Michael Stahl <mstahl@redhat.com>
-
Caolán McNamara yazdı
maybe this will clear coverity#1413007 Out-of-bounds access Change-Id: I26a0d83df543c1e9dab98b0f1219dadef5c8502d Reviewed-on: https://gerrit.libreoffice.org/39679Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Caolán McNamara <caolanm@redhat.com> Tested-by:
Caolán McNamara <caolanm@redhat.com>
-
Justin Luth yazdı
In LO4.3, commit 2496eaa5 forced every shape to be "in background". That is not correct. Word appears to use negative z-indexes to indicate behind-text zordering and positive numbers for in-front placement. Change-Id: I4083ae67ef76152e1c0a894d810950d249ac13b4 Reviewed-on: https://gerrit.libreoffice.org/39670Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Justin Luth <justin_luth@sil.org> Reviewed-by:
Miklos Vajna <vmiklos@collabora.co.uk>
-
Noel Grandin yazdı
stick to single-arg function calls, sometimes parens in multi-arg calls might be there for clarity Change-Id: Ib80190c571ce65b5d219a88056687042de749e74 Reviewed-on: https://gerrit.libreoffice.org/39676Tested-by:
Jenkins <ci@libreoffice.org> Reviewed-by:
Noel Grandin <noel.grandin@collabora.co.uk>
-