Re: [ulogd RFC PATCH 0/2] introduce new string output plugin

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

On Sat, 2014-03-29 at 13:23 +0900, Ken-ichirou MATSUZAWA wrote:
>   Hello,
>  
> I started to add new output plugin which work with graphite or statsd. Would you
> review these patches? I think this SPRINT output plugin has two issues and may
> I ask?
>  
> * multiple files for one plugin
>   I use flex and bison and need to separate source files. Is it acceptable?

I'm a bit reluctant to use flex and bison for that. How complicated
would it be to use a hand-made parsing here ?
 
> * use double calc operation
>   To tell statsd total amount bytes, I introduce calc operation (but '+' only now)
>   not using correct ULOGD_RET_ type but regard all of them as double.
>   It seems it is enough as of my purpose.

BR,
-- 
Eric Leblond <eric@xxxxxxxxx>
Blog: https://home.regit.org/

--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux