On Mon, Feb 09, 2004 at 06:02:27PM -0800, Scott wrote:Thats Fine.
There was a problem with this patch, which unlikely got noticed due to the fact that it would not have caused an error until you used the ReadWrite wheel with a depricated filter object. The test suite does not check this behavior, so all tests and all code I ran through passed, but never the less there would have been a problem given the right circumstances.
Attached is a patch for the readwrite wheel (post applying the patches previously sent to the mailing list) and a fixed wheel's patch, incase you haven't yet applied these patches. Any further feedback would be apprecicated.
I'm withholding comment until the list at large has had a chance to speak. It was suggested to me that as Grand Poebah my opinion might carry extra weight. Giving it too eagerly might therefore squelch other opinions, and I agree that would be bad.
In IRC I asked you to post your patch here for a couple reasons.
First, the list is archived. Your patches are now saved and browsable by everyone... FOREVER! MUAH HA HA HA HA! The same goes for any design discussion and decisions that arise from it.
Second, IRC people are generally more accepting of mailing lists than list-centric people are of IRC. You should therefore have a larger audience here, consisting of both list- and IRC-centric people. That should lead to more and better feedback.
That's the theory, anyway. I think it's too early to tell if it works. You posted your patch on a weekend, and weekends don't seem to count for much in mailing list time.
However, I just want to know if anyone applied the patch, had problems applying the patch or running the tests afterwards, or any such thing.
I think a $_[SESSION]->call() is a great idea for a number of reasons,
offers an interface to high level event routing for component writers should
they want to do so, offers an optimization for the wheel implementations
(Although the wheel things have been argued to suck for a variety of reasons,
no reason to not make them faster), and a yield-like convienent intra-session
interface for call ($_[SESSION]->call(state => @args) - obviously).
But again, I'm just really itching to know if anyone had any luck with these patches...c'mon I'm looking for feedback here people.!
- Scott
