From e5b3471d579937f19e446f8a380464e0fc059567 Mon Sep 17 00:00:00 2001 From: Kazunori Fujiwara Date: Wed, 7 Aug 2013 18:56:42 +0900 Subject: [3015] Added Notes about the use of int64_t, long long, long and int --- src/lib/cc/data.h | 15 +++++++++++++++ 1 file changed, 15 insertions(+) (limited to 'src/lib/cc') diff --git a/src/lib/cc/data.h b/src/lib/cc/data.h index ca1c5eff65..805060752c 100644 --- a/src/lib/cc/data.h +++ b/src/lib/cc/data.h @@ -166,6 +166,8 @@ public: /// the right type. Set the value and return true if the Elements /// is of the correct type /// + /// Notes: Read notes of IntElement definition about the use of + /// long long int, long int and int. //@{ virtual bool setValue(const long long int v); bool setValue(const long int i) { return (setValue(static_cast(i))); }; @@ -273,6 +275,9 @@ public: /// underlying system). /// (Note that that is different from an NullElement, which /// represents an empty value, and is created with Element::create()) + /// + /// Notes: Read notes of IntElement definition about the use of + /// long long int, long int and int. //@{ static ElementPtr create(); static ElementPtr create(const long long int i); @@ -373,6 +378,16 @@ public: //@} }; +/// Notes: IntElement type is changed to int64_t. +/// Due to C++ problems on overloading and automatic type conversion, +/// (C++ tries to convert integer type values and reference/pointer +/// if value types do not match exactly) +/// We decided the storage as int64_t, +/// three (long long, long, int) override function defintions +/// and cast int/long/long long to int64_t via long long. +/// Therefore, call by value methods (create, setValue) have three +/// (int,long,long long) definitions. Others use int64_t. +/// class IntElement : public Element { int64_t i; private: -- cgit v1.2.3