BIND 10 #2165: update Message::addRRset() to be unaware of signedness

BIND 10 Development do-not-reply at isc.org
Tue Aug 28 20:52:55 UTC 2012


#2165: update Message::addRRset() to be unaware of signedness
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  muks
  jinmei                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:         |  Sprint-20120904
  medium                             |            Resolution:
                  Component:         |             Sensitive:  0
  libdns++                           |           Sub-Project:  DNS
                   Keywords:         |  Estimated Difficulty:  5
            Defect Severity:  N/A    |           Total Hours:  2.91
Feature Depending on Ticket:         |
  scalable inmemory                  |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------

Comment (by jinmei):

 Replying to [comment:16 vorner]:
 > Hello
 >
 > Replying to [comment:15 muks]:
 > > > I think the first one. The message needs it only to render it, why
 would it need a non-const pointer. The second one is obvious why it is
 needed, but probably wrong. Anyway, the second one will be deprecated
 soon, won't it?
 > >
 > > `setTTL()` is eventually called by the `Message` class on the passed
 `RRset`.

 How exactly?  At least in the new API `Message` shouldn't have to
 modify the passed RRsets for responses.  As for setTTL(), I cannot
 even find a code path in `Message` where setTTL() happens in the
 rendering mode.

-- 
Ticket URL: <http://bind10.isc.org/ticket/2165#comment:17>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list