Change-Id: Ib97f667c9acb644c3becb81f851ad8c7e4ff31ea

Change-Id: I53e7e767517581511892884f273edd703448cb6f

Change-Id: I3bccfa773448e17ac3c8ece6b99f85c6d399b0cf

Change-Id: I97d91a758dd82d64768d75c1d2ddd279de5f6034

Modules sal, salhelper, cppu, cppuhelper, codemaker (selectively) and odk
have kept them, in order not to break external API (the automatic using declaration
is LO-internal).

Change-Id: I588fc9e0c45b914f824f91c0376980621d730f09

Change-Id: I6378c79f77ab897bf5a4e39c00944037beeef328
Reviewed-on: https://gerrit.libreoffice.org/3220Reviewed-by: Andras Timar <atimar@suse.com>
Tested-by: Andras Timar <atimar@suse.com>

Acquire the yield mutex in the VirtualDevice destructor only in a dbgutil
build on OS X. (I added that acquire yesterday to fix an assertion when
exiting on OS X.) Seems to fix a hang in "make subsequentcheck" on Linux. No
idea what is going on exactly, sigh.

Change-Id: Ifcb1897720998d0725a8db0d5f8c75848c6a30f7

Change-Id: Ifa07f9b5613b4a75c5b72178cb276b9c0b495a62

Change-Id: I8766ba5258f923d47474fa77e14eb7fbff530ffc

Change-Id: Id0fdba93a5d3a46a79cebea8839bdfd467ef4f6f

Change-Id: I42991d7a9c9e0bd4a023739051393935efa5c29f

The LayoutManager for some reason has both a ToolbarLayoutManager
pointer field, and a uno::Reference to an aspect (or whatever term one
should use) of the same ToolbarLayoutManager. (I.e. esssentially two
fields for the same thing. Why it doesn't create such variables where
needed instead I don't know.)

Anyway, for some reason there were lots of instances where a local
variable was initialised with this second field but then never used. I
removed those. (Surely just copying the field into a local variable
doesn't have any interesting side effect that would explain this
pattern?)

Change-Id: Ibdfbd9476c39d3e83b58e81469b94d9a87444ca8

Change-Id: I1afd93fc9a56e85a30991bce9ca2350764cf1ec2

It seems to be the type_info most commonly looked up dynamically, even
the only one in an initial test. I think it is a good idea to avoid
dlsym() if possible.

Change-Id: I0379c534e10efefafdd253ee651f6c74e4aa47d5

Change-Id: If0a6296c9b226c479413b475788d0c11d6ee7318

Otherwise we get an assertion failure in a dbgutil build from the
DBG_TESTSOLARMUTEX() in OutputDevice::ImplReleaseGraphics() when
quitting, at least in 64-bit LO on OS X.

Change-Id: Ieb54e6b58033279f807fe91f4f6139018f643903

The end result is still not satisfactory but it is getting closer.

Oh how nice it would have been to have some clean useful documentation for how
the text layout machinery works. The number of entry points in the low-level
"backends" (like CoreTextLayout) is quite small, so I doubt it would have
required lots of text to document the "protocol" used between the upper layers
and the layout machinery. Now one just have to use educated guessing aided by
lots of debugging printout. But this is true for most of the codebase, of
course.

Change-Id: I791f190819a187418865279f6a5b4aa598e01d3c

Change-Id: I184c8c69f4a9afdb0019193b09f61c65e47d36ae

Change-Id: I89040b62c0961be160fcfaa793e48326fa7a584c

Change-Id: I9373647e5d710acb26c9818e6d06d4bfb862a7df

CTLineGetImageBounds() does not include space taken by trailing whitespace in
the line.

Change-Id: I403906daed952bac58fa11634beb6c7faa39b9b3

But like there, I don't know if it actually is needed or not.

Change-Id: I25bf5d8cf160b56ee4deb47ab9774313eb9590c8

Change-Id: I6f185b4f55418e44956f9dd51e1e300f553ac355

Change-Id: Ie98d242777cec92db6512f53fe102ecc65dc469c

Change-Id: I06121b065f360ecfeddb25c9e1487f4849e453a2

Change-Id: Ib875973d31f71bab75973f683621118eb9cb927b

Change-Id: I144c93d0bdd8758dcdf490f29051c8dcaea500d8

Change-Id: Ide7c37c9738ca640e7d80ede904ad8d7702e5a34

Change-Id: Ic6e39b81c04d5e416d1a000def39b3a127fa7940

Change-Id: I0e28425c25e07abe65c4162025fa35a4abe14121

Change-Id: Id1720a55bf7984e2b1316fbfadb0d12ae7fd6b4d

Change-Id: I3cd2129d98c3d936d8ec31f89554b37f12427c1c

Change-Id: Ib6339c29f6820e75ff99aeb0547145597771d584

Change-Id: I73ce022d064c7ccfeb71f5560f112aac3d777bf7

Change-Id: Ife833566122da25c366938c9c2d127fca0903bb7

Change-Id: I52d2ed4b0454233da21e9040dd324772247113c1

Change-Id: Iac1d8d0646bff131b1ade79bb47a914b350f31f2

$(lastword $(MAKEFILE_LIST)) is not what is expected if the makefile
includes other makefile as the first step (as some do). See
UnpackedTarball.mk, where I already tried to workaround the problem.

Change-Id: Ib713a698f52ba16f46fbbc4c50b43edd69c9a472

Change-Id: I47af280e24bff248e6404ec18c1afef8c461b40b

Change-Id: I8f290b1551663b7e967c6d77e8836a1f3d655834