[Bro-Dev] #972: Default arguments for functions
Bro Tracker
bro at tracker.bro.org
Fri May 3 15:57:08 PDT 2013
#972: Default arguments for functions
------------------------------+------------------------
Reporter: robin | Owner:
Type: Feature Request | Status: new
Priority: Low | Milestone: Bro2.2
Component: Bro | Version: git/master
Resolution: | Keywords:
------------------------------+------------------------
Comment (by seth):
> global foo: function(a: string &default="hello");
Oooh, you used an existing idiom even! Nice job.
> # TODO: adding &default should be optional on impl if decl has it?
> function foo(a: string &default="hello")
Yikes, I have no clue on this. I'm inclined to say that it should be ok
for it to be optional.
> # TODO: should be a parse error?
> # (i.e. follow C++ rules that default params must be rightmost ones)
> function baz(a: string &default="err", b: string)
Yes, definitely should be a parse error.
Do these defaults work with events and hooks too? I have no clue if I'd
ever want to use defaults for those, but it seems like it should to keep
things consistent. Although for events it could be a bit weird because
the default value would have to be applied by the calling process so that
the default value is serialized and sent to the remote process as-is. No
--
Ticket URL: <http://tracker.bro.org/bro/ticket/972#comment:1>
Bro Tracker <http://tracker.bro.org/bro>
Bro Issue Tracker
More information about the bro-dev
mailing list