BIND 10 #1502: xfrin.diff module should distinguish RRSIGs for different covered

[BIND 10 Development]

#1502: xfrin.diff module should distinguish RRSIGs for different covered
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  jinmei                             |                Status:  reviewing
                       Type:         |             Milestone:
  defect                             |  Sprint-20120110
                   Priority:         |            Resolution:
  critical                           |             Sensitive:  0
                  Component:  xfrin  |           Sub-Project:  DNS
                   Keywords:         |  Estimated Difficulty:  5
            Defect Severity:  N/A    |           Total Hours:  0
Feature Depending on Ticket:  none   |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------

Comment (by jinmei):

 Replying to [comment:7 vorner]:

 > I believe the code does what it should. However, the logic of same_type
 seems hard to follow. May I propose replacing the first two conditions
 with something like this?
 > {{{#!python
 >       if rrset1.get_type() != isc.dns.RRType.RRSIG() or \
 >               rrset2.get_type != isc.dns.RRType.RRSIG():
 >               return rrset1.get_type() == rrset2.get_type()
 > }}}

 Okay, applied.  I guess it's now ready for merge based on the tone of
 the comment, but will give the ticket back to you just in case.

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