- 10 Nis, 2019 12 kayıt (commit)
-
-
Michael Stahl yazdı
It turns out that the situation fixed in commit 1be0a3fa also applies to the definition of the styles themselves. To implement the same style import as Word, the style definitions need to be stored twice: once as read from the file, and another time with attributes defaulted and deduplicated vs. the parent style; the second representation is then sent to the domain mapper. To make this easier, add a bool parameter to cloneAndDeduplicate() to disable the implicit pPr dereferencing that happens when creating the hard formatted paragraph properties (this could potentially be cleaned up further if those paragraph properties would use pPr wrapper themselves). Also implement defaulting of line spacing in getDefaultSPRM(). Change-Id: I4810e917697b3af244e5dbdd7f5a45b4767c93fc Reviewed-on: https://gerrit.libreoffice.org/70320 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
-
Noel Grandin yazdı
Change-Id: I1c94090c4846e232bd6bb9c86a0c94b750ccbf5c Reviewed-on: https://gerrit.libreoffice.org/70499 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
-
Johnny_M yazdı
Change-Id: I3799788cd72b34d603f4bffc6820f5b656cbf427 Reviewed-on: https://gerrit.libreoffice.org/70471Reviewed-by: Michael Stahl <Michael.Stahl@cib.de> Tested-by: Michael Stahl <Michael.Stahl@cib.de>
-
Michael Stahl yazdı
Missed one call where the return value must be forwarded. (regression from 7d481f7a and 943d9be7) Change-Id: Ic405b29a1dce982bfdd81eeb5d678ceb79690bfc Reviewed-on: https://gerrit.libreoffice.org/70469 Tested-by: Jenkins Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
-
Michael Stahl yazdı
(regression from c180c944) Change-Id: Ie3c935ee5dd42187ca8ad2b28406b80e63c0d1e3 Reviewed-on: https://gerrit.libreoffice.org/70467Reviewed-by: Michael Stahl <Michael.Stahl@cib.de> Tested-by: Michael Stahl <Michael.Stahl@cib.de>
-
Gabor Kelemen yazdı
Found with bin/find-unneeded-includes Only removal proposals are dealt with here. There were some changes since last run and some omitted files Change-Id: I666ac8ed7d06684e252ca590e3d7d454e9e10975 Reviewed-on: https://gerrit.libreoffice.org/70497 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
-
Stephan Bergmann yazdı
Change-Id: I20ed4c7daa9cde48ba65f92e5b4cc7e00972fe81 Reviewed-on: https://gerrit.libreoffice.org/70477 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
-
Miklos Vajna yazdı
It's cleaner to avoid member functions and constructors when we inherit from a C struct and we pass it to pdfium's C API. Change-Id: Iac9ab99b3d1b66241bb49d3a7707e242078ab5df Reviewed-on: https://gerrit.libreoffice.org/70472 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
-
Grzegorz Araminowicz yazdı
now all transition arrows are created in cycle diagrams Change-Id: I69e932f0060786b702dbecae72245bb624fa602b Reviewed-on: https://gerrit.libreoffice.org/70457 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
-
Andrea Gelmini yazdı
Change-Id: Ida96d8a0a993daac87c4e03fe4ba0d22a69f90cb Reviewed-on: https://gerrit.libreoffice.org/70487 Tested-by: Jenkins Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
-
Andrea Gelmini yazdı
Change-Id: Id9d19965511760839e8e93488b1cc6a98fbd4a5c Reviewed-on: https://gerrit.libreoffice.org/70493 Tested-by: Jenkins Reviewed-by: himajin100000 <himajin100000@gmail.com> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
-
Andrea Gelmini yazdı
Change-Id: Ia0dd6a96a039056f15e494baf844e11fb9b5e9aa Reviewed-on: https://gerrit.libreoffice.org/70488 Tested-by: Jenkins Reviewed-by: himajin100000 <himajin100000@gmail.com> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
-
- 09 Nis, 2019 28 kayıt (commit)
-
-
Tamás Zolnai yazdı
Change-Id: I1cb16b180f4cc5bf4d65485f03c44a06414d3580 Reviewed-on: https://gerrit.libreoffice.org/70481 Tested-by: Jenkins Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
-
Mike Kaganski yazdı
Change-Id: I91f04cae91cbb5510f70baaf8c31ecc59f6402c7 Reviewed-on: https://gerrit.libreoffice.org/70479 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
-
andreas kainz yazdı
Change-Id: Ibdcae261ee19e95a36b094401c9b526c41e261ad Reviewed-on: https://gerrit.libreoffice.org/70484Reviewed-by: andreas_kainz <kainz.a@gmail.com> Tested-by: andreas_kainz <kainz.a@gmail.com>
-
andreas kainz yazdı
Change-Id: I0a3ba3beb671cae6956fa616308997fcc2bd3d1f Reviewed-on: https://gerrit.libreoffice.org/70476 Tested-by: Jenkins Reviewed-by: andreas_kainz <kainz.a@gmail.com>
-
Stephan Bergmann yazdı
After recent additions of 61c4f96d "Support AdoptOpenJDK" and 41507db5 "Support Amazon Corretto" to our hard-coded list, there is now reports that at least Debian and Ubuntu tried to distribute versions of OpenJDK with the java.vendor propety set to string like "Debian" or "Ubuntu". Instead of trying to catch up with an ever-growing hard-coded list, it is probably better to stop relying exclusively on such a hard-coded list, and for unknown vendor values, try out whether the SunInfo backend (which supports the "generic" OpenJDK) would be able to handle the given JRE. (For simplicity, assume that any versions of such JREs are supported. Our baseline is Java 6, and there are unlikely any older versions of JREs from unknown vendors out there. If this turns out to be problematic, we could include information about problematic vendors after all, or add a general check that JREs from unknown vendors are at least Java 6.) Many functions in jvmfwk/inc/vendorplugin.hxx that used to take a set of sVendor/sMinVersion/sMaxVerison/arExcludeList paramters had to be revised to take a vendorSettings parameter instead, and VendorSettings::getVersionInformation has been changed to return a boost::optional, so that unknown vendors can be handled gracefully. Change-Id: Ibf915f2ddd59e09b77e2c03be688cac0547b9ac9 Reviewed-on: https://gerrit.libreoffice.org/70460 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
-
Mike Kaganski yazdı
... which could help catch copy-paste errors when wrong guard is cleared second time. Also an assert added that when resetting, there's something to reset (i.e., no descendant class had cleared protected pResetT, making reset impossible, and thus actually unable to guard anything). framework/source/layoutmanager/layoutmanager.cxx: made sure to not call clear() second time framework/source/layoutmanager/toolbarlayoutmanager.cxx: restored lock lost in commit 777bc22c forms/source/misc/InterfaceContainer.cxx: removed a leftover from commit a19cd21e which reduced guarded scope forms/source/component/DatabaseForm.cxx: fixed clear-reset sequence broken from the initial commit bf4154eb Change-Id: Ibab6660c79561eee31faf3e6c1128ab141a7e8a3 Reviewed-on: https://gerrit.libreoffice.org/70381 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
-
Jan-Marek Glogowski yazdı
... as we can't drop them. IMHO it's still better then just a crash. Has to use NULL, as the API is pre-C++11 compatible. Change-Id: Idb4395d14a9185f4d7d1c2f048ad950518eef133 Reviewed-on: https://gerrit.libreoffice.org/70465Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Jenkins
-
andreas kainz yazdı
Change-Id: Icc7822c942236904b63c26984190312109cfc909 Reviewed-on: https://gerrit.libreoffice.org/70462 Tested-by: Jenkins Reviewed-by: andreas_kainz <kainz.a@gmail.com>
-
Christian Lohmaier yazdı
Change-Id: Ib1d4c660ff8578d86dc2dddb1aeb55b95f127cb8
-
Johnny_M yazdı
Please feel free to suggest a better name, if desired Change-Id: I9ddf5007930b90058d448c81ae5733cbc7192d1a Reviewed-on: https://gerrit.libreoffice.org/70374 Tested-by: Jenkins Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
-
Johnny_M yazdı
Please feel free to suggest a better name, if desired Change-Id: I31fd05dbd2e7c86c1aab6967c630c13bd2da455f Reviewed-on: https://gerrit.libreoffice.org/70373 Tested-by: Jenkins Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
-
Johnny_M yazdı
Please feel free to suggest a better name, if desired Change-Id: Ia5b7b6100fc588ba366ed31e249e35ce501e0d1a Reviewed-on: https://gerrit.libreoffice.org/70372 Tested-by: Jenkins Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
-
Johnny_M yazdı
Please feel free to suggest a better name, if desired Change-Id: I1bd5805c206547a5f70d4d5186ecec2e624845fe Reviewed-on: https://gerrit.libreoffice.org/70364 Tested-by: Jenkins Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
-
Jan-Marek Glogowski yazdı
... instead of the nested MutexGuard and clear. Really had to look twice, that the MutexGuard and ResettableGuard use the same Mutex - who came up with this... yup myself. Change-Id: I70326772a9be8352f169a26467529a7c4100a20d Reviewed-on: https://gerrit.libreoffice.org/70385 Tested-by: Jenkins Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
-
Caolán McNamara yazdı
Change-Id: I4ba1a3c467526b85f576e5549e7c47a1e0fb66a0 Reviewed-on: https://gerrit.libreoffice.org/70456 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-
Gabor Kelemen yazdı
So it gives no false positives as currently seen in e.g. include/sfx2/charmapcontrol.hxx include/sfx2/docinsert.hxx Change-Id: I087a949875df20c4ef25a10c80571bb57334cf80 Reviewed-on: https://gerrit.libreoffice.org/70400 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
-
Caolán McNamara yazdı
Change-Id: I6a748d76bb24661404d280236f4c850a9136bb30 Reviewed-on: https://gerrit.libreoffice.org/70430 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-
Caolán McNamara yazdı
Change-Id: I8e41e28313a22148033c566edce50f5bfa58d065 Reviewed-on: https://gerrit.libreoffice.org/70452 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-
Tomaž Vajngerl yazdı
You get BitmapColor only from reading pixels from a Bitmap and we want to avoid usage of Bitmap outside of VCL (and use BitmapEx as the alternative which will eventually replace Bitmap). Change-Id: Iddfa3ef739bfdd4dce5fb47fd9f67a5a36f3388b Reviewed-on: https://gerrit.libreoffice.org/70447 Tested-by: Jenkins Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
-
Caolán McNamara yazdı
Change-Id: I38302045bfe096781ce9c59c6f2dbe0a7625efda Reviewed-on: https://gerrit.libreoffice.org/70453 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-
andreas kainz yazdı
Change-Id: I00ae42334935a807ebbe6855323e031441e191d1 Reviewed-on: https://gerrit.libreoffice.org/70450 Tested-by: Jenkins Reviewed-by: andreas_kainz <kainz.a@gmail.com>
-
Caolán McNamara yazdı
Change-Id: I7f200543a77d9bbf2f892498bca8cbecdf9628a1 Reviewed-on: https://gerrit.libreoffice.org/70429 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-
Caolán McNamara yazdı
Change-Id: I425cdf3d868b37a4975f502f3cade4298f3a948a Reviewed-on: https://gerrit.libreoffice.org/70428 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-
Caolán McNamara yazdı
Change-Id: I6c1af0332d9c34dbe22d6ee1325513824f95fb05 Reviewed-on: https://gerrit.libreoffice.org/70427 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-
Caolán McNamara yazdı
Change-Id: I0f442efa9e69923fc412a5c1b737dfbc7662c6a5 Reviewed-on: https://gerrit.libreoffice.org/70426 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-
Stephan Bergmann yazdı
Change-Id: I637d86ca3bfa0a526c4d02e991c8460f553f491f
-
Miklos Vajna yazdı
Only add the correct type to new signatures to avoid breaking the hash of old ones. Change-Id: I30f892b292f84a0575a3d4ef5ccf3eddbe0090ca Reviewed-on: https://gerrit.libreoffice.org/70424 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
-
Alexander Farrow yazdı
GlyphItem::m_aLinearPos is a tools::Point variable. Eventually it will be changed to a basegfx::B2DPoint variable - to add support for sub pixel glyph positioning. basegfx::B2DPoint has the getX() method to retrieve a point's X coordinate. tools::Point has two methods to retrieve a point's X coordinate: 1. X() 2. getX() Both of these methods return the same value because getX() just calls X(). So any call to X() can replaced by getX() and vice versa. By replacing calls to X() with getX() now we won't have to make these changes later when we swap from tools::Point to basegfx::B2DPoint. Change-Id: Idfa0584cb3a7ce246b5499814b7e29f86167d8b8 Reviewed-on: https://gerrit.libreoffice.org/70436 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
-