BIND 10 #904: gen-wiredata.py need more documentation.

[BIND 10 Development]

#904: gen-wiredata.py need more documentation.
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  jinmei                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:  major  |  Sprint-20110816
                  Component:         |            Resolution:
  documentation                      |             Sensitive:  0
                   Keywords:         |           Sub-Project:  DNS
            Defect Severity:  N/A    |  Estimated Difficulty:  0.0
Feature Depending on Ticket:         |           Total Hours:  0
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------

Comment (by jinmei):

 Replying to [comment:9 vorner]:

 > First, it seems git doesn't contain the
 `src/lib/util/python/Makefile.am` file.

 Oops, sorry for the mess.  Added.

 > Otherwise, I'm not sure if referencing the default values in code is OK.
 Do we generate something from it? If so, wouldn't they be missing?

 I don't understand the comment...do you mean you don't like to
 hardcode the default values in the source code like this?

 {{{
 class A(RR):
 [...]
     RDLEN_DEFAULT = 4           # fixed by default
     address = '192.0.2.1'
 }}}

 If so, I don't have a very strong opinion, and although I normally
 don't like any hardcoding, I think this case is acceptable (or perhaps
 even useful), partly because it's just a test utility so conciseness
 would be preferred over full generality/flexibility, and partly
 because the hardcoded values will be shown via pydoc.

 If you mean something else, please clarify.

-- 
Ticket URL: <https://bind10.isc.org/ticket/904#comment:10>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development