Part of code
   1299             ImplHeadItemList::iterator it = mpItemList->begin();
   1300             ::std::advance( it, nPos );
   1301             ImplHeadItem* pItem = *it;
   1302             mpItemList->erase( it );
   1303             if ( nNewPos < nPos ) <-- case move to left
   1304                 nPos = nNewPos;
   1305             it = mpItemList->begin();
   1306             ::std::advance( it, nPos );
   1307             mpItemList->insert( it, pItem );
So in move to right case, we erase and insert pItem at the same place
We just have to use nNewPos in all cases

regression from bd24a310

Also add back "ImplUpdate" call erroneously removed in merge
commit 67d2af6f
(there is an ImplUpdate call in each parent of that merge commit)

Change-Id: I100b0c6874b366f777c5d38470fcee7a9a7326df
Reviewed-on: https://gerrit.libreoffice.org/14777Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Lionel Elie Mamane <lionel@mamane.lu>

Change-Id: I97d58f0e6780986356479279f91eba46951b1be0
Reviewed-on: https://gerrit.libreoffice.org/14783Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>

aHdl is an SdrHdlList, so reading aHdl.GetHdl() as "get the handle of the
handle" was confusing.

Change-Id: Ib21fa74cc1daa4b6d2034f07715147a8fb0cf934
Reviewed-on: https://gerrit.libreoffice.org/14779Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>

Change-Id: Icc59bfc612903d993fb4b091a28efe23e366475f

BASIC error type was printing as "Error "+ odd char
 instead of "Error " + error number

Change-Id: I7407833626d6117f932cc29c6dab1a8a64c50b5a
Reviewed-on: https://gerrit.libreoffice.org/14776Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>

Change-Id: I608cf04cf4b0b991f4a45a868566faeae5be77c9

Change-Id: I42c893677c1b8a35ee502a9ae438d4c0f19867d3

Change-Id: I04c1a00903cc0c057fc436e391f93b1e05eed8a1

Change-Id: Id5cea16da184b12b6a630dc7e9c0f7d199b1da49

Change-Id: Ib0c8801841e122be58f5a931c7413b08487e1f23
Reviewed-on: https://gerrit.libreoffice.org/14694Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>

...appears to be occasionally necessary, probably since
d8e7bc04 "Remove hack of replacing 0 with
1/LONG_MAX"

Change-Id: I1eb7277bfdc9722c761bd03678cd770a3763de1f

For whatever reason "objdump.exe -t xicontent.o" on Windows now
lists Start@ScRefreshTimer@@EAEXXZ (probably because it's now
a virtual function). Also the reference to SetRefreshDelay
vanished after dropping the virtual keyword from SetRefreshDelay.

The linux object file doesn't even refer to a ScRefreshTimer
function. Start() can stay private, as the timer is handled via
SetRefreshDelay.

Probably Stop() should also become private in this case.

This also reverts 2c0189a8

Change-Id: I2a6900cd6bd76c4fec9f30131152b60a5bd2c2e7

Change-Id: I7ee56f06e8e50180e31c01253c9100c11b485c0d

Change-Id: I97c7977fdfea57b5d1c45ce0c5902b74cc719db9

Change-Id: Ic7c02736505969c3d74acc59b80794b3be62bf6b

Change-Id: I6522e7a2bb895973d9d8cf1b7cfbad2f2abc62a2

Change-Id: I5bf4d5bb84cc85e631b27741a6fc171ec99eead4

Change-Id: Ib42304e9ceb6b49e2dd8ed92f2a86409921e9ef6

looking into gtk it doesn't move anything, it just
exposes the moved from and moved to areas to force them
to be redrawn, its not in practice an optimization

Change-Id: I38ea91d2463094bab50c436302022849986a3b0c

Change-Id: Ief4da1f1241fc4175f5ad132fd103e6d29470b58

Change-Id: Ib7fe0d9ed2f5b5aae95b73a01f1eb059a01dc2b4

Change-Id: Ia12b325e41238b9e12e2f388273bd01b96d61c08

Change-Id: Idbcb1f7461ac5a1d5e07d7d65f6a7d86f2e3cc57

Change-Id: I7714a4638b03d020820b276028c0819ef054fa26

Change-Id: I4f2c1d743ce2f30e8c24180b73f0716fc13b459e

Change-Id: I6676c1012c4321b53b6ba01ebd4bfa32d56afcf9

Change-Id: Id50533bc52f36d2daf31bfda4fabfb6d126df4b9

Change-Id: Icdc5abf9dca727a8cc312ddb5861f7a34a38bbe1

Change-Id: Icdf85d6f6f5734a3cf952def8782efe9edc7c0a9

Change-Id: I762062a4782438ec5bbcb53da9a42f44db65c40d

Change-Id: I45d6fc28e898b945c7754b9ee1f2842247a071e6

Change-Id: I15654a7aa5a99cb3aa6f8ba222cf2aa1d2f040ab

Change-Id: Iaac3618501b4e382afe8d183836a370eea699af4

Change-Id: I03db46afcc0cb5e5d7a134b1bdd327abb542e63c

Change-Id: Id3852abe9bc7bbe774331a7d691abd2a79ff59b4

Change-Id: I12290bed7e4f298ab90393b8de6e2b6e7061e53f

Change-Id: I12290bed7e4f298ab90393b8de6e2b6e7061e53f

Change-Id: I4cfbb53763001854d37fbcd304e77e65279173b6

Change-Id: I8ca272481e573bf3338c5c1b9873a39022928812

Change-Id: I2c73bb2b2edd3b830dad5d1f7d8f7bc72622b11c