cppcheck: unreadVariable

Change-Id: Id4c43a6bcdf915b98410af9ceb0dbf20df3a9498

vcl/source/window/mnemonic.cxx

@@ -287,49 +287,12 @@ OUString MnemonicGenerator::CreateMnemonic( const OUString& _rKey )
                             nIndex--;
                     }
                     rKey = rKey.replaceAt( nIndex, 0, aStr );
-                    bChanged = true;
                     break;
                 }
             }
         }
     }
 
-// #i87415# Duplicates mnemonics are bad for consistent keyboard accessibility
-// It's probably better to not have mnemonics for some widgets, than to have ambiguous ones.
-//    if( ! bChanged )
-//    {
-//        /*
-//         *  #97809# if all else fails use the first character of a word
-//         *  anyway and live with duplicate mnemonics
-//         */
-//        nIndex = 0;
-//        do
-//        {
-//            c = aKey.GetChar( nIndex );
-
-//            nMnemonicIndex = ImplGetMnemonicIndex( c );
-//            if ( nMnemonicIndex != MNEMONIC_INDEX_NOTFOUND )
-//            {
-//                maMnemonics[nMnemonicIndex] = 0;
-//                rKey.Insert( MNEMONIC_CHAR, nIndex );
-//                bChanged = true;
-//                break;
-//            }
-
-//            // Search for next word
-//            do
-//            {
-//                nIndex++;
-//                c = aKey.GetChar( nIndex );
-//                if ( c == ' ' )
-//                    break;
-//            }
-//            while ( nIndex < nLen );
-//            nIndex++;
-//        }
-//        while ( nIndex < nLen );
-//    }
-
     return rKey;
 }