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