[Octave-bug-tracker] [bug #61583] [octave forge] (tisean) Remove usage of "error_state"

[Markus Mützel]

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

                 Summary: [octave forge] (tisean) Remove usage of
"error_state"
                 Project: GNU Octave
            Submitted by: mmuetzel
            Submitted on: Mon 29 Nov 2021 02:07:09 PM CET
                Category: Octave Forge Package
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group: Build Failure
                  Status: None
             Assigned to: None
         Originator Name: 
        Originator Email: 
             Open/Closed: Open
                 Release: dev
         Discussion Lock: Any
        Operating System: Any

    _______________________________________________________

Details:

Quoting jwe from bug #61551:
> The global 'error_state' variable has been useless for quite some time now. 
It's been removed on the default branch (what will become version 8.1 in about
a year).  Since about 6 years ago, it has never had a value other than 0 in
Octave code and is no longer useful for checking whether an error has occurred
because exceptions are thrown, so checking the state of a variable after an
error occurs is not useful except inside a 'catch' clause that handles
'octave::execution_exception'.
> All uses of this variable should be removed from the video package and any
other packages that still use it.  You should have been seeing a warning at
compile time about it being deprecated since before version 6 was released (it
was marked as deprecated in May 2019).

This also applies to the tisean package.
I'll attach a possible patch as soon as I'll have a bug number.




    _______________________________________________________

Reply to this item at:

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

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