BIND 10 #1456: Create diff-normalizer utility functions

[BIND 10 Development]

#1456: Create diff-normalizer utility functions
-------------------------------------+-------------------------------------
                   Reporter:  jelte  |                 Owner:  vorner
                       Type:  task   |                Status:  reviewing
                   Priority:         |             Milestone:
  medium                             |  Sprint-20120529
                  Component:  DDNS   |            Resolution:
                   Keywords:         |             Sensitive:  0
            Defect Severity:  N/A    |           Sub-Project:  DNS
Feature Depending on Ticket:  DDNS   |  Estimated Difficulty:  5
        Add Hours to Ticket:  0      |           Total Hours:  0
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => vorner


Comment:

 Replying to [comment:11 vorner]:
 >
 > I noticed one thing. If the same RR is both added and deleted, the
 diff/journal will contain both the addition and deletion. While it is OK
 for DDNS, the proposed use of the single-update mode was to support IXFR
 from differences too. That one would probably work by deleting everything
 and then adding everything that comes. Most of the zone could be the same,
 so this would eliminate the changes. I think we should support this
 feature in the diff ‒ not necessarily now in this ticket, but a new one to
 keep track of it should be created.
 >

 Ack, created #2008.

 > Also, I have few minor comments:
 >  * The docstring of `diff.__init__` says that „The first addition and
 the first addition still…“ ‒ the second should be deletion.

 fixed

 >  * The docstring of `diff.get_single_update_buffers` is not true, the
 example output it shows has wrong structure.

 ack, fixed

 >
 > I think it should be put into a function (something like
 `addWithSoaCheck`) and called as:
 > {{{#!python
 > self.__addWithSoaCheck(self.__additions, "addition", operation, rr)
 > }}}

 Ack, done, !__append_with_soa_check(). (reused the already set operation
 string in the error, whether it's 'addition' or 'add' doesn't seem to
 matter that much :))

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