[Octave-bug-tracker] [bug #56037] qp exits with an indexing error for a specific input with OpenBLAS

[Rik]

Follow-up Comment #11, bug #56037 (project octave):

Nice that the bug fix is so simple in the end.

For the commit message, my understanding of the GNU standards which Octave has
chosen to follow is

1) First sentence is a concise summary of the overall changeset
 1a) Octave-specific modification is that if the change occurred in an m-file
the name of the m-file followed by a colon starts the line.
 1b) Octave occasionally use the prefixes "doc:", "maint:", "test:" if the
changeset is for one of these purposes.
2) Additional sentences describe more detail about the overall change, perhaps
including reasoning for a particular approach over another.
3) Changes on a per-file basis document what was done, not why it was done
(that is the global reasoning in parts 1 & 2)

So for the commit message I might try something like


Fix obscure corner case when calculating qp caused by a typo (bug #56037).

* __qp__.cc (qp): Fix typo where "eigenvalH" was used instead of
"eigenvalrH".




    _______________________________________________________

Reply to this item at:

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

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