Date: prev next · Thread: first prev next last
2010 Archives by date, by thread · List index


I've pushed the branch 'feature/vos-removal' for review
I have rebased against master Saturday morning GMT, with relatively
few conflict.

The resulting build passed a trivial 'smoketest' (openining a couple
of spreadsheet, text document ...)

I have NOT built against Win or MacOS... there are almost certainly
some oops for these platforms.
I'll appreciate very a much feedback from person that can build on
these targets.

There is still a little bit of vos left.... but the end is near.

For reviewers:
commit marked as 'merge of vosremoval-xxx' are meant to be
more-or-less a transcription of the original patch in dev300 with the
same name.
Most of these patches did not apply very well, actually for many of
them they mis-apply so badly that I just used the original patch
as a guide, without even bothering with trying to apply.

Along the way I did some re factoring around Guard on the SolarMutex.

Few modules had there own variation of a Guard class for the
SolarMutex, some other used directly

vos:OGuard aGuard(Application::GetSolarMutex())

Now for these use-pattern I mostly use

::SolarMutexGuard aGuard


Below is the relevant git-log

n_th@tpa10 ~/git/oo/build/clone/libs-gui (feature/vos-removal)$ g log
master...HEAD
===== artwork =====
===== base =====
commit 042d9be3f340fb68407edc23eda8275da730e9bd
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:13:32 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually

commit e10b3f985f9ecc9043ba0b3829a178931bae66d3
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Fri Oct 15 12:22:54 2010 -0500

   merge vosremoval-reference.diff

commit 11628553d86d9223f2d56a0a420627c529da309a
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 22:00:34 2010 -0500

   repl OClearableGuard(SolarMutex) by SolarMutexGuard as none needed clear()

   Four site used a Clearbale Guard on the SolarMutex, but non of these site
   actually needed to clear the Guard, so SolarMutexGuard fit nicely

commit d502d4f6a6e08e72b1a8f22b55b23c74354f653b
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Wed Oct 13 01:38:41 2010 -0500

   use SolarMutexGuard to guard the SolarMutex
===== bootstrap =====
commit 09243b571ef85a77ce5f76eed4b9f5d7cea16d61
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:14:13 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually
===== calc =====
commit cfe3128ea5689d1a1539f50994c98e324f8de56d
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:15:13 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually

commit f5aae02fc75814fe8f54345adc06276bd00d4c21
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Fri Oct 15 12:20:01 2010 -0500

   merge vosremoval-reference.diff

commit ccb5d7fcc1521348bd1d42a35b85a31400379e11
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 22:22:13 2010 -0500

   Use a SolarMutexGuard rather that direct acquire/release on the SolarMutex

commit 548db1dced20c99d75611e25c4b5729766b41eb1
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 22:20:56 2010 -0500

   Replace a suspicious Guard on a solar mutex.

   This site use a OClearableGuard that is being converted to a SolarMutexGuard
   Guard will do... What is suspicious is the point where it is cleared.
   It seems that it is way to early to serve its declared purpose.

commit 52d14efd1cf18656dcddec5e6468d4a934cb5e3c
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 22:18:02 2010 -0500

   replace ScUnoGuard by SolarMutexGuard

   ScUnoGaurd provide a Guard wrapper around Application::GetSolarMutex().
   This is now provided by the class SolarMutexGuard.

commit c382d82381cbf69cc930f05321278b483abc53a3
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Wed Oct 13 01:41:46 2010 -0500

   use SolarMutexGuard to guard the SolarMutex
===== components =====
commit c439041d0ed166fc586b140284aa54c73a06bce6
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 04:30:41 2010 -0500

   migrate vos::OModule to osl:Module

commit f8802b92d4c10092f23ba19fce17306fb84fff54
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:15:49 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually

commit fb5f1c44fc068f86b44ed23cdc05be5a83a4c916
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Fri Oct 15 12:18:51 2010 -0500

   merge vosremoval-reference.diff

