BIND 10 #2266: remove RRsetList class
BIND 10 Development
do-not-reply at isc.org
Wed Oct 10 16:08:08 UTC 2012
#2266: remove RRsetList class
-------------------------------------+-------------------------------------
Reporter: | Owner: jinmei
jinmei | Status: reviewing
Type: | Milestone:
defect | Sprint-20121023
Priority: | Resolution:
medium | Sensitive: 0
Component: | Sub-Project: DNS
libdns++ | Estimated Difficulty: 3
Keywords: | Total Hours: 0
Defect Severity: N/A |
Feature Depending on Ticket: |
Add Hours to Ticket: 0 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by jinmei):
Replying to [comment:7 muks]:
> `ChangeLog` entry:
> {{{
> +XXX. [func] muks
> + The isc::dns::RRsetList class has been removed. It was unused
> + inside the BIND 10 codebase.
> + (Trac #2266, git ...)
> }}}
Just saying "unused" may be weak. In the case of public API, I think
it's possible (in theory) that we publish it even if we don't use it
internally. So I suggest something like this:
{{{
XXX. [func] muks
The isc::dns::RRsetList class has been removed. It was now unused
inside the BIND 10 codebase, and the interface was considered
prone to misuse.
(Trac #2266, git ...)
}}}
> `make distcheck` passes.
Okay, then please merge (unless you want to discuss changelog wording
more).
--
Ticket URL: <http://bind10.isc.org/ticket/2266#comment:8>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list