clean up places accessing the NULL at the of an OUString

There were only a couple of real bugs fixed, but we're a little
bit safer now.
This also fixes the assert and the comment in OUString::operator[]
about this.

Change-Id: Ibe16b5794e0ba7ecd345fa0801586d25b015974c

1 files changed, 3 insertions, 3 deletions

diff --git a/vcl/source/window/mnemonic.cxx b/vcl/source/window/mnemonic.cxx
index 0b55245d146b..fe45ae8928cd 100644
--- a/vcl/source/window/mnemonic.cxx
+++ b/vcl/source/window/mnemonic.cxx
@@ -208,14 +208,14 @@ OUString MnemonicGenerator::CreateMnemonic( const OUString& _rKey )
             }
 
             // Search for next word
-            do
+            nIndex++;
+            while ( nIndex < nLen )
             {
-                nIndex++;
                 c = aKey[ nIndex ];
                 if ( c == ' ' )
                     break;
+                nIndex++;
             }
-            while ( nIndex < nLen );
             nIndex++;
         }
         while ( nIndex < nLen );