]> www.ginac.de Git - cln.git/blobdiff - src/complex/input/cl_N_read.cc
Avoid "this statement may fall through" warnings.
[cln.git] / src / complex / input / cl_N_read.cc
index 21c839674158deb04693d55450d8bdbc2da06137..9365972a5d71227fbc1c49c7dd936681fa339353 100644 (file)
@@ -1,30 +1,34 @@
 // read_complex().
 
 // General includes.
-#include "cl_sysdep.h"
+#include "base/cl_sysdep.h"
 
 // Specification.
-#include "cl_complex_io.h"
+#include "cln/complex_io.h"
 
 
 // Implementation.
 
-#include <string.h>
-#include "cl_input.h"
-#include "cl_real_io.h"
-#include "cl_float_io.h"
-#include "cl_rational_io.h"
-#include "cl_integer_io.h"
-#include "cl_integer.h"
-#include "cl_I.h"
-#include "cl_F.h"
-#include "cl_C.h"
-#include "cl_abort.h"
+#include <cstring>
+#include <sstream>
+#include "cln/input.h"
+#include "cln/real_io.h"
+#include "cln/float_io.h"
+#include "cln/rational_io.h"
+#include "cln/integer_io.h"
+#include "cln/integer.h"
+#include "integer/cl_I.h"
+#include "float/cl_F.h"
+#include "complex/cl_C.h"
+#include "cln/exception.h"
 
 #undef floor
-#include <math.h>
+#include <cmath>
 #define floor cln_floor
 
