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>
Mon, 25 Nov 2019 17:31:23 +0000 (17:31 +0000)
commit9dd5d20dd4c701f1393d3bf32c3994b56601494a
tree510c4eba28beca5a4fde338368509c5579345613
parent2c1af1f5e8f64ee4f4a5092263dacb60544dd88e
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