diff options
author | arv@chromium.org <arv@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-01 01:02:07 +0000 |
---|---|---|
committer | arv@chromium.org <arv@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-01 01:02:07 +0000 |
commit | fb534c949f92bec8c6312b58ffcc04c5dc488f2d (patch) | |
tree | 84384bdb8b06e62662ce1a5e9619ae455e9a69a8 /base | |
parent | efeb669b05db99c54309771e4884b1a17d604a37 (diff) | |
download | chromium_src-fb534c949f92bec8c6312b58ffcc04c5dc488f2d.zip chromium_src-fb534c949f92bec8c6312b58ffcc04c5dc488f2d.tar.gz chromium_src-fb534c949f92bec8c6312b58ffcc04c5dc488f2d.tar.bz2 |
Rename Real* to Double* in values.* and dependent files
BUG=None
TEST=Compiles and passes all tests
Review URL: http://codereview.chromium.org/6248026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73232 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base')
-rw-r--r-- | base/json/json_reader.cc | 2 | ||||
-rw-r--r-- | base/json/json_reader_unittest.cc | 72 | ||||
-rw-r--r-- | base/json/json_writer.cc | 4 | ||||
-rw-r--r-- | base/json/json_writer_unittest.cc | 6 | ||||
-rw-r--r-- | base/values.cc | 42 | ||||
-rw-r--r-- | base/values.h | 18 | ||||
-rw-r--r-- | base/values_unittest.cc | 38 |
7 files changed, 91 insertions, 91 deletions
diff --git a/base/json/json_reader.cc b/base/json/json_reader.cc index dd5d088..c8fe78d 100644 --- a/base/json/json_reader.cc +++ b/base/json/json_reader.cc @@ -396,7 +396,7 @@ Value* JSONReader::DecodeNumber(const Token& token) { double num_double; if (StringToDouble(WideToUTF8(num_string), &num_double) && base::IsFinite(num_double)) - return Value::CreateRealValue(num_double); + return Value::CreateDoubleValue(num_double); return NULL; } diff --git a/base/json/json_reader_unittest.cc b/base/json/json_reader_unittest.cc index c00c976..db0ab63 100644 --- a/base/json/json_reader_unittest.cc +++ b/base/json/json_reader_unittest.cc @@ -75,60 +75,60 @@ TEST(JSONReaderTest, Reading) { // storage as doubles root.reset(JSONReader().JsonToValue("2147483648", false, false)); ASSERT_TRUE(root.get()); - double real_val; - ASSERT_TRUE(root->IsType(Value::TYPE_REAL)); - real_val = 0.0; - ASSERT_TRUE(root->GetAsReal(&real_val)); - ASSERT_DOUBLE_EQ(2147483648.0, real_val); + double double_val; + ASSERT_TRUE(root->IsType(Value::TYPE_DOUBLE)); + double_val = 0.0; + ASSERT_TRUE(root->GetAsDouble(&double_val)); + ASSERT_DOUBLE_EQ(2147483648.0, double_val); root.reset(JSONReader().JsonToValue("-2147483649", false, false)); ASSERT_TRUE(root.get()); - ASSERT_TRUE(root->IsType(Value::TYPE_REAL)); - real_val = 0.0; - ASSERT_TRUE(root->GetAsReal(&real_val)); - ASSERT_DOUBLE_EQ(-2147483649.0, real_val); + ASSERT_TRUE(root->IsType(Value::TYPE_DOUBLE)); + double_val = 0.0; + ASSERT_TRUE(root->GetAsDouble(&double_val)); + ASSERT_DOUBLE_EQ(-2147483649.0, double_val); // Parse a double root.reset(JSONReader().JsonToValue("43.1", false, false)); ASSERT_TRUE(root.get()); - ASSERT_TRUE(root->IsType(Value::TYPE_REAL)); - real_val = 0.0; - ASSERT_TRUE(root->GetAsReal(&real_val)); - ASSERT_DOUBLE_EQ(43.1, real_val); + ASSERT_TRUE(root->IsType(Value::TYPE_DOUBLE)); + double_val = 0.0; + ASSERT_TRUE(root->GetAsDouble(&double_val)); + ASSERT_DOUBLE_EQ(43.1, double_val); root.reset(JSONReader().JsonToValue("4.3e-1", false, false)); ASSERT_TRUE(root.get()); - ASSERT_TRUE(root->IsType(Value::TYPE_REAL)); - real_val = 0.0; - ASSERT_TRUE(root->GetAsReal(&real_val)); - ASSERT_DOUBLE_EQ(.43, real_val); + ASSERT_TRUE(root->IsType(Value::TYPE_DOUBLE)); + double_val = 0.0; + ASSERT_TRUE(root->GetAsDouble(&double_val)); + ASSERT_DOUBLE_EQ(.43, double_val); root.reset(JSONReader().JsonToValue("2.1e0", false, false)); ASSERT_TRUE(root.get()); - ASSERT_TRUE(root->IsType(Value::TYPE_REAL)); - real_val = 0.0; - ASSERT_TRUE(root->GetAsReal(&real_val)); - ASSERT_DOUBLE_EQ(2.1, real_val); + ASSERT_TRUE(root->IsType(Value::TYPE_DOUBLE)); + double_val = 0.0; + ASSERT_TRUE(root->GetAsDouble(&double_val)); + ASSERT_DOUBLE_EQ(2.1, double_val); root.reset(JSONReader().JsonToValue("2.1e+0001", false, false)); ASSERT_TRUE(root.get()); - ASSERT_TRUE(root->IsType(Value::TYPE_REAL)); - real_val = 0.0; - ASSERT_TRUE(root->GetAsReal(&real_val)); - ASSERT_DOUBLE_EQ(21.0, real_val); + ASSERT_TRUE(root->IsType(Value::TYPE_DOUBLE)); + double_val = 0.0; + ASSERT_TRUE(root->GetAsDouble(&double_val)); + ASSERT_DOUBLE_EQ(21.0, double_val); root.reset(JSONReader().JsonToValue("0.01", false, false)); ASSERT_TRUE(root.get()); - ASSERT_TRUE(root->IsType(Value::TYPE_REAL)); - real_val = 0.0; - ASSERT_TRUE(root->GetAsReal(&real_val)); - ASSERT_DOUBLE_EQ(0.01, real_val); + ASSERT_TRUE(root->IsType(Value::TYPE_DOUBLE)); + double_val = 0.0; + ASSERT_TRUE(root->GetAsDouble(&double_val)); + ASSERT_DOUBLE_EQ(0.01, double_val); root.reset(JSONReader().JsonToValue("1.00", false, false)); ASSERT_TRUE(root.get()); - ASSERT_TRUE(root->IsType(Value::TYPE_REAL)); - real_val = 0.0; - ASSERT_TRUE(root->GetAsReal(&real_val)); - ASSERT_DOUBLE_EQ(1.0, real_val); + ASSERT_TRUE(root->IsType(Value::TYPE_DOUBLE)); + double_val = 0.0; + ASSERT_TRUE(root->GetAsDouble(&double_val)); + ASSERT_DOUBLE_EQ(1.0, double_val); // Fractional parts must have a digit before and after the decimal point. root.reset(JSONReader().JsonToValue("1.", false, false)); @@ -303,9 +303,9 @@ TEST(JSONReaderTest, Reading) { ASSERT_TRUE(root.get()); ASSERT_TRUE(root->IsType(Value::TYPE_DICTIONARY)); DictionaryValue* dict_val = static_cast<DictionaryValue*>(root.get()); - real_val = 0.0; - ASSERT_TRUE(dict_val->GetReal("number", &real_val)); - ASSERT_DOUBLE_EQ(9.87654321, real_val); + double_val = 0.0; + ASSERT_TRUE(dict_val->GetDouble("number", &double_val)); + ASSERT_DOUBLE_EQ(9.87654321, double_val); Value* null_val = NULL; ASSERT_TRUE(dict_val->Get("null", &null_val)); ASSERT_TRUE(null_val->IsType(Value::TYPE_NULL)); diff --git a/base/json/json_writer.cc b/base/json/json_writer.cc index dbf43ec..fa8f239 100644 --- a/base/json/json_writer.cc +++ b/base/json/json_writer.cc @@ -75,10 +75,10 @@ void JSONWriter::BuildJSONString(const Value* const node, break; } - case Value::TYPE_REAL: + case Value::TYPE_DOUBLE: { double value; - bool result = node->GetAsReal(&value); + bool result = node->GetAsDouble(&value); DCHECK(result); std::string real = DoubleToString(value); // Ensure that the number has a .0 if there's no decimal or 'e'. This diff --git a/base/json/json_writer_unittest.cc b/base/json/json_writer_unittest.cc index 937d083..6d7714b 100644 --- a/base/json/json_writer_unittest.cc +++ b/base/json/json_writer_unittest.cc @@ -29,19 +29,19 @@ TEST(JSONWriterTest, Writing) { delete root; // Test Real values should always have a decimal or an 'e'. - root = Value::CreateRealValue(1.0); + root = Value::CreateDoubleValue(1.0); JSONWriter::Write(root, false, &output_js); ASSERT_EQ("1.0", output_js); delete root; // Test Real values in the the range (-1, 1) must have leading zeros - root = Value::CreateRealValue(0.2); + root = Value::CreateDoubleValue(0.2); JSONWriter::Write(root, false, &output_js); ASSERT_EQ("0.2", output_js); delete root; // Test Real values in the the range (-1, 1) must have leading zeros - root = Value::CreateRealValue(-0.8); + root = Value::CreateDoubleValue(-0.8); JSONWriter::Write(root, false, &output_js); ASSERT_EQ("-0.8", output_js); delete root; diff --git a/base/values.cc b/base/values.cc index a4a6ff4..9c96f26 100644 --- a/base/values.cc +++ b/base/values.cc @@ -80,7 +80,7 @@ FundamentalValue* Value::CreateIntegerValue(int in_value) { } // static -FundamentalValue* Value::CreateRealValue(double in_value) { +FundamentalValue* Value::CreateDoubleValue(double in_value) { return new FundamentalValue(in_value); } @@ -107,7 +107,7 @@ bool Value::GetAsInteger(int* out_value) const { return false; } -bool Value::GetAsReal(double* out_value) const { +bool Value::GetAsDouble(double* out_value) const { return false; } @@ -158,7 +158,7 @@ FundamentalValue::FundamentalValue(int in_value) } FundamentalValue::FundamentalValue(double in_value) - : Value(TYPE_REAL), real_value_(in_value) { + : Value(TYPE_DOUBLE), double_value_(in_value) { } FundamentalValue::~FundamentalValue() { @@ -176,10 +176,10 @@ bool FundamentalValue::GetAsInteger(int* out_value) const { return (IsType(TYPE_INTEGER)); } -bool FundamentalValue::GetAsReal(double* out_value) const { - if (out_value && IsType(TYPE_REAL)) - *out_value = real_value_; - return (IsType(TYPE_REAL)); +bool FundamentalValue::GetAsDouble(double* out_value) const { + if (out_value && IsType(TYPE_DOUBLE)) + *out_value = double_value_; + return (IsType(TYPE_DOUBLE)); } FundamentalValue* FundamentalValue::DeepCopy() const { @@ -190,8 +190,8 @@ FundamentalValue* FundamentalValue::DeepCopy() const { case TYPE_INTEGER: return CreateIntegerValue(integer_value_); - case TYPE_REAL: - return CreateRealValue(real_value_); + case TYPE_DOUBLE: + return CreateDoubleValue(double_value_); default: NOTREACHED(); @@ -212,9 +212,9 @@ bool FundamentalValue::Equals(const Value* other) const { int lhs, rhs; return GetAsInteger(&lhs) && other->GetAsInteger(&rhs) && lhs == rhs; } - case TYPE_REAL: { + case TYPE_DOUBLE: { double lhs, rhs; - return GetAsReal(&lhs) && other->GetAsReal(&rhs) && lhs == rhs; + return GetAsDouble(&lhs) && other->GetAsDouble(&rhs) && lhs == rhs; } default: NOTREACHED(); @@ -367,8 +367,8 @@ void DictionaryValue::SetInteger(const std::string& path, int in_value) { Set(path, CreateIntegerValue(in_value)); } -void DictionaryValue::SetReal(const std::string& path, double in_value) { - Set(path, CreateRealValue(in_value)); +void DictionaryValue::SetDouble(const std::string& path, double in_value) { + Set(path, CreateDoubleValue(in_value)); } void DictionaryValue::SetString(const std::string& path, @@ -430,13 +430,13 @@ bool DictionaryValue::GetInteger(const std::string& path, return value->GetAsInteger(out_value); } -bool DictionaryValue::GetReal(const std::string& path, - double* out_value) const { +bool DictionaryValue::GetDouble(const std::string& path, + double* out_value) const { Value* value; if (!Get(path, &value)) return false; - return value->GetAsReal(out_value); + return value->GetAsDouble(out_value); } bool DictionaryValue::GetString(const std::string& path, @@ -533,13 +533,13 @@ bool DictionaryValue::GetIntegerWithoutPathExpansion(const std::string& key, return value->GetAsInteger(out_value); } -bool DictionaryValue::GetRealWithoutPathExpansion(const std::string& key, - double* out_value) const { +bool DictionaryValue::GetDoubleWithoutPathExpansion(const std::string& key, + double* out_value) const { Value* value; if (!GetWithoutPathExpansion(key, &value)) return false; - return value->GetAsReal(out_value); + return value->GetAsDouble(out_value); } bool DictionaryValue::GetStringWithoutPathExpansion( @@ -742,12 +742,12 @@ bool ListValue::GetInteger(size_t index, int* out_value) const { return value->GetAsInteger(out_value); } -bool ListValue::GetReal(size_t index, double* out_value) const { +bool ListValue::GetDouble(size_t index, double* out_value) const { Value* value; if (!Get(index, &value)) return false; - return value->GetAsReal(out_value); + return value->GetAsDouble(out_value); } bool ListValue::GetString(size_t index, std::string* out_value) const { diff --git a/base/values.h b/base/values.h index 135a30f..408ca8b 100644 --- a/base/values.h +++ b/base/values.h @@ -50,7 +50,7 @@ class Value { TYPE_NULL = 0, TYPE_BOOLEAN, TYPE_INTEGER, - TYPE_REAL, + TYPE_DOUBLE, TYPE_STRING, TYPE_BINARY, TYPE_DICTIONARY, @@ -65,7 +65,7 @@ class Value { static Value* CreateNullValue(); static FundamentalValue* CreateBooleanValue(bool in_value); static FundamentalValue* CreateIntegerValue(int in_value); - static FundamentalValue* CreateRealValue(double in_value); + static FundamentalValue* CreateDoubleValue(double in_value); static StringValue* CreateStringValue(const std::string& in_value); static StringValue* CreateStringValue(const string16& in_value); @@ -89,7 +89,7 @@ class Value { // returned; otherwise, false is returned and |out_value| is unchanged. virtual bool GetAsBoolean(bool* out_value) const; virtual bool GetAsInteger(int* out_value) const; - virtual bool GetAsReal(double* out_value) const; + virtual bool GetAsDouble(double* out_value) const; virtual bool GetAsString(std::string* out_value) const; virtual bool GetAsString(string16* out_value) const; virtual bool GetAsList(ListValue** out_value); @@ -132,7 +132,7 @@ class FundamentalValue : public Value { // Subclassed methods virtual bool GetAsBoolean(bool* out_value) const; virtual bool GetAsInteger(int* out_value) const; - virtual bool GetAsReal(double* out_value) const; + virtual bool GetAsDouble(double* out_value) const; virtual FundamentalValue* DeepCopy() const; virtual bool Equals(const Value* other) const; @@ -140,7 +140,7 @@ class FundamentalValue : public Value { union { bool boolean_value_; int integer_value_; - double real_value_; + double double_value_; }; DISALLOW_COPY_AND_ASSIGN(FundamentalValue); @@ -237,7 +237,7 @@ class DictionaryValue : public Value { // value at that path, even if it has a different type. void SetBoolean(const std::string& path, bool in_value); void SetInteger(const std::string& path, int in_value); - void SetReal(const std::string& path, double in_value); + void SetDouble(const std::string& path, double in_value); void SetString(const std::string& path, const std::string& in_value); void SetString(const std::string& path, const string16& in_value); @@ -259,7 +259,7 @@ class DictionaryValue : public Value { // the end of the path can be returned in the form specified. bool GetBoolean(const std::string& path, bool* out_value) const; bool GetInteger(const std::string& path, int* out_value) const; - bool GetReal(const std::string& path, double* out_value) const; + bool GetDouble(const std::string& path, double* out_value) const; bool GetString(const std::string& path, std::string* out_value) const; bool GetString(const std::string& path, string16* out_value) const; bool GetStringASCII(const std::string& path, std::string* out_value) const; @@ -274,7 +274,7 @@ class DictionaryValue : public Value { Value** out_value) const; bool GetIntegerWithoutPathExpansion(const std::string& key, int* out_value) const; - bool GetRealWithoutPathExpansion(const std::string& key, + bool GetDoubleWithoutPathExpansion(const std::string& key, double* out_value) const; bool GetStringWithoutPathExpansion(const std::string& key, std::string* out_value) const; @@ -377,7 +377,7 @@ class ListValue : public Value { // in the specified form. bool GetBoolean(size_t index, bool* out_value) const; bool GetInteger(size_t index, int* out_value) const; - bool GetReal(size_t index, double* out_value) const; + bool GetDouble(size_t index, double* out_value) const; bool GetString(size_t index, std::string* out_value) const; bool GetString(size_t index, string16* out_value) const; bool GetBinary(size_t index, BinaryValue** out_value) const; diff --git a/base/values_unittest.cc b/base/values_unittest.cc index 774dfca..5f901b5 100644 --- a/base/values_unittest.cc +++ b/base/values_unittest.cc @@ -60,7 +60,7 @@ TEST_F(ValuesTest, List) { scoped_ptr<ListValue> mixed_list(new ListValue()); mixed_list->Set(0, Value::CreateBooleanValue(true)); mixed_list->Set(1, Value::CreateIntegerValue(42)); - mixed_list->Set(2, Value::CreateRealValue(88.8)); + mixed_list->Set(2, Value::CreateDoubleValue(88.8)); mixed_list->Set(3, Value::CreateStringValue("foo")); ASSERT_EQ(4u, mixed_list->GetSize()); @@ -74,7 +74,7 @@ TEST_F(ValuesTest, List) { ASSERT_FALSE(mixed_list->GetInteger(0, &int_value)); ASSERT_EQ(0, int_value); - ASSERT_FALSE(mixed_list->GetReal(1, &double_value)); + ASSERT_FALSE(mixed_list->GetDouble(1, &double_value)); ASSERT_EQ(0.0, double_value); ASSERT_FALSE(mixed_list->GetString(2, &string_value)); ASSERT_EQ("", string_value); @@ -85,7 +85,7 @@ TEST_F(ValuesTest, List) { ASSERT_TRUE(bool_value); ASSERT_TRUE(mixed_list->GetInteger(1, &int_value)); ASSERT_EQ(42, int_value); - ASSERT_TRUE(mixed_list->GetReal(2, &double_value)); + ASSERT_TRUE(mixed_list->GetDouble(2, &double_value)); ASSERT_EQ(88.8, double_value); ASSERT_TRUE(mixed_list->GetString(3, &string_value)); ASSERT_EQ("foo", string_value); @@ -327,8 +327,8 @@ TEST_F(ValuesTest, DeepCopy) { original_dict.Set("bool", original_bool); FundamentalValue* original_int = Value::CreateIntegerValue(42); original_dict.Set("int", original_int); - FundamentalValue* original_real = Value::CreateRealValue(3.14); - original_dict.Set("real", original_real); + FundamentalValue* original_double = Value::CreateDoubleValue(3.14); + original_dict.Set("double", original_double); StringValue* original_string = Value::CreateStringValue("hello"); original_dict.Set("string", original_string); StringValue* original_string16 = @@ -375,14 +375,14 @@ TEST_F(ValuesTest, DeepCopy) { ASSERT_TRUE(copy_int->GetAsInteger(©_int_value)); ASSERT_EQ(42, copy_int_value); - Value* copy_real = NULL; - ASSERT_TRUE(copy_dict->Get("real", ©_real)); - ASSERT_TRUE(copy_real); - ASSERT_NE(copy_real, original_real); - ASSERT_TRUE(copy_real->IsType(Value::TYPE_REAL)); - double copy_real_value = 0; - ASSERT_TRUE(copy_real->GetAsReal(©_real_value)); - ASSERT_EQ(3.14, copy_real_value); + Value* copy_double = NULL; + ASSERT_TRUE(copy_dict->Get("double", ©_double)); + ASSERT_TRUE(copy_double); + ASSERT_NE(copy_double, original_double); + ASSERT_TRUE(copy_double->IsType(Value::TYPE_DOUBLE)); + double copy_double_value = 0; + ASSERT_TRUE(copy_double->GetAsDouble(©_double_value)); + ASSERT_EQ(3.14, copy_double_value); Value* copy_string = NULL; ASSERT_TRUE(copy_dict->Get("string", ©_string)); @@ -459,7 +459,7 @@ TEST_F(ValuesTest, Equals) { DictionaryValue dv; dv.SetBoolean("a", false); dv.SetInteger("b", 2); - dv.SetReal("c", 2.5); + dv.SetDouble("c", 2.5); dv.SetString("d1", "string"); dv.SetString("d2", ASCIIToUTF16("http://google.com")); dv.Set("e", Value::CreateNullValue()); @@ -519,8 +519,8 @@ TEST_F(ValuesTest, DeepCopyCovariantReturnTypes) { original_dict.Set("bool", original_bool); FundamentalValue* original_int = Value::CreateIntegerValue(42); original_dict.Set("int", original_int); - FundamentalValue* original_real = Value::CreateRealValue(3.14); - original_dict.Set("real", original_real); + FundamentalValue* original_double = Value::CreateDoubleValue(3.14); + original_dict.Set("double", original_double); StringValue* original_string = Value::CreateStringValue("hello"); original_dict.Set("string", original_string); StringValue* original_string16 = @@ -542,7 +542,7 @@ TEST_F(ValuesTest, DeepCopyCovariantReturnTypes) { Value* original_dict_value = &original_dict; Value* original_bool_value = original_bool; Value* original_int_value = original_int; - Value* original_real_value = original_real; + Value* original_double_value = original_double; Value* original_string_value = original_string; Value* original_string16_value = original_string16; Value* original_binary_value = original_binary; @@ -551,7 +551,7 @@ TEST_F(ValuesTest, DeepCopyCovariantReturnTypes) { scoped_ptr<Value> copy_dict_value(original_dict_value->DeepCopy()); scoped_ptr<Value> copy_bool_value(original_bool_value->DeepCopy()); scoped_ptr<Value> copy_int_value(original_int_value->DeepCopy()); - scoped_ptr<Value> copy_real_value(original_real_value->DeepCopy()); + scoped_ptr<Value> copy_double_value(original_double_value->DeepCopy()); scoped_ptr<Value> copy_string_value(original_string_value->DeepCopy()); scoped_ptr<Value> copy_string16_value(original_string16_value->DeepCopy()); scoped_ptr<Value> copy_binary_value(original_binary_value->DeepCopy()); @@ -560,7 +560,7 @@ TEST_F(ValuesTest, DeepCopyCovariantReturnTypes) { EXPECT_TRUE(original_dict_value->Equals(copy_dict_value.get())); EXPECT_TRUE(original_bool_value->Equals(copy_bool_value.get())); EXPECT_TRUE(original_int_value->Equals(copy_int_value.get())); - EXPECT_TRUE(original_real_value->Equals(copy_real_value.get())); + EXPECT_TRUE(original_double_value->Equals(copy_double_value.get())); EXPECT_TRUE(original_string_value->Equals(copy_string_value.get())); EXPECT_TRUE(original_string16_value->Equals(copy_string16_value.get())); EXPECT_TRUE(original_binary_value->Equals(copy_binary_value.get())); |