I don't recall - I think it was more subtle in the results of the associated integration tests.
On 06/07/2011, at 12:14 PM, John Fallows wrote: > No worries, Brett. > > When you say you found it was broken, do you mean that you also saw the NPE > during integration tests or that something else was wrong so you needed add > the temporary "return null" ? > > tc, > -john. > > On Tue, Jul 5, 2011 at 6:21 PM, Brett Porter <[email protected]> wrote: > >> On 06/07/2011, at 8:40 AM, John Fallows wrote: >> >>> Out of curiousity, what was the rationale for removing it and has that >> issue >>> been addressed, allowing us to re-enable the feature? >> >> >> When I started making integration tests, I found it was broken, and I >> wasn't sure if it had ever worked. Didn't have time to fix it then, so the >> tests stayed disabled. >> >> Sorry, probably not the answer you were hoping for :) >> >> - Brett >> >> -- >> Brett Porter >> [email protected] >> http://brettporter.wordpress.com/ >> http://au.linkedin.com/in/brettporter >> >> >> >> >> > > > -- >> |< Kaazing Corporation >|< > John Fallows | CTO | +1.650.960.8148 > 444 Castro St, Suite 1100 | Mountain View, CA 94041, USA -- Brett Porter [email protected] http://brettporter.wordpress.com/ http://au.linkedin.com/in/brettporter