commit 3eac7d8aa940fef1234e3ae6bd43cadda6fbc4af
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 22:03:58 2010 -0500

   migrate use of OClearableGuard(SolarMutex) to SolarMutexGuard

   3 helper class and a site were using OClearableGuard on the SolarMutex
   None of these actually needed the Clearable feature, so
   they were replaced by a simple SolarMutexGuard.

commit 755807090d1a5e6d679763891034d1856e62bf6b
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 14:00:57 2010 -0500

   remove the class TLBSolarMutex and use SolarMutexGuard instead

   The class TBLSolarMutex is a wrapper guard arounf the SolarMutex.
   the class SolarMutexGuard already provide this function.

commit 29b75b7e833f545e8a680ff5a8dab90143e4cd8f
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 13:46:51 2010 -0500

   remove the class ALBSolarGuard and replace it's use with SolarMutexGuard

   ALBSolarGuard is awrapper guard around the SolarMutex
   SolarMutexGuard provide this function now.

commit 107852407569f15472b4f5eb59b5739c5fb04b67
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 13:29:19 2010 -0500

   replace BBSolarGuard with SolarMutexGuard

   BBSloarGuard iw a simple guard wrapper around the SolarMutex.
   the generic SolarMutexGuard provide this feature.

commit 5105c45dc3bdbcc6130d49f3e2bcac3eb38b2121
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 13:27:27 2010 -0500

   forgot base clase init call

commit 8701db99d0a4f4762bfa6e5063ed68759620fe57
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 08:35:22 2010 -0500

   replace the now superfluous TCSolarGuard with SolarMutexGuard

   TCSolarGuard is a class that basically wrapped a Guard around
Application::GetSolarMutex()
   This function is now provided directly by the SolarMutexGuard class.

commit 5be931acf48e436bf3cea239347e0de645b7e35e
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Wed Oct 13 04:35:41 2010 -0500

   use SolarMutexGuard to guard the SolarMutex

commit 1509f022211cda3e1b84c3183fef7a29389912f6
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 22:07:16 2010 -0500

   merge vosremoval-pipe.diff

commit 736414e3fe3b850a86de211a4eaa4ca026d2e186
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 20:57:51 2010 -0500

   merge vosremoval-timer.diff

commit 6162f3ea420f26a5e4910ea3b5bde1becce885e8
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 16:11:20 2010 -0500

   merge vosremoval-conditn patch
===== extensions =====
commit c999e24ca5dcdb6ce37ad7a13fb3bac493247859
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:41:25 2010 -0500

   replace a residual use of vos::TTimeValue by salhelper::TTimeValue

commit 84c7ae3cf74d5231045088042713b3868e2f9605
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 20:57:51 2010 -0500

   merge vosremoval-timer.diff
===== extras =====
===== filters =====
commit df3731fe7916a0e3ab58d38d8fce3b04e4b33178
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:49:49 2010 -0500

   migrate vos::OModule to osl:Module

commit a84d6c919edf2f4f1f42415a513e92a00b114831
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:16:59 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually

commit 0ba60084ca9f341d1235521cccc2e7116dcce2e0
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Fri Oct 15 12:12:33 2010 -0500

   merge vosremoval-reference.diff

commit 06b292c3e38b8df9f0137798175854ee335d720b
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 23:10:04 2010 -0500

   replace a pair acquire/release on the SolarMutex with a SolarMutexGuard

commit 2c6f8f314985cd9bdf065918f0c49c3287dfbe8c
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 14:59:39 2010 -0500

   Use SolarMutexGuard where appropriate

commit 5a5469675887504c55309fecdc03a02a338514da
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 14:55:48 2010 -0500

   Remove the class ScUnoGuard and replace with SolarMutexGuard

   ScUnoGuard is a guard wrapper around the SolarMutex.
   SolarMutexGuard provide that service.

commit 17083f8c8980ec0a0b90c5d0e2e864f6dc130967
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Wed Oct 13 04:34:50 2010 -0500

   use SolarMutexGuard to guard the SolarMutex

