[Bro-Dev] #513: topic/jsiwek/update-restdoc-target
Bro Tracker
bro at tracker.bro-ids.org
Sun Jul 24 15:38:43 PDT 2011
#513: topic/jsiwek/update-restdoc-target
----------------------------+------------------------
Reporter: jsiwek | Owner: jsiwek
Type: Merge Request | Status: assigned
Priority: Normal | Milestone: Bro1.6
Component: Bro | Version: git/master
Resolution: | Keywords:
----------------------------+------------------------
Changes (by robin):
* owner: => jsiwek
* status: new => assigned
Comment:
Questions about some of the script changes in here:
- you added many @loads. Are they necessary only so that the doc
generation works and not otherwise because normally the scripts would be
pulled in implicitly? If indeed so, this is not so great: we will probably
run into the problem all the time because it will only show up when docs
are generated. However, I don't have a good idea right now how to address
that.
- The change from ProtocolFound to Protocol_Found: is that according to
our new naming conventions?
- One chunk adds a commit `#This should be optional`: why isn't it?
--
Ticket URL: <http://tracker.bro-ids.org/bro/ticket/513#comment:1>
Bro Tracker <http://tracker.bro-ids.org/bro>
Bro Issue Tracker
More information about the bro-dev
mailing list