Change-Id: Ic3dca6f1f0c0f7de7070801793938569e76e1907

Change-Id: I6b2a40c4b95555c4d8bf0d8674fce46accd49965

Based on a previous patch by Noel Grandin,
<https://gerrit.libreoffice.org/#/c/3613/>, and borrowing from
boost::is_base_and_derived (see comment in include/com/sun/star/uno/Reference.h)
to avoid including Boost headers in URE headers.

Change-Id: Iade5af144dd73ef03bd7d96000134c7a66a5e591
Reviewed-on: https://gerrit.libreoffice.org/3699Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>

Change-Id: Ie1e853fd1dd3154c6805eb8129c35a49150b458c
Reviewed-on: https://gerrit.libreoffice.org/3806Reviewed-by: Tor Lillqvist <tml@iki.fi>
Tested-by: Tor Lillqvist <tml@iki.fi>

Change-Id: I7384a4fd82c8807dfaf9049306b65fe3ef5cab42

It could happen that externals were built incrementally, in case
something the ExternalProject depends on was updated.  To prevent this,
change the dependencies so that the UnpackedTarball is unpacked again
when the ExternalProject's dependencies are newer.  This is possible
without introducing a new target for the purpose due to the refactorings
in previous commits that enforce the name of UnpackedTarball.

Change-Id: Ie7a84064ec2ffc76175cd2b7792517e68664a461

... now that everything is consistent.

Change-Id: I96c15159648815554280202eb1b6d274ead4e7b8

Change-Id: I4d11dc634c4f93ffcdc7e1f359e5d6a0d001aaae

It must always be used exactly once, so replace it with constructor
parameter.

Change-Id: Ifbe87065c19a5185a5705dc461656179002ece5d

This is a bit of a mess, apparently there are some headers here that may
come from the system and others that are always internal?

Change-Id: Ifbf486d3016bc87849e2593d83b3d48dae3aeca0

Change-Id: Ifd85c8e2c4ec16db42001657009b8678d89a8fe1

Change-Id: Iae077cdb1f79a6f539734ecc9897249675f2aa73

Change-Id: I67d0801d8d353951f9c9b1ca7f08b23de9b34067

Change-Id: Ie60d8e9ff8f0abcccc85253be15b6c2c8de944cb

Change-Id: I99ebfcf2a52fe108599938abad87ab73552e6e2a

Change-Id: Ie87b9b2d7dc87ee60c072a2cf80cda5b191d1209

Change-Id: I1168fca3ad3091f9562dfe25aee45329ebbb4988

Change-Id: Iddafa9068edcc20135c08ed205b8fb7fb5382eba

Change-Id: I8e6748eef04f25603851a33d049cb9585fa04cc6

Change-Id: Ia583cfe1d123903ac8f61ad4a224364af7c77ef7

This reverts commit eacf12ee.

Appears to be broken.

Conflicts:
	vcl/source/filter/graphicfilter.cxx

Change-Id: I9ae3fbac2e7e9d08484c063cdb10e080ca89def2

Change-Id: I7ab931bb7c4f0d83dd7701650ec0d6a1ef83fda2

In e2e2cc61 the method was removed
automatically leaving several line breaks etc. as visual noise.

Change-Id: I4581026627fe509895d471f5c28089aaaee85f58

Change-Id: I236c7887895d26a90c0f39655891d84b97b7b3ba

HarfBuzz integration should be functional now, so to give it more wider
testing it is made now a required dependency on non-Windows non-Mac OSs.
By default text layout is now done by HarfBuzz but ICU LayoutEngine is
kept as a fallback and can be enabled with SAL_USE_ICULE env variable.

After 4.1.x is branched, ICU LayoutEngine should be removed completely.

Change-Id: I4fe3beeaf6092f33dd436906c11b83aeafdfbd5d

It turns out storing the width in the layout is not so good idea,
because in some mysterious cases when font fallback is involved we call
GetTextWidth() without calling LayoutText() first, and we return the
width of the previous text run.

It seems all I needed is to pass down the X offset with the glyph item,
and take it into account when calculating the width.

Change-Id: Idbdb6bba00573fb6ca773701757d667e21ac0912

Change-Id: I7881af8907c0735e4d8e8c327103c8d8488ec496

It does not make sense to repeat the same check many times.

Change-Id: I71fac6a4e8283d293a6bd50637ff3a1e6c2ad40d

d3f063cc, by instead restricting the use of the
Boost pointer containers to non-inline code (watch out for implicitly generated
copy ctors and copy assignment ops), thus preventing MSVC from trying to
generate full template instantiations of those containers (similar to
f4227c82 "Blind fix for MSVC, take two").

Turns out the ScConditionalFormat copy ctor was unused anyway (but the
ScConditionalFormatList copy ctor isn't, so only prevent generation of an
implicit copy assignment op there).

Change-Id: I23303a98df7024dbce95638ac13097b71a9c5c04

...uses of which had been introduced with
2fab32e8 "mtclip01: #i112022# Allow extended
clipboard creation" but which did not introduce any implementations of those
services, so this might have been a hook for StarOffice-only functionality.

Change-Id: I4d3c977547147aaca78b6079d1df8fbf6b196650

With 428bde35 this should enable ICU to
bounds-check font tables as mentioned in
http://site.icu-project.org/download/51#TOC-Known-Issues

With this patch also needed is 7de7267f
otherwise application will crash.

Change-Id: I42e31778f762afc80ea21c1fb20952eab42791dc

This function is for running arbitrary command just after unpacking. It
turns out that gb_UnpackedTarball_set_post_action is not enough :-(

Change-Id: Ibad9d7fbcdd2b95a16cc838ad8773eef5c6da019

A different way to do 1c0669af2f1f58e6431b5e489ac48a883e242ba7.
Sometimes one piece of code can be represented several times in the AST,
e.g. with default function arguments.

Change-Id: Ic7799fa0bd918a638bdc8ebef69e6aa91d355bdc

Change-Id: Ibc487aa78d9eba3f936b4f4cc0298adfaef60d5c

Mechanical removal of usage together with OUString ctor, done
by compiler plugin.

Change-Id: I554227f76df0dac620b1b46fca32516f78b462c5

This does not always work well, e.g. when building a return value
in a return statement from a temporary, there is CXXConstructExpr
containing CXXTemporaryObjectExpr, which both share the same location.
This reverts commit 1c0669af2f1f58e6431b5e489ac48a883e242ba7.