# \[Bro-Commits\] \[git/bro\] fastpath: sorry. the patch for the set\_separator. (6bf733c)

**URL:** <https://community.zeek.org/t/bro-commits-git-bro-fastpath-sorry-the-patch-for-the-set-separator-6bf733c/2421>\
**Category:** Development\
**Tags:** development\
**Created:** [August 27, 2012, 4:25pm UTC](https://community.zeek.org/t/bro-commits-git-bro-fastpath-sorry-the-patch-for-the-set-separator-6bf733c/2421 "2012-08-27T16:25:36Z")\
**Posts on this page:** 5\
**Page:** 1

<div class="post-metadata">

**Author:** ![robin](https://yyz1.discourse-cdn.com/flex011/user_avatar/community.zeek.org/robin/32/599_2.png) [@robin](https://community.zeek.org/u/robin)\
**Post date:** [August 27, 2012, 4:25pm UTC](https://community.zeek.org/t/bro-commits-git-bro-fastpath-sorry-the-patch-for-the-set-separator-6bf733c/2421/1 "2012-08-27T16:25:36Z")

</div>

Looks like it doesn't support multiple-character set separators. Does  
the code check if the user specifies one with length \> 0?

Robin

---

<div class="post-metadata">

**Author:** ![johanna](https://avatars.discourse-cdn.com/v4/letter/j/50afbb/32.png) [@johanna](https://community.zeek.org/u/johanna)\
**Post date:** [August 27, 2012, 4:40pm UTC](https://community.zeek.org/t/bro-commits-git-bro-fastpath-sorry-the-patch-for-the-set-separator-6bf733c/2421/2 "2012-08-27T16:40:50Z")

</div>

Not really… if is mentioned in the documentation / in input/readers/ascii.bro  
for each of the separators.

Johanna

---

<div class="post-metadata">

**Author:** ![robin](https://yyz1.discourse-cdn.com/flex011/user_avatar/community.zeek.org/robin/32/599_2.png) [@robin](https://community.zeek.org/u/robin)\
**Post date:** [August 27, 2012, 4:53pm UTC](https://community.zeek.org/t/bro-commits-git-bro-fastpath-sorry-the-patch-for-the-set-separator-6bf733c/2421/3 "2012-08-27T16:53:37Z")

</div>

Can you add that check? Seems like an easy mistake to make, and it  
sounds like it's silently doing the wrong thing in that case at the  
moment.

Robin

---

<div class="post-metadata">

**Author:** ![johanna](https://avatars.discourse-cdn.com/v4/letter/j/50afbb/32.png) [@johanna](https://community.zeek.org/u/johanna)\
**Post date:** [August 27, 2012, 5:47pm UTC](https://community.zeek.org/t/bro-commits-git-bro-fastpath-sorry-the-patch-for-the-set-separator-6bf733c/2421/4 "2012-08-27T17:47:39Z")

</div>

Actually I was mistaken. It does check and it is throwing an Error.  
I really should not trust my memory - I keep forgetting which features  
I already support...

Johanna

---

<div class="post-metadata">

**Author:** ![system](https://canada1.discourse-cdn.com/flex011/uploads/zeek/original/1X/f09d732bc2cc7c7cc7e35db67cf4e1d5233ce7a7.png) [@system](https://community.zeek.org/u/system)\
**Post date:** [May 6, 2022, 3:40pm UTC](https://community.zeek.org/t/bro-commits-git-bro-fastpath-sorry-the-patch-for-the-set-separator-6bf733c/2421/5 "2022-05-06T15:40:32Z")

</div>