[Libreoffice-commits] .: sw/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Fri Dec 7 04:55:44 PST 2012


 sw/source/ui/shells/txtattr.cxx |    5 -----
 1 file changed, 5 deletions(-)

New commits:
commit d4ca0036d723963b1874901f7185072a70789a99
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Dec 7 12:52:43 2012 +0000

    eEscape can never be FN_SET_SUPER_SCRIPT
    
    orthogonal enum and slot id things
    
    If it was supposed to be if(eEscape == SVX_ESCAPEMENT_SUPERSCRIPT) then setting
    superscript would instead sets subscript. Seeing as this is wrong since day 0
    and the code should never goes into that branch I reckon its a forgotten
    remnant from before superscript was implemented as it is now and its better to
    remove the code entirely
    
    Change-Id: I73010cf9428c96dfc75033c138694c5781a73c50

diff --git a/sw/source/ui/shells/txtattr.cxx b/sw/source/ui/shells/txtattr.cxx
index a746419..6802fe5 100644
--- a/sw/source/ui/shells/txtattr.cxx
+++ b/sw/source/ui/shells/txtattr.cxx
@@ -123,11 +123,6 @@ void SwTextShell::ExecCharAttr(SfxRequest &rReq)
                 aEscape.GetEsc() = DFLT_ESC_AUTO_SUPER;
             else if(eEscape == SVX_ESCAPEMENT_SUBSCRIPT)
                 aEscape.GetEsc() = DFLT_ESC_AUTO_SUB;
-            if(eState != STATE_OFF )
-            {
-                if(eEscape == FN_SET_SUPER_SCRIPT)
-                    aEscape.GetEsc() *= -1;
-            }
             rSh.SetAttr( aEscape );
             rReq.AppendItem( aEscape );
             rReq.Done();


More information about the Libreoffice-commits mailing list