ec.c: Don't lose error status when constructing points from a sequence.
authorMark Wooding <mdw@distorted.org.uk>
Sun, 24 Nov 2019 22:30:48 +0000 (22:30 +0000)
committerMark Wooding <mdw@distorted.org.uk>
Wed, 27 Nov 2019 15:10:44 +0000 (15:10 +0000)
commite91150b266bbd768905258166601fdf333d3d987
treeb932f073462bcc3944523e3f78e17dac2377661e
parent08720fe03bb49ed4df3b0fbd99c2c8eec145fdcc
ec.c: Don't lose error status when constructing points from a sequence.

The code would call `ecptxl_2' or `ecptxl_3' as appropriate, stash the
error status in `rc', and then fall out of the `if/else if' ladder to
`ok' -- which clobbers `rc', losing the error.  This is unfortunate if
the point has been partially filled in.

The right fix is to go unconditionally to `end'.  We already have `rc'
set appropriately, so `ok' isn't necessary; `ecptxl_2' has already
converted the point to internal form, and `ecptxl_3' constructs the
point in internal form anyway, so it would be wrong to drop into `fix'.
ec.c