[Octave-bug-tracker] [bug #65191] Refactor dynamic stack frame for bytecode interpreter

[Markus Mützel]

Follow-up Comment #1, bug#65191 (group octave):

I haven't looked at the functional differences of the patch yet. Just some
initial comments (mostly style - sorry for nagging):

Please, include the bug number in the first line of the commit message.
Typo "unecessary" -> "unnecessary" in commit message.
Use two hash characters "##" if comments start at the beginning of a line in
.m files.
Use two spaces and a hash character "  #" for inline comments in .m files.
Use one space after the negation operator. I was once told to read it as the
English word "not" which would also be separated with a space from the
following word.
Use empty assignment list for .m file functions that take no arguments (e.g.,
"nested2 ()").



    _______________________________________________________

Reply to this item at:

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

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