commit c03a665225860a4efc9ddaa5a582643a061c9fc7
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 00:53:55 2010 -0500

   merge vosremoval-security patch
===== help =====
===== impress =====
commit cd91f2bfe9c5d6f8e00f2b10ee870e6aacba0c3c
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:50:23 2010 -0500

   migrate vos::Omodule to osl:Module

commit aff382807498a07defc76d85cc76c9c63d821117
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:17:28 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually

commit 6c34db28e1c353e8f8d5b35eb2b9cc09ad7bd5a6
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Fri Oct 15 12:11:00 2010 -0500

   merge vosremoval-reference.diff

commit dec2a021b1d80e89e29cd39e858ab5b2d5ca8a67
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 23:10:46 2010 -0500

   replace a pair acquire/release on the SolarMutex by a SolarMutexGuard

commit 03a9f2c99dea609d7b35262dc0b63c62f6329382
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Wed Oct 13 02:37:50 2010 -0500

   use SolarMutexGuard to guard the SolarMutex
===== libs-core =====
commit 680553f274c62dc0217eb1520ee6554819ec3667
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:50:58 2010 -0500

   migrate vos::OModule to osl:Module

commit 47c4bb6b85f65e57b2a55f89129e28e062c8cb74
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:18:35 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually

commit cff87237eed267eceadfd7e627228f9ab17bcc17
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Fri Oct 15 12:10:06 2010 -0500

   merge vosremoval-reference.diff

commit 7ac0eb709e61579c0236ebaed9b0a17ebb993ab7
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 23:11:47 2010 -0500

   replace a OClareableGuard on the SolarMutex by a SolarClearableMutex

commit 6fbd04aaaf74a98a819665b181a7ea0be9e6ecdb
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 22:09:00 2010 -0500

   convert OClearableGuard(SolarMutex) to SolarMutex(Clearable)Guard.

   Most OClearableGuard of the Solar Mutex could be converted
   to a simple SolarMutexGuard.
   a couple of site were converted to a SolarMutexClearableGuard,
   were converting to a SolarMutexGuard would have made the code more
   convoluted.

commit 38fd87150a7a2dfb25d5bd5a3b6ae6bb16df2207
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 16:43:16 2010 -0500

   Use SolarMutexGuard where appropriate

commit c3af36248dde7be796c317a7c75417e950598f21
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Wed Oct 13 02:47:36 2010 -0500

   use SolarMutexGuard to guard the SolarMutex

commit 95669b854f6ebd1e25d1c132008d5832692a8da7
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 22:07:16 2010 -0500

   merge vosremoval-pipe.diff

commit 72aa147a6e94c477e7ab2dcca22c2960c30510a7
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 21:03:18 2010 -0500

   merge vosremoval-timer.diff

commit 0bd2b6e0f0afa1781388afc1069f7d74a15907c9
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 01:26:23 2010 -0500

   merge vosremoval-signal patch

commit 669d1cdc3ddae8a373ff7d579c07fa254deeede9
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 00:53:56 2010 -0500

   merge vosremoval-security patch
===== libs-extern =====
===== libs-extern-sys =====
===== libs-gui =====
commit f02552db50a2f65dee7208facc3605c99ff7014d
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 11:11:55 2010 -0500

   fix missing vos::Guard creeping back-up after rebase

commit 957eaa02c32ce7895b814bc44220a1c5a19250a0
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 04:45:26 2010 -0500

   fix a mishap during the OGuard migration for aqua.

commit 9c323ffb4463178d245ce2bf2011a13d9e80ab51
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:47:46 2010 -0500

   remove vos::OModule

commit a3042c0e6b3845a7e95f160921fdafb23ba8cf3b
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:20:00 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually

commit 728fa09fb6ca1499d4827065f842bb8ee834193c
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Fri Oct 15 11:54:41 2010 -0500

   merge vosremoval-reference.diff

commit 63ebf6d5ffa2aaf99e74dd6b55e7544319e41498
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 23:38:23 2010 -0500

   use SolarMutexGuard instead of LocalGuard in canvas

   LocalGuard is a wrapper guard around the SolarMutex.
   SolarMutexGuard already provide this function.

