[Bro-Dev] IP6 addresses

[Gregor Maier]

>> I wouldn't do any compile time flags. That's what we have right now
>> and it isn't pretty at all.
> 
> I probably wasn't clear, so restating in case:  I meant that I agreed with getting rid of the compile-time flag to toggle IPv6 support and just supporting it by default.  But a potential new compile-time flag could be offered to let users choose between the two different approaches Robin was talking about if we can't decide on one that's clearly better (CPU vs. memory tradeoff).  i.e. maybe it has to be implemented and profiled both ways anyway before a decision is made.

Yeah. That's how I understood it. My concern is that we replace one
compile time flag with another. While we would have IPv6 support for all
builds then, we still have #ifdefs in the code and two different code
paths that need to be tested and maintained. So I don't see a real
improvement over the current situation there.

OTOH, I agrees that we might want to / have to implement both approach
and profile them to see what kind of overhead we'll actually have.


cu
gregor
-- 
Gregor Maier
<gregor at icir.org>  <gregor at icsi.berkeley.edu>
Int. Computer Science Institute (ICSI)
1947 Center St., Ste. 600
Berkeley, CA 94704, USA
http://www.icir.org/gregor/