some id and string renaming from b08e2730 was reverted, those parts of the changes are reapplied here

This reverts commit 83a28682.

- iterators should be compared with !=, not <
- iterator invalidation on erase should be avoided by using the
  iterator that is returned by erase
- in WrongList::operator== only one iterator was incremented

Also, change the contained text slightly.
Thanks to William Gathoye for noticing and producing a first patch

With big thanks to Lapo Calamandrei for his willingness to relicense his Save and Save As icons.

Copyrights:
icon-themes/tango/cmd/[l,s]c_reload.png and icon-themes/tango/res/[l,s]c05508.png are from the Public Domain Tango artwork available under tango.freedesktop.org/releases/tango-icon-theme-0.8.90.tar.gz

icon-themes/tango/cmd/lc_save.svg
icon-themes/tango/cmd/[l,s]c_save.png,
icon-themes/tango/cmd/[l,s]c_saveas.png,
icon-themes/tango/cmd/[l,s]c_savebasicas.png,
icon-themes/tango/res/[l,s]c05502.png,
icon-themes/tango/res/[l,s]c05505.png are
(C) Lapo Calamandrei, as available from http://git.gnome.org/browse/gnome-icon-theme/, relicensed to LGPL3+/MPL

Move everything that requires ObjectiveC stuff to the ObjC++ files in
the aqua subdirectory, where it belongs.

Another code-review follow-up fix from fdo#44533 - don't use
Sequence::getArray, that tends to create a new copy everytime you
call.

Follow-up fix for 540963d8
std::algos use iterator arguments by value, and may change them.

This fixes fdo#44533.

Regression from 3.4.5.