BIND 10 #806: Implement RP record type
BIND 10 Development
do-not-reply at isc.org
Mon Apr 11 18:21:23 UTC 2011
#806: Implement RP record type
-------------------------------------+-------------------------------------
Reporter: stephen | Owner: jinmei
Type: | Status: reviewing
enhancement | Milestone:
Priority: | Sprint-20110419
critical | Resolution:
Component: | Sensitive: 0
b10-auth | Add Hours to Ticket: 0
Keywords: | Total Hours: 0
Estimated Number of Hours: 3.0 |
Billable?: 1 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by jinmei):
Replying to [comment:9 vorner]:
> > Hmm, it's strange that it didn't cause an error with clang++ (since we
cannot use the default copy constructor for the base Rdata class we should
define it explicitly). Anyway, the simple and correct fix to this is to
define the trivial copy constructor. Done in commit ad04d07.
>
> Maybe there was no need to copy it anywhere? But then, why g++ did fail?
Well, no matter.
Actually, when I experimentally removed the declaration of the copy
constructor the compiler complained. But it's probably not worth pursuing
at least this time. I've simply merged the latest code to master. Will
close ticket.
Thanks for the review.
--
Ticket URL: <http://bind10.isc.org/ticket/806#comment:10>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list