commit f44a1e21225635edcf488fb3735d066315d121c2
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 22:12:16 2010 -0500

   migrate OClearableGuard(SoalrMutex) to SolarMutexGuard

   Factorized the Guarded piece of code to locate the TopWindow under a
   given Point, which was repeated verbatim in 4 different functions
   That allowed as a bonus to remove the need for a ClearableGuard, and
   allow the use of a straight SolarMutexGuard.

commit 9467987a6bf0e16215003d617472c1720d41f1b5
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 17:40:58 2010 -0500

   Add a new class SolarMutexClearbleGuard

   Few place in the code use vos::OClearbleGuard
aGuard(Application::GetSolarMutex());
   or something similar.
   This new class hide this ugliness into
   SolarMutexClearableGuard aGuard;

   This also has the merit to push down the vos:: dependency down
into one location
   for this kind of access, which will simplify the vos dependency removal.

commit bd29afc34adf7b69b833248f4fd1bb7999a2d037
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Thu Oct 14 16:43:53 2010 -0500

   Use SolarMutexGuard where appropriate

commit ea3f302580964225b9e0fdcf3a4658acfda1bf70
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Wed Oct 13 01:47:23 2010 -0500

   use SolarMutexGuard to guard the SolarMutex

commit 291134b3244b496710af468300b36c225b8855dd
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Tue Oct 12 23:56:51 2010 -0500

   create a class SolarMutexGuard to take a Guard on the SolarMutex

   Most user of OGuard, use it to guard the SolarMutex.
   using something like
   vos::OGuard aGuard(Application::GetSolarMutex())

   Which will now be written
   vcl::SolarMutexGuard aGuard;

   This will also, as a side effect remove the use of vos::Guard in many
   sources, simplifying the migration out of ::vos

commit ce60fa6e8bd3bc3c7139e1d6460f10598b12506f
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 22:07:17 2010 -0500

   merge vosremoval-pipe.diff

commit 0e03d2c308e177dbf4bf85134edfc41e2c115561
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 20:57:53 2010 -0500

   merge vosremoval-timer.diff

commit 45618623601cf3c886bd3e1de54de4f3b05bff52
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 16:12:59 2010 -0500

   merge vosremoval-conditn patch

commit 95d8c1b2ca054251fff8c45dc04071e26ca02daf
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 01:26:24 2010 -0500

   merge vosremoval-signal patch

commit 324a27adbc03de8c2962ea000cec762d78670583
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 00:53:57 2010 -0500

   merge vosremoval-security patch
===== postprocess =====
===== sdk =====
===== testing =====
===== ure =====
commit 1b4a25d8cdd47da738aef341e8871472de9b379b
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:38:17 2010 -0500

   kill bonobo-related code in the codemaker

commit 746edc1909490335fec9a44db14d2b69c67bc52a
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:21:22 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually

commit f5ee02cf4043ff0332780c19484ff8f042bf87e9
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Wed Oct 13 01:56:29 2010 -0500

   Create an abstract interface to be used to implement a SolarMutex

commit 53aa2631927c29c9e7f3f183754f96bb2b07edc5
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Mon Oct 11 21:06:25 2010 -0500

   merge vosremoval-timer.diff
===== writer =====
commit abfd817b49b757e8e1e4b215d4a6c90ed21c1deb
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Sat Oct 16 03:22:02 2010 -0500

   merge vosremoval-mutex.diff

   In practice the changeset is 'inspired' by vosremoval-mutex.diff
   but was essentially redone manually

commit 56fa34307c51b8dad5479d9698d0233752eda5d1
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Fri Oct 15 11:44:38 2010 -0500

   merge vosremoval-reference.diff

commit 1d88bfa001c21d82c7423e622afb41e46a14b869
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Wed Oct 13 01:44:10 2010 -0500

   use SolarMutexGuard to guard the SolarMutex


Norbert

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.