tdf#120703 PVS: revert commit 8ae98805

... which is apparently not needed anymore for #i41635. The revert allows
to properly skip "deselect" accessible event when there's no selection
previously.

Also removed the other identical correction, which was added in commit
4883fd31.

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

svtools/source/control/valueset.cxx

@@ -1761,7 +1761,7 @@ void ValueSet::SelectItem( sal_uInt16 nItemId )
     if ( !((mnSelItemId != nItemId) || mbNoSelection) )
         return;
 
-    sal_uInt16 nOldItem = mnSelItemId ? mnSelItemId : 1;
+    const sal_uInt16 nOldItem = mnSelItemId;
     mnSelItemId = nItemId;
     mbNoSelection = false;
 
@@ -2857,7 +2857,7 @@ void SvtValueSet::SelectItem( sal_uInt16 nItemId )
     if ( !((mnSelItemId != nItemId) || mbNoSelection) )
         return;
 
-    sal_uInt16 nOldItem = mnSelItemId ? mnSelItemId : 1;
+    const sal_uInt16 nOldItem = mnSelItemId;
     mnSelItemId = nItemId;
     mbNoSelection = false;