+
+namespace cln {
+
 // Step forward over all digits, to the end of string or to the next non-digit.
 static const char * skip_digits (const char * ptr, const char * string_limit, unsigned int base)
 {
@@ -56,13 +60,13 @@ static const cl_N read_complex_number_rest (const cl_read_flags& flags, const ch
   if (end_of_parse)                                                    \
     { *end_of_parse = (ptr); }                                         \
   else                                                                 \
-    { if ((ptr) != string_limit) { read_number_junk((ptr),string,string_limit); } }
+    { if ((ptr) != string_limit) { throw read_number_junk_exception((ptr),string,string_limit); } }
 
 const cl_N read_complex (const cl_read_flags& flags, const char * string, const char * string_limit, const char * * end_of_parse)
 {
        // If no string_limit is given, it defaults to the end of the string.
        if (!string_limit)
-               string_limit = string + strlen(string);
+               string_limit = string + ::strlen(string);
        if (flags.syntax & syntax_rational) {
                // Check for rational number syntax.
                var unsigned int rational_base = flags.rational_base;
@@ -89,12 +93,12 @@ const cl_N read_complex (const cl_read_flags& flags, const char * string, const
                                                goto not_rational_syntax;
                                        var cl_I base = read_integer(10,0,ptr,0,base_end_ptr-ptr);
                                        if (!((base >= 2) && (base <= 36))) {
-                                               fprint(cl_stderr, "Base must be an integer in the range from 2 to 36, not ");
-                                               fprint(cl_stderr, base);
-                                               fprint(cl_stderr, "\n");
-                                               cl_abort();
+                                               std::ostringstream buf;
+                                               fprint(buf, "Base must be an integer in the range from 2 to 36, not ");
+                                               fprint(buf, base);
+                                               throw runtime_exception(buf.str());
                                        }
-                                       rational_base = FN_to_UL(base); ptr = base_end_ptr;
+                                       rational_base = FN_to_UV(base); ptr = base_end_ptr;
                                        break;
                                }
                                ptr++;
@@ -104,7 +108,7 @@ const cl_N read_complex (const cl_read_flags& flags, const char * string, const
                var cl_signean sign = 0;
                if (ptr == string_limit) goto not_rational_syntax;
                switch (*ptr) {
-                       case '-': sign = ~sign;
+                       case '-': sign = ~sign; // fallthrough
                        case '+': ptr++;
                        default: break;
                }
@@ -154,35 +158,33 @@ not_rational_syntax:
                var cl_signean sign = 0;
                if (ptr == string_limit) goto not_float_syntax;
                switch (*ptr) {
-                       case '-': sign = ~sign;
+                       case '-': sign = ~sign; // fallthrough
                        case '+': ptr++;
                        default: break;
                }
                var const char * ptr_after_sign = ptr;
                var const char * ptr_after_intpart = skip_digits(ptr_after_sign,string_limit,float_base);
-               var cl_boolean have_dot = cl_false;
                var const char * ptr_before_fracpart = ptr_after_intpart;
                var const char * ptr_after_fracpart = ptr_after_intpart;
                ptr = ptr_after_intpart;
                if (ptr != string_limit)
                  if (*ptr == '.') {
-                       have_dot = cl_true;
                        ptr_before_fracpart = ptr+1;
                        ptr_after_fracpart = skip_digits(ptr_before_fracpart,string_limit,float_base);
                }
                ptr = ptr_after_fracpart;
                var char exponent_marker;
-               var cl_boolean have_exponent;
+               var bool have_exponent;
                var const char * ptr_in_exponent = ptr;
                var const char * ptr_after_exponent = ptr;
                if ((ptr == string_limit) || !(((*ptr >= '0') && (*ptr <= '9')) || ((*ptr >= 'A') && (*ptr <= 'Z') && (*ptr != 'I')) || ((*ptr >= 'a') && (*ptr <= 'z') && (*ptr != 'i')) || (*ptr == '.') || (*ptr == '/'))) {
                        // No exponent.
-                       have_exponent = cl_false;
+                       have_exponent = false;
                        // Must have at least one fractional part digit.
                        if (ptr_after_fracpart == ptr_before_fracpart) goto not_float_syntax;
                        exponent_marker = 'E';
                } else {
-                       have_exponent = cl_true;
+                       have_exponent = true;
                        // Must have at least one digit.
                        if (ptr_after_sign == ptr_after_intpart)
                                if (ptr_after_fracpart == ptr_before_fracpart)
@@ -210,21 +212,21 @@ not_rational_syntax:
                }
                ptr = ptr_after_exponent;
                var const char * ptr_after_prec = ptr;
-               var cl_float_format_t prec;
+               var float_format_t prec;
                if ((ptr != string_limit) && (*ptr == '_')) {
                        ptr++;
                        ptr_after_prec = skip_digits(ptr,string_limit,10);
                        if (ptr_after_prec == ptr) goto not_float_syntax;
                        var cl_I prec1 = digits_to_I(ptr,ptr_after_prec-ptr,10);
-                       var uintL prec2 = cl_I_to_UL(prec1);
-                       prec = (float_base==10 ? cl_float_format(prec2)
-                                              : (cl_float_format_t)((uintL)((1+prec2)*log((double)float_base)*1.442695041)+1)
+                       var uintC prec2 = cl_I_to_ulong(prec1);
+                       prec = (float_base==10 ? float_format(prec2)
+                                              : (float_format_t)((uintC)((1+prec2)*::log((double)float_base)*1.442695041)+1)
                               );
                } else {
                        switch (exponent_marker) {
-                               case 'S': prec = cl_float_format_sfloat; break;
-                               case 'F': prec = cl_float_format_ffloat; break;
-                               case 'D': prec = cl_float_format_dfloat; break;
+                               case 'S': prec = float_format_sfloat; break;
+                               case 'F': prec = float_format_ffloat; break;
+                               case 'D': prec = float_format_dfloat; break;
                                case 'L': prec = flags.float_flags.default_lfloat_format; break;
                                case 'E': prec = flags.float_flags.default_float_format; break;
                                default: NOTREACHED
@@ -233,14 +235,14 @@ not_rational_syntax:
                                // Count the number of significant digits.
                                ptr = ptr_after_sign;
                                while (ptr < ptr_after_fracpart && (*ptr == '0' || *ptr == '.')) ptr++;
-                               var uintL num_significant_digits =
+                               var uintC num_significant_digits =
                                  (ptr_after_fracpart - ptr) - (ptr_before_fracpart > ptr ? 1 : 0);
-                               var uintL prec2 = (num_significant_digits>=2 ? num_significant_digits-2 : 0);
-                               var cl_float_format_t precx =
-                                 (float_base==10 ? cl_float_format(prec2)
-                                                 : (cl_float_format_t)((uintL)((1+prec2)*log((double)float_base)*1.442695041)+1)
+                               var uintC prec2 = (num_significant_digits>=2 ? num_significant_digits-2 : 0);
+                               var float_format_t precx =
+                                 (float_base==10 ? float_format(prec2)
+                                                 : (float_format_t)((uintC)((1+prec2)*::log((double)float_base)*1.442695041)+1)
                                  );
-                               if ((uintL)precx > (uintL)prec)
+                               if ((uintC)precx > (uintC)prec)
                                        prec = precx;
                        }
                }
@@ -289,13 +291,12 @@ not_float_syntax:
                return complex(realpart,imagpart);
        }
 not_complex_syntax:
-bad_syntax:
        if (flags.syntax & syntax_maybe_bad) {
                ASSERT(end_of_parse);
                *end_of_parse = string;
                return 0; // dummy return
        }
-       read_number_bad_syntax(string,string_limit);
+       throw read_number_bad_syntax_exception(string,string_limit);
 }
 
 static const cl_N read_complex_number_rest (const cl_read_flags& flags, const char * string_rest, const char * string, const char * string_limit, const char * * end_of_parse, const cl_R& x)
@@ -315,7 +316,7 @@ static const cl_N read_complex_number_rest (const cl_read_flags& flags, const ch
                        return complex(0,x);
                }
                switch (*ptr) {
-                       case '+': ptr++;
+                       case '+': ptr++; // fallthrough
                        case '-': break;
                        default: goto not_complex_syntax;
                }
@@ -337,3 +338,5 @@ not_complex_syntax:
        at_end_of_parse(string_rest);
        return x;
 }
+
+}  // namespace cln