[Octave-bug-tracker] [bug #37099] eval() with '; ' as second parameter causes parse error

[John W. Eaton]

Follow-up Comment #3, bug #37099 (project octave):

Vivek,

Thanks for submitting a patch.  I'm not applying it because there are some
problems with the change you are proposing.

First, I don't think it makes sense to allow ';' to literally be an
expression.  A semicolon is a separator between expressions, not an expression
itself.  With the change you made, things like ;++ are accepted as valid
syntax but cause an immediate segfault.  Even if you avoid the crash, that
syntax doesn't make sense to me.

Further, with your patch applied, make doesn't complete properly for me.  It
fails when trying to build figures for the documentation.  If I skip that part
of the build and run "make check" in the test subdirectory, there are some new
test failures.  You should get in the habit of running "make check" after
making any change so that you can be sure you haven't introduced any
unexpected problems.

Also, a few minor points about submitting changes:

* Please don't use tabs for indenting in Octave source files.

* Please follow the conventions of commit messages as outlined here:
http://hg.savannah.gnu.org/hgweb/octave/file/94d512d712e3/doc/interpreter/contrib.txi#l235.
 If you'd like to see examples of commit messages, look at a sampling of the
commit messages for changes that I have made.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?37099>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/