tdf#120703 PVS: V519 variable is assigned values twice successively

V519 The 'hResult' variable is assigned values twice successively.
     Perhaps this is a mistake. Check lines: 1097, 1098.

Change-Id: Ib6f9c3467578ebc68b3a15f8dbea4054af45707d
Reviewed-on: https://gerrit.libreoffice.org/62411Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>

fpicker/source/win32/VistaFilePickerImpl.cxx

@@ -1091,11 +1091,9 @@ TFileDialogCustomize VistaFilePickerImpl::impl_getCustomizeInterface()
 static void lcl_removeControlItemsWorkaround(const TFileDialogCustomize& iCustom   ,
                                             ::sal_Int16           nControlId)
 {
-    ::sal_Int32 i       = 0;
-    HRESULT   hResult;
-
-    hResult = iCustom->SetSelectedControlItem(nControlId, 1000);
-    hResult = S_OK;
+    (void)iCustom->SetSelectedControlItem(nControlId, 1000); // Don't care if this fails (useless?)
+    DWORD i = 0;
+    HRESULT hResult = S_OK;
     while ( SUCCEEDED(hResult) )
         hResult = iCustom->RemoveControlItem(nControlId, i++);
 }