BIND 10 #401: Timouts in the recursor
BIND 10 Development
do-not-reply at isc.org
Mon Nov 22 20:24:24 UTC 2010
#401: Timouts in the recursor
------------------------------+---------------------------------------------
Reporter: vorner | Owner: stephen
Type: enhancement | Status: reviewing
Priority: major | Milestone:
Component: recurser | Resolution:
Keywords: | Sensitive: 0
Estimatedhours: 0.0 | Hours: 0
Billable: 1 | Totalhours: 0
Internal: 0 |
------------------------------+---------------------------------------------
Changes (by vorner):
* owner: vorner => stephen
Comment:
Replying to [comment:5 stephen]:
> > I added comments to them, but not doxygen ones (having interface of
tests in the API
> > documentation does not seem useful).
>
> It's always possible to exclude the test files from the Doxygen
documentation (see item 10 in
http://www.stack.nl/~dimitri/doxygen/faq.html) if we want to. But
otherwise, if you're modifying the code you will need to modify the test
code as well so may well want to see the documentation for it. However,
leave it as is for now.
Well, I understand the doxygen as documentation for people using the code
from outside. For them, the documentation of tests is useless, since tests
aren't part of the public API. On the other hand, for someone modifying
the code the documentation is not enough, such person needs to read the
code. And then there's not much difference between comments directly in
the code and the documentation pulled out to doxygen. The important part
of doxygen ‒ indices and links ‒ aren't needed, as the person knows which
part of code to look into. But then, maybe I'm wrong and I should ask
tomorrow on the call.
> > Everything else should be done now. Do you think it is clean to sync
before merge?
>
> Yes.
It is synced. I added some parts to join the merged changes (configuration
of timeouts, bits of logging). Can you have a look at them?
Thanks
--
Ticket URL: <https://bind10.isc.org/ticket/401#comment:6>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list