Change-Id: I82113fcd4182f9557ba0e7258498b5d4e798bf03

Change-Id: I19f98152b5bd8c9b2cdfacb560d13fc0c1f8c4be

Change-Id: I591188e0323bb75c55cad9a078eea188f79b3a03

Change-Id: I89f0cfc9049503318955d58dde3f9a3e1197c45d

Change-Id: I0cffc9df6d971393ae874060a8c306e12966370c

Change-Id: I30b61fa17a941b785a4f148e2ce6c912dbd3c645

Change-Id: I662c89e6e3e0917b51e8084bc50a9ea38d6c0838

Change-Id: I809e771e4f052ad64300b09174a1bdbc769e15d3

Change-Id: I7031e7408f5bcd9ae85fc50952253e14d2fb2edf

I don't think this actually matters

Change-Id: Ia7190f436eeb47888b1aace1391cf0b5aef9f9f0

The inner loop which iterates over the characters of
m_pTxtNode->GetTxt() is already bounded to the length of the
string, so there's no need to pay the price of checking its length
for each array position

Change-Id: I7674ea2b46db75fea30dd016b96ec932068fd73b
Reviewed-on: https://gerrit.libreoffice.org/11784Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

(Spotted while valgrinding an unrelated issue)

Change-Id: I2eab4c08e251d79f427fd01442c4dce20d7d89f0
Reviewed-on: https://gerrit.libreoffice.org/11785Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

Change-Id: I3a9e01237d8cade5d99955f02c01c25618ccd9de

Change-Id: I3ac385b8f21409b5083b1224652283fec8bb2fa4

The same pattern as introduced elsewhere recently.

Change-Id: Ie7b4fd1ec26d417f63bee26dd336dfc54f4847dc

Change-Id: I74a61005fa9534b3f96cd392c6e38c54c5fd35e1

to make it's intended purpose clearly distinguishable from AddNextRef

Change-Id: I5da780b48b19fd873667b648031bc394113f953b
Reviewed-on: https://gerrit.libreoffice.org/11763Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

Change-Id: I26a1164d44f464a44ed7deb93ac39465feb0541c
Reviewed-on: https://gerrit.libreoffice.org/11768Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

The entire button appears on mouse over.

Change-Id: I27e713f75a30ec4a2b271d813fcbcbe8c91a4180

Change-Id: Ia406baec2867605bf515dac0ba090500e2bbd1ce

Change-Id: I545328a1c126b3bfeee8c1f662c5eb9a471d33ca

You used to have to convert formula cells to static values by copying
them and pasting them as values onto the original place.  Why not just
allow converting them in place?

This is something I've always wanted to implement.

Change-Id: Ie1c80029a762fcf92dc20ffe74260ea54c61da28

Change-Id: I385e756109ab0a47feeeb4407ba4a2a4b68548dd
Reviewed-on: https://gerrit.libreoffice.org/11728Reviewed-by: Peter Foley <pefoley2@verizon.net>
Tested-by: Peter Foley <pefoley2@verizon.net>

Change-Id: Ie154bfb15a19ae98b2bc4eb6626240a00cf182d4

Change-Id: Ia6186be4ed48fd75d1063eed233a88e09b675bf1
Reviewed-on: https://gerrit.libreoffice.org/11774Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>

Change-Id: Id4bc638c81d695745fc44bde16252e3040f0a03c

Change-Id: Iacbf9f46a1b5683b6de5dd93002a69078af46951

I just ran SlickEdit's 'beautify' command with some follow-up manual
editing.

Change-Id: I402e782e7d147c4c95ebaa8ffcc40b50a0c565a7

Change-Id: I18a9355bb902b19f0eeecb26d33eba4fb25db8cb

Change-Id: I01f473658708a75c3eac6219efd38cd8ec5aa44f

We need to set the "paragraph summation" flag also for .pptx import. (As is
already done for .ppt import.)

This is what in the Options (Preferences) UI is called "Add spacing between
paragraphs and tables (in current document)", on the LibreOffice
Impress:General page. This is a setting that when set becomes local to the
document. I actually don't really understand why this needs to be a
user-visible option, wouldn't it be enough for it to be an internal flag that
is set for documents imported from .ppt and .pptx, and is clear otherwise?

Change-Id: I525178957276e7d587b6ac4be8ae7919fc245a27

Change-Id: I449fe34be396114529357388c87162aaf8976292

Change-Id: I4412a0c1e4dee94cd717f24b8df5e257d895f721

This is supported in GCC 4.6.0 already:
https://gcc.gnu.org/onlinedocs/gcc-4.6.0/gcc/Diagnostic-Pragmas.html

Change-Id: I2f67e588eea3a323a2e9c81e39e56ab2e715a817

This has been supported by GCC and clang for a very long time.

Change-Id: I410a2b39004c932003f8cbefe935aedb109b1163

Change-Id: I10dfb947bbd9e792c98932608acb2f0987fd8605

this fix is only for simple color backgrounds; gradient, hatching and bitmap are not implement.

Change-Id: I5ec31c0105491c6d27835d63648fedff21998926
Reviewed-on: https://gerrit.libreoffice.org/11762Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>

otherwise you're just deleting a char right after whereever
the cursor used to be.

It can be possible to not GotoField if the field in question
is in a redline-deleted portion and show changes is not on. i.e.
its actually already deleted

Change-Id: Ic4833d82c5c33e556f3e159e9732a6db071f5622

Change-Id: Ide4e95828fad867fbc3cd1300f29e832b3c83017