by:
- using for range loops
- using returned iterator
- using find_if
- using std::distance (with a find_if) to avoid looping for searching a position
- using erase-find_if idiom
- declaring "const" all the aTab vars
- avoiding creating a variable used once
- using "++<var>" instead of "+= <var>+1"

Change-Id: I4139a24e43c10151dd5f0baf57a7aef5bd994ec7
Reviewed-on: https://gerrit.libreoffice.org/47195Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>