xcalc: Changes to 'master'

Alan Coopersmith alanc at kemper.freedesktop.org
Sat May 5 19:23:05 UTC 2018


 math.c |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

New commits:
commit 8df4b463f69fc3e7e08ce5de284ed7f318935c1e
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 5 12:20:12 2018 -0700

    Reword fall through comments to appease gcc -Wimplicit-fallthrough
    
    Gets rid of these warnings:
    math.c:707:24: warning: this statement may fall through [-Wimplicit-fallthrough=]
       case kSQR:   flagINV = !flagINV; /* fall through to */
                    ~~~~~~~~^~~~~~~~~~
    math.c:708:3: note: here
       case kSQRT:  if (flagINV) dnum=dnum*dnum;
       ^~~~
    math.c:711:24: warning: this statement may fall through [-Wimplicit-fallthrough=]
       case k10X:   flagINV = !flagINV; /* fall through to */
                    ~~~~~~~~^~~~~~~~~~
    math.c:712:3: note: here
       case kLOG:   if (flagINV) dnum=pow(10.0,dnum);
       ^~~~
    math.c:715:24: warning: this statement may fall through [-Wimplicit-fallthrough=]
       case kEXP:   flagINV = !flagINV; /* fall through to */
                    ~~~~~~~~^~~~~~~~~~
    math.c:716:3: note: here
       case kLN:    if (flagINV) dnum=exp(dnum);
       ^~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 7a90211e35841758ff11e79a1633494e2055df88
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 5 12:15:19 2018 -0700

    Fix misleading indentation in math.c
    
    math.c: In function ‘numeric’:
    math.c:267:3: warning: this ‘if’ clause does not guard... [-Wmisleading-indentat
    ion]
       if ((int) strlen(dispstr) >= MAXDISP)
       ^~
    math.c:270:5: note: ...this statement, but the latter is misleadingly indented a
    s if it were guarded by the ‘if’
         switch (keynum){
         ^~~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>



More information about the xorg-commit mailing list