* 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

The data structure holding the UI elements in the browser listbox was a
terrible mess - it held the items in an unordered_map, but then accessed
them via a vector containing iterators to this unordered_map.

Fixed the problem (and cleaned all this up) by removing the vector of
iterators, and turning the unordered_map into a normal vector.  When we need
access by name, we just go through all the items; it is always just a handful
of them anyway.

We use pretty old compiler for the official build, so it should be compatible
with most systems. The old system libraries breaked 3rd-party extensions.
There were also problems with exceptions hadnling.

Although fdo#45330 was a duplicate of fdo#45117 which was fixed
earlier, this patch makes the layout look better when the button
text is very long, because it resizes the boundary boxes of fixed
text and ruler, too. I've changed the sequence of
the functions in InitSize() and the size calculations.

This is an inline function, so there should be no problems with binary
compatibility.