BIND 10 #2391: support generic version of rdata::createRdata(text) in AFSDB, DLV RDATA
BIND 10 Development
do-not-reply at isc.org
Mon Mar 25 12:48:33 UTC 2013
#2391: support generic version of rdata::createRdata(text) in AFSDB, DLV RDATA
-------------------------------------+-------------------------------------
Reporter: jinmei | Owner:
Type: task | pselkirk
Priority: medium | Status:
Component: libdns++ | reviewing
Keywords: | Milestone:
Sensitive: 0 | Sprint-20130402
Sub-Project: DNS | Resolution:
Estimated Difficulty: 3 | CVSS Scoring:
Total Hours: 0 | Defect Severity: N/A
| Feature Depending on Ticket:
| loadzone-ng
| Add Hours to Ticket: 0
| Internal?: 0
-------------------------------------+-------------------------------------
Changes (by vorner):
* owner: vorner => pselkirk
Comment:
Hello
Few notes:
Some of the changes contain tabs instead of spaces.
The doxygen of the new constructor seems to be a copy-paste from MX. It
talk about Exchange name, for example, which doesn't seem to be mentioned
at all by the AFSDB rfc.
Also, the tests seem to crash for me:
{{{
[ RUN ] Rdata_AFSDB_Test.createFromLexer
run_unittests: /usr/include/boost/smart_ptr/shared_ptr.hpp:418:
boost::shared_ptr<T>::reference boost::shared_ptr<T>::operator*() const
[with T = isc::dns::rdata::Rdata; boost::shared_ptr<T>::reference =
isc::dns::rdata::Rdata&]: Assertion `px != 0' failed.
/bin/bash: line 5: 3315 Aborted (core dumped) /bin/bash
../../../../libtool --mode=execute ${dir}$tst
FAIL: run_unittests
===================================
}}}
Anyway, applying similar changes to MX might be a good idea, so some code
duplication is reduced.
--
Ticket URL: <http://bind10.isc.org/ticket/2391#comment:9>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list