7d1f4cde changed String::GetToken() to
expect and return a sal_Int32 and return -1 instead of STRING_NOTFOUND.
So far so good though not consistent, but unfortunately it also replaced
STRING_NOTFOUND with -1 in a few other places like where
String::Search() is called but that still returns STRING_NOTFOUND ...

Change-Id: I1ef1891cef220c1f1f9032af173d80f3f0e29e71