This reverts commit 6396ab80.
per the author's request

* boolean, string conversion.
* removing virtual keywords when it makes no sense.
* privatize methods that are called only from within.
* removed methods that are not used.
* etc.

This should fix Stephan's tinderbox error.

createTextCursorByRange() likes to throw runtime exception, even though it
just means 'we were unable to create the cursor'.

Some windowmanagers on small devides do not have the close window button,
so all dialogs should have its own way how to diappear.

It is now done like in other application. They use "Close" button instead
of "OK". They put it into the right side and delimit it by a line, so
it looks like another dialog in the application.

* for includes from other modules use <> instead of "", i.e.
  <boost/ptr_container/ptr_vector.hpp>
* use size_t instead of int for container positions and count
* no need for DeleteAndDestroy(), in fact delete p followed by
  ptr_vector::erase() attempted to delete the instance twice and would crash
* adapted places that accessed DeleteAndDestroy() to operate on the vector
  instead
  * introduced GetDataVector() for that
* changed the DeleteOnTab() loop that used DeleteAndDestroy() to properly
  iterate over the container instead
* changed UpdateReference() to a loop using iterator
* changed operator==() to use size_t instead of sal_uInt16
  * note aside: mixing sal_uInt16 with a container's size isn't a good idea
* adapted places that access Count() and GetObject() to use size_t instead of
  sal_uInt16
* made GetObject() const and return const ScDetOpData*
* changed indentation to 4 spaces per level

converts usage of SV_DECL_PTRARR_DEL in sc/inc/detdata.hxx and associated code
to boost::ptr_vector

...where the outer JobQueue::enter blocks on m_cndWait after it has been reset
again due to m_lstJob.empty().

this is second part of the fix for the two bugs