And fixed the fallouts of this change to get it to build.

This reverts commit ed2aa8e1ebd4bea03d958b86e721400ef59cb76d.

These data members were indeed necessary. Let's not remove them.

This reverts commit 4f46e40020699ea79d0d7254431aff27b087e4e2.

I found one abuse of sal_Bool being used as an integer.  The block
mode was a boolean but had 3 different states.  Replaced that with
enum.

These two can be used as local variables as they are only used in
one method.

this unit test is disabled for now because there is a problem with ScProgress in ScDocument::MoveTab during unit tests

Simple NULL pointer checks will prevent crashes....

Now that we always set locale to en_US during unit test, this test
should be okay.

DDE is deprecated, and it relies on platform dependent file paths which
are non-portable.  Better use external references for paste link
operations.

The size of the array may change during the loop, so we need to always
query it dynamically via size() instead of caching it at the beginning
of the loop.
Signed-off-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

the DECLARE_LIST container decremented the internal index pointin to the current element when REMOVE was called, when we now iterate over the new container and remove an element we must do this ourself