[Bro-Dev] #825: topic/dnthayer/bif-tests

[Bro Tracker]

#825: topic/dnthayer/bif-tests
----------------------------+------------------------
  Reporter:  dnthayer       |      Owner:
      Type:  Merge Request  |     Status:  new
  Priority:  Normal         |  Milestone:  Bro2.1
 Component:  Bro            |    Version:  git/master
Resolution:                 |   Keywords:
----------------------------+------------------------

Comment (by dnthayer):

 Replying to [comment:2 robin]:
 >
 > - type_name(): I've extended the vector description. For files, you're
 right, but I'm leaving that as is for now; the right fix would be to
 remove the file's type argument altogether (I don't think it's used for
 anything).
 >
 > - fmt(): I've removed documentation for that, seems like an odd use case
 anyways. Also remove the stringn check in the code; that's already taken
 care of elsewhere.

 OK, thanks.

 > - order(): the output seems to be correct to me; what doesn't make sense
 to you?

 I looked at it again, and I think I understand it now.
 The problem is the documentation is not clear (I will improve that).

-- 
Ticket URL: <http://tracker.bro-ids.org/bro/ticket/825#comment:5>
Bro Tracker <http://tracker.bro-ids.org/bro>
Bro Issue Tracker