[Libreoffice-commits] core.git: sc/source

jailletc36 christophe.jaillet at wanadoo.fr
Wed Feb 6 04:51:14 PST 2013


 sc/source/ui/optdlg/opredlin.cxx |    4 ----
 1 file changed, 4 deletions(-)

New commits:
commit 8f3a6129e47d52a9898baf89a096bbd55abbca72
Author: jailletc36 <christophe.jaillet at wanadoo.fr>
Date:   Wed Feb 6 12:25:21 2013 +0100

    Remove useless function calls.
    
    nPos has already been computed the 3 lines above. No need to recompute it.
    
    Change-Id: Iad739d6fffd2c40cc633789edfe88d13e87dd28b
    Signed-off-by: jailletc36 <christophe.jaillet at wanadoo.fr>
    Reviewed-on: https://gerrit.libreoffice.org/2015
    Reviewed-by: Tomáš Chvátal <tchvatal at suse.cz>
    Tested-by: Tomáš Chvátal <tchvatal at suse.cz>
    Reviewed-by: Norbert Thiebaud <nthiebaud at gmail.com>

diff --git a/sc/source/ui/optdlg/opredlin.cxx b/sc/source/ui/optdlg/opredlin.cxx
index 6badae1..cf4b486 100644
--- a/sc/source/ui/optdlg/opredlin.cxx
+++ b/sc/source/ui/optdlg/opredlin.cxx
@@ -89,7 +89,6 @@ sal_Bool ScRedlineOptionsTabPage::FillItemSet( SfxItemSet& /* rSet */ )
     nPos = aContentColorLB.GetSelectEntryPos();
     if (nPos != LISTBOX_ENTRY_NOTFOUND)
     {
-        nPos = aContentColorLB.GetSelectEntryPos();
         if (nPos!=0)
             nNew= aContentColorLB.GetEntryColor(nPos).GetColor();
         else
@@ -101,7 +100,6 @@ sal_Bool ScRedlineOptionsTabPage::FillItemSet( SfxItemSet& /* rSet */ )
     nPos = aMoveColorLB.GetSelectEntryPos();
     if (nPos != LISTBOX_ENTRY_NOTFOUND)
     {
-        nPos = aMoveColorLB.GetSelectEntryPos();
         if (nPos!=0)
             nNew= aMoveColorLB.GetEntryColor(nPos).GetColor();
         else
@@ -113,7 +111,6 @@ sal_Bool ScRedlineOptionsTabPage::FillItemSet( SfxItemSet& /* rSet */ )
     nPos = aInsertColorLB.GetSelectEntryPos();
     if (nPos != LISTBOX_ENTRY_NOTFOUND)
     {
-        nPos = aInsertColorLB.GetSelectEntryPos();
         if (nPos!=0)
             nNew= aInsertColorLB.GetEntryColor(nPos).GetColor();
         else
@@ -125,7 +122,6 @@ sal_Bool ScRedlineOptionsTabPage::FillItemSet( SfxItemSet& /* rSet */ )
     nPos = aRemoveColorLB.GetSelectEntryPos();
     if (nPos != LISTBOX_ENTRY_NOTFOUND)
     {
-        nPos = aRemoveColorLB.GetSelectEntryPos();
         if (nPos!=0)
             nNew= aRemoveColorLB.GetEntryColor(nPos).GetColor();
         else


More information about the Libreoffice-commits mailing list