[Octave-bug-tracker] [bug #62013] Multiple test failures when Octave is compiled with -DNDEBUG

[Rik]

URL:
  <https://savannah.gnu.org/bugs/?62013>

                 Summary: Multiple test failures when Octave is compiled with
-DNDEBUG
                 Project: GNU Octave
            Submitted by: rik5
            Submitted on: Wed 09 Feb 2022 02:58:26 PM PST
                Category: Configuration and Build System
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group: Incorrect Result
                  Status: Confirmed
             Assigned to: None
         Originator Name: 
        Originator Email: 
             Open/Closed: Open
                 Release: 7.0.90
         Discussion Lock: Any
        Operating System: GNU/Linux

    _______________________________________________________

Details:

I set the shell variable CPPFLAGS to '-DNDEBUG' which should have the effect
that assert() statements are ignored in the code.  The pre-processor
definition "NDEBUG" stands for "No DEBUG" and is appropriate when code is
transitioned from development to release.

There may be a slight performance benefit to removing code that is only used
during the development phase.  Even if there is no performance improvement,
the code should compile with or without this definition and work correctly.

However, when I compile the code I get several warnings


libinterp/corefcn/dot.cc:41:54: warning: unused parameter ‘y’
[-Wunused-parameter]
libinterp/corefcn/graphics.cc:5500:52: warning: unused parameter ‘mode’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:127:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/operators/ops.h:142:59: warning: unused parameter ‘idx’
[-Wunused-parameter]
libinterp/parse-tree/lex.ll:3181:9: warning: variable ‘nread’ set but not
used [-Wunused-but-set-variable]
libinterp/parse-tree/lex.ll:3305:62: warning: ‘long_int_val’ may be used
uninitialized in this function [-Wmaybe-uninitialized]
liboctave/array/Array-util.cc:345:19: warning: unused variable ‘idx_n’
[-Wunused-variable]


When I run Octave, simple things like hex constants no longer work.


octave:1> y = 0xF
y = 0


Running 'make check' also shows multiple test failures.

One question for this bug report is whether this should apply to the 7.0.90
release and get fixed there, or whether we should just fix the issue on the
development branch.




    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?62013>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/