[pLog-svn] r6088 - plog/branches/lifetype-1.2/class/security

Mark Wu markplace at gmail.com
Thu Nov 29 09:09:35 EST 2007


>  	Well, hopefully Paul will chime in here.  I am pretty 
> sure I understand what you did.  Did you read Paul's (I 
> assume it was Paul) comment about why he added the second 
> loop?  I think we need to run the entire pipeline twice, 
> particularly those filters that ran the first time.

Yep, I read his comments about add second run , that why I think the change
is okay for us. Becasue I can make sure all pipeline filters run once. No
more no less :D

Hope he can give me some advice, but seems he did not show up in this
mailling for a long time.

>  	The thing that we need to have is the "executed" flag 
> passed into each filter, and then each filter can decide 
> whether it wants to run twice or not.

Yes, this is what I did in rev. 6088.  

After review the whole pipeline class, I decide to combine these two
for-loop together in rev. 6089, It makes the code easier to understand and
"less" code.

BTW, did you test it yet? Does it work for you?

Mark



More information about the pLog-svn mailing list