4.0 alpha compiler warnings

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

4.0 alpha compiler warnings

Dave Bollinger
lhf wrote:
 >> Could you please send us the exact location so that we can se if it's
 >> something that can be "fixed"?

  Following are the warnings generated by Borland C++ Builder 4.  Nothing
too serious, but thought you might want to check them out anyway.  Hope the
formatting survives the email journey:

[C++ Warning] lcode.c(564): W8084 Suggest parentheses to clarify
precedence.
        case OP_PUSHINT: i = CREATE_S(OP_ADDI, -GETARG_S(i)); break; /*
`a-k' */
                                                          ^
[C++ Warning] lcode.c(587): W8084 Suggest parentheses to clarify
precedence.
      if (i == CREATE_U(OP_PUSHNIL, 1))  /* `a~=nil' */
                                     ^
[C++ Warning] lcode.c(588): W8084 Suggest parentheses to clarify
precedence.
        i = CREATE_S(OP_JMPT, NO_JUMP);
                                     ^
[C++ Warning] lcode.c(597): W8084 Suggest parentheses to clarify
precedence.
      if (i == CREATE_U(OP_PUSHNIL, 1)) {  /* `a==nil' */
                                     ^
[C++ Warning] lcode.c(614): W8084 Suggest parentheses to clarify
precedence.
        case OP_NOT: i = CREATE_S(invertjump(o), NO_JUMP); break;
                                                        ^
[C++ Warning] lcode.c(630): W8084 Suggest parentheses to clarify
precedence.
    case iU: i = CREATE_U(o, arg1); break;
                                 ^
[C++ Warning] lcode.c(631): W8084 Suggest parentheses to clarify
precedence.
    case iS: i = CREATE_S(o, arg1); break;
                                 ^
[C++ Warning] lparser.c(326): W8066 Unreachable code.
      break;
          ^
[C++ Warning] lparser.c(331): W8070 Function should return a value.
}
^
[C++ Warning] lparser.c(486): W8004 'nargs' is assigned a value that is
never used.
    }
    ^
[C++ Warning] lundump.c(95): W8008 Condition is always true.
 if (sizeof(Instruction)==4)
^
[C++ Warning] lundump.c(99): W8071 Conversion may lose significant digits.
   c=p[0]; p[0]=p[3]; p[3]=c;
                           ^
[C++ Warning] lundump.c(100): W8071 Conversion may lose significant digits.
   c=p[1]; p[1]=p[2]; p[2]=c;
                           ^
[C++ Warning] lundump.c(102): W8008 Condition is always false.
 else if (sizeof(Instruction)==2)
^
[C++ Warning] lundump.c(103): W8066 Unreachable code.
  while (size--)
      ^
[C++ Warning] lundump.c(106): W8071 Conversion may lose significant digits.
   c=p[0]; p[0]=p[1]; p[1]=c;
                           ^
  Cheers,

  Dave