BIND 10 #1258: "preserve order" option for Message::fromWire()

BIND 10 Development do-not-reply at isc.org
Fri Sep 30 12:46:09 UTC 2011


#1258: "preserve order" option for Message::fromWire()
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  jinmei                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:         |  Sprint-20111011
  blocker                            |            Resolution:
                  Component:         |             Sensitive:  0
  libdns++                           |           Sub-Project:  DNS
                   Keywords:         |  Estimated Difficulty:  0
            Defect Severity:  N/A    |           Total Hours:  0
Feature Depending on Ticket:         |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => jinmei


Comment:

 I've pushed a trivial comment change in message.h (and my editor removed a
 few
 spaces in that file)

 Code looks good; just one small comment:

 message_python.cc:688: Shouldn't we just let the normal python error be
 propagated here? It should also specify which types are wanted, albeit
 only
 from the second call. Otherwise, I'd propose to at least add which types
 we
 expect to the error message (just 'invalid arguments' is true but not
 extremely
 helpful).

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


More information about the bind10-tickets mailing list