On Tue, 26 Jul 2011, Henrik Krohns wrote:

On Mon, Jul 25, 2011 at 07:42:59AM -0700, John Hardin wrote:
On Mon, 25 Jul 2011, Kevin A. McGrail wrote:

Because I jumped the gun, for which I apologize.

Err, as best I can tell, you committed to trunk and that's a
commit THEN review procedure.  And you emailed about it on dev.
No harm no foul.

That's what I understood to be the case as well.

In any case, I think it was pointless to remove the patch based on few
comments..  I say put it back and lets discuss it in a bug if someone
doesn't like it. :-)

Michael actually does have a point that it is redundant. The only reason I wrote it is I couldn't remember "HitFreqsRuleTiming" and couldn't find it via Google or searching the sources.

--
 John Hardin KA7OHZ                    http://www.impsec.org/~jhardin/
 [email protected]    FALaholic #11174     pgpk -a [email protected]
 key: 0xB8732E79 -- 2D8C 34F4 6411 F507 136C  AF76 D822 E6E6 B873 2E79
-----------------------------------------------------------------------
  Win95: Where do you want to go today?
  Vista: Where will Microsoft allow you to go today?
-----------------------------------------------------------------------
 10 days until the 276th anniversary of John Peter Zenger's acquittal

Reply via email to