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