BIND 10 #3015: Change type of IntElement to int64_t
BIND 10 Development
do-not-reply at isc.org
Wed Aug 7 08:24:42 UTC 2013
#3015: Change type of IntElement to int64_t
-------------------------------------+-------------------------------------
Reporter: fujiwara | Owner:
Type: defect | fujiwara
Priority: medium | Status:
Component: Inter-module | reviewing
communication | Milestone:
Keywords: | Sprint-20130820
Sensitive: 0 | Resolution:
Sub-Project: Core | CVSS Scoring:
Estimated Difficulty: 1 | Defect Severity: N/A
Total Hours: 0.90 | Feature Depending on Ticket:
| Add Hours to Ticket: 0
| Internal?: 0
-------------------------------------+-------------------------------------
Changes (by vorner):
* owner: vorner => fujiwara
* totalhours: 0 => 0.90
Comment:
Hello
Replying to [comment:11 fujiwara]:
> The reason I did not use int64_t in some parts is that C++ overloading
and automatic type conversion do not work well for derived types (int64_t
or int32_t).
> Overloading function definition side, we cannot define these five
functions f(int64_t), f(int32_t), f(long long), f(long), f(int) because
some types are the same sizes.
OK, now I see. So it was indeed a decision, not just oversight of
inconsistency.
Can you add a short note about that (being it a decision and hint for the
reason) somewhere near the function definitions, so someone will not try
to „fix“ the inconsistency in future and get himself bitten by compiler?
After that, it can be merged.
--
Ticket URL: <http://bind10.isc.org/ticket/3015#comment:13>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list