BIND 10 #1028: Large memory footprint for b10-xfrin

[BIND 10 Development]

#1028: Large memory footprint for b10-xfrin
-------------------------------------+-------------------------------------
                   Reporter:  shane  |                 Owner:  jinmei
                       Type:         |                Status:  reviewing
  defect                             |             Milestone:
                   Priority:  major  |  Sprint-20111108
                  Component:  xfrin  |            Resolution:
                   Keywords:         |             Sensitive:  0
            Defect Severity:         |           Sub-Project:  DNS
  Medium                             |  Estimated Difficulty:  9
Feature Depending on Ticket:         |           Total Hours:  0
        Add Hours to Ticket:         |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => jinmei


Comment:

 There is another circular reference; !XfrinState.!__state it doesn't
 really appear to cause any problems, but perhaps we should probably unset
 it in the *End() handlers.

 I've fixed one trivial typo in a comment.

 Code looks good.

 Yes, we should move the other increfs too. I do not think that in their
 current uses, there are any cases where parent can be NULL, though IIRC I
 originally wanted to put said functionality into util/python somewhere as
 well (and there could be very valid cases for some other types). I have no
 problems with either suggestion.

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