BIND 10 #2432: define and implement base and libdns++ version of RRsetCollection

BIND 10 Development do-not-reply at isc.org
Wed Jan 2 15:51:01 UTC 2013


#2432: define and implement base and libdns++ version of RRsetCollection
-------------------------------------+-------------------------------------
            Reporter:  jinmei        |                        Owner:
                Type:  task          |  vorner
            Priority:  medium        |                       Status:
           Component:  libdns++      |  reviewing
            Keywords:                |                    Milestone:
           Sensitive:  0             |  Sprint-20130108
         Sub-Project:  DNS           |                   Resolution:
Estimated Difficulty:  5             |                 CVSS Scoring:
         Total Hours:  0             |              Defect Severity:  N/A
                                     |  Feature Depending on Ticket:
                                     |  loadzone-ng
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------

Comment (by jinmei):

 Replying to [comment:20 muks]:

 > Documentation added. For renaming to `Iterator`, it seems that for
 collections with `std::iterator`s, the preferred naming is
 `CollectionType::iterator` which is why it was named such. If you still
 think this should be changed to `Iterator`, please tell me and I'll update
 it.

 What's the source of the preference?  But, in any event, I'd leave it
 to you two - I don't have a strong opinion.  But if the result is to
 keep "iterator" (lower cased), please document why we don't follow our
 style guideline here.  In fact, I guessed it might be related to the
 fact that std iterator classes are named "iterator", but it's not
 clear from the naming itself.

 > > - I'd explain what kind of collection is constructed by the default
 constructor.
 >
 > I did not understand this.

 I meant describing the initial state of the collection if it's
 constructed by the default constructor (there's no RR, etc).

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


More information about the bind10-tickets mailing list