I left comments, not sure if it helps much though :)<br><br><div class="gmail_quote">On Tue, May 31, 2011 at 22:42, Peter Royle <span dir="ltr"><<a href="mailto:howardmoon@screamingcoder.com">howardmoon@screamingcoder.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word"><div>No worries thanks for taking a look Jason. Here's the specific commits for those interested in commenting:</div>
<div><br></div><div><a href="https://github.com/seam/cron/commit/61a516dffee906b68e111770f0f26d4f2814d027" target="_blank">https://github.com/seam/cron/commit/61a516dffee906b68e111770f0f26d4f2814d027</a></div><div><br></div>
<font color="#888888"><div>Pete</div></font><div><div></div><div class="h5"><div><br></div><div><br><div><div>On 01/06/2011, at 2:37 PM, Jason Porter wrote:</div><br><blockquote type="cite">Either it's too late to review this right now, or I'm not the best to review it. Could be both, also, could you list the commits to look at so those that do review it can add comments via GitHub?<br>
<br><div class="gmail_quote">
On Thu, May 26, 2011 at 00:36, Pete Royle <span dir="ltr"><<a href="mailto:howardmoon@screamingcoder.com" target="_blank">howardmoon@screamingcoder.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
Speaking of code reviews. I've just made @Asynchronous portable using an<br>
adaptation of Stuart Douglas' ThreadLocal idea, plus some<br>
InvocationContext fiddling. I tried to understand the Interceptor Spec<br>
and how it related to what I was trying to do, but I still wasn't sure<br>
exactly what was expected of me to keep the spec happy. In the end I<br>
managed to get it working by watching the behavior of the threads in the<br>
debugger and pandering to that. It seemed consistent between OWB and<br>
Weld at least, and the resulting solution works in both containers.<br>
<br>
Anyway, I'm clearly not 100% trustworthy of this code, particularly wrt<br>
being within the spec, so would appreciate some experts giving it the<br>
once-over.<br>
<br>
<a href="https://github.com/seam/cron/tree/master/spi/src/main/java/org/jboss/seam/cron/spi/asynchronous" target="_blank">https://github.com/seam/cron/tree/master/spi/src/main/java/org/jboss/seam/cron/spi/asynchronous</a><br>
<br>
Cheers,<br>
<br>
Pete R.<br>
_______________________________________________<br>
seam-dev mailing list<br>
<a href="mailto:seam-dev@lists.jboss.org" target="_blank">seam-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/seam-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/seam-dev</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>Jason Porter<br><a href="http://lightguard-jp.blogspot.com/" target="_blank">http://lightguard-jp.blogspot.com</a><br><a href="http://twitter.com/lightguardjp" target="_blank">http://twitter.com/lightguardjp</a><br>
<br>Software Engineer<br>Open Source Advocate<br>Author of Seam Catch - Next Generation Java Exception Handling<br><br>PGP key id: 926CCFF5<br>PGP key available at: <a href="http://keyserver.net/" target="_blank">keyserver.net</a>, <a href="http://pgp.mit.edu/" target="_blank">pgp.mit.edu</a><br>
</blockquote></div><br></div></div></div></div></blockquote></div><br><br clear="all"><br>-- <br>Jason Porter<br><a href="http://lightguard-jp.blogspot.com" target="_blank">http://lightguard-jp.blogspot.com</a><br><a href="http://twitter.com/lightguardjp" target="_blank">http://twitter.com/lightguardjp</a><br>
<br>Software Engineer<br>Open Source Advocate<br>Author of Seam Catch - Next Generation Java Exception Handling<br><br>PGP key id: 926CCFF5<br>PGP key available at: <a href="http://keyserver.net" target="_blank">keyserver.net</a>, <a href="http://pgp.mit.edu" target="_blank">pgp.mit.edu</a><br>