BIND 10 #2100: Update ZoneTable class
BIND 10 Development
do-not-reply at isc.org
Fri Aug 24 23:06:54 UTC 2012
#2100: Update ZoneTable class
-------------------------------------+-------------------------------------
Reporter: | Owner: jinmei
jinmei | Status: reviewing
Type: task | Milestone:
Priority: | Sprint-20120904
medium | Resolution:
Component: data | Sensitive: 0
source | Sub-Project: DNS
Keywords: | Estimated Difficulty: 4
Defect Severity: N/A | Total Hours: 0
Feature Depending on Ticket: |
scalable inmemory |
Add Hours to Ticket: 0 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by jinmei):
Replying to [comment:12 jelte]:
> due to the nature of these buffers, there will probably always be ways
to shoot yourself in the foot :)
Right, and although it might be a bit more likely happen in this case
"due to the nature", but in some sense it's no different from things
like this:
{{{#!cpp
Foo foo; // this is a very complicated class, ctor does some black
magic
memset(&foo, 0, sizeof(foo)); // but it's broken here, compiler allows
it
}}}
so, I think it's reasonable to at least prevent the self-disruption by
the object's own method (and in particular, by a const member
function).
> however, i think this is a reasonable compromise, and the code looks ok,
so please include it, and merge this branch.
Okay, thanks, merge done, closing.
--
Ticket URL: <http://bind10.isc.org/ticket/2100#comment:13>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list