From 3a2ccb0b4bae955f7a07b52de8b2860de80102e5 Mon Sep 17 00:00:00 2001 From: Han-Wen Nienhuys Date: Wed, 18 Oct 2006 22:42:22 +0000 Subject: [PATCH] *** empty log message *** --- ChangeLog | 2 ++ flower/file-cookie.cc | 2 +- flower/string-convert.cc | 22 ++++++++++++---------- 3 files changed, 15 insertions(+), 11 deletions(-) diff --git a/ChangeLog b/ChangeLog index 74add93852..be1aae5b67 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,7 @@ 2006-10-19 Han-Wen Nienhuys + * flower/string-convert.cc (dec2double): clean-up warnings. + * configure.in (CXXFLAGS): add -Werror. * VERSION (PACKAGE_NAME): release 2.9.25 diff --git a/flower/file-cookie.cc b/flower/file-cookie.cc index 8f4c9c4f85..49b5518f36 100644 --- a/flower/file-cookie.cc +++ b/flower/file-cookie.cc @@ -7,7 +7,7 @@ using namespace std; extern "C" { - static bool + bool is_memory_stream (void *foo) { Memory_out_stream *cookie = (Memory_out_stream *) foo; diff --git a/flower/string-convert.cc b/flower/string-convert.cc index f9449363f0..43f0389bde 100644 --- a/flower/string-convert.cc +++ b/flower/string-convert.cc @@ -74,9 +74,9 @@ String_convert::dec2int (string dec_string) return 0; long l = 0; - int conv = sscanf (dec_string.c_str (), "%ld", &l); - assert (conv); - + if (!sscanf (dec_string.c_str (), "%ld", &l)) + assert (false); + return (int)l; } @@ -94,9 +94,11 @@ String_convert::dec2double (string dec_string) { if (!dec_string.length ()) return 0; - double d = 0; - int conv = sscanf (dec_string.c_str (), "%lf", &d); - assert (conv); + + double d = 0.0; + if (!sscanf (dec_string.c_str (), "%lf", &d)) + assert (false); + return d; } @@ -125,10 +127,10 @@ string String_convert::hex2bin (string hex_string) { string str; - // silly, asserts should alway be "on"! - // assert (!hex2bin (hex_string, str) ); - int error_i = hex2bin (hex_string, str); - assert (!error_i); + + if (hex2bin (hex_string, str)) + assert (false); + return str; } -- 2.39.2