Mercurial > hg > octave-jordi
diff libinterp/dldfcn/convhulln.cc @ 20824:66cd994d1b79
eliminate return statements after calls to error
* __delaunayn__.cc, __eigs__.cc, __fltk_uigetfile__.cc, __glpk__.cc,
__init_fltk__.cc, __magick_read__.cc, __osmesa_print__.cc,
__voronoi__.cc, audiodevinfo.cc, audioread.cc, ccolamd.cc, colamd.cc,
convhulln.cc: Eliminate return statements after calls to error.
| author | John W. Eaton <jwe@octave.org> |
|---|---|
| date | Tue, 08 Dec 2015 14:20:05 -0500 (2015-12-08) |
| parents | c2d9556d51d0 |
| children | 1142cf6abc0d |
line wrap: on
line diff
--- a/libinterp/dldfcn/convhulln.cc +++ b/libinterp/dldfcn/convhulln.cc @@ -63,10 +63,7 @@ int maxval = std::numeric_limits<int>::max (); if (dim > maxval || n > maxval) - { - error ("%s: dimension too large for Qhull", who); - return false; - } + error ("%s: dimension too large for Qhull", who); } return true; @@ -147,10 +144,7 @@ options += " " + tmp(i); } else - { - error ("convhulln: OPTIONS must be a string, cell array of strings, or empty"); - return retval; - } + error ("convhulln: OPTIONS must be a string, cell array of strings, or empty"); } boolT ismalloc = false; @@ -168,10 +162,7 @@ if (outfile) frame.add_fcn (close_fcn, outfile); else - { - error ("convhulln: unable to create temporary file for output"); - return retval; - } + error ("convhulln: unable to create temporary file for output"); // qh_new_qhull command and points arguments are not const... @@ -204,11 +195,8 @@ nonsimp_seen = true; if (cmd.find ("QJ") != std::string::npos) - { - // Should never happen with QJ. - error ("convhulln: qhull failed: option 'QJ' returned non-simplicial facet"); - return retval; - } + // Should never happen with QJ. + error ("convhulln: qhull failed: option 'QJ' returned non-simplicial facet"); } if (dim == 3)