Re: [sr-dev] git:master: modules:sipcapture: fixed mirroring mode, added BPF functionality.

Hello, I see you define static variables in header file, they will be cloned in all files where the header file is included. The header file should contain ususally the prototype for functions, defines and definitions of structures. Variables should be defined in .c files, to avoid duplicates. If needed to be accessed from other files, they can be declared again as extern in header file (not for static vars, though). Maybe this time the header file is included only in one .c file, but still, the variables should be moved in .c file to be safe in the future. On a different subject, just wanted to tell that instead of: if(method =~ "^(INVITE|UPDATE|NOTIFY|SUBSCRIBE|OPTIONS|REGISTER)") you can use: if(is_method("INVITE|UPDATE|NOTIFY|SUBSCRIBE|OPTIONS|REGISTER")) it is faster and safer in the case someone sends a new method like INVITEXYZ, the function is part of textops module. Cheers, Daniel On 9/29/11 2:52 PM, Alexandr Dubovikov wrote: -- Daniel-Constantin Mierla -- http://www.asipto.com Kamailio Advanced Training, Oct 10-13, Berlin: http://asipto.com/u/kat http://linkedin.com/in/miconda -- http://twitter.com/miconda