<br><tt><font size=2>tapx-dev-bounces@hexten.net wrote on 10/06/2008 13:44:02:<br>
<br>
> I think this one is for Steve. I commented out the TODO tests in t/
<br>
> parser-subclass.t because our smoke script was seeing them as a failure.<br>
> <br>
> Steve: in what sense are they "TODO"? Can they be reinstated?</font></tt>
<br>
<br><tt><font size=2>Oh yeah, forgot about that one.</font></tt>
<br>
<br><tt><font size=2>Basically, I wasn't sure how to test a non-perl source
across platforms. Obviously, putting #!/bin/sh won't work on win32,
so I just marked it TODO and forgot about it. I'm not even sure if
I completed the test :-/</font></tt>
<br>
<br>
<br><tt><font size=2>> I'll fix the smoke script to DRT but it'd be
nice to resolve the TODOs <br>
> anyway.</font></tt>
<br>
<br><tt><font size=2>Agreed - I'll have a look at it later on today/this
evening.</font></tt>
<br>
<br><tt><font size=2>I also wanted to write some tests to prove the Grammar
& Source classes are using the parser to make stuff.</font></tt>
<br>
<br><tt><font size=2>Finally, I've a nagging voice in the back of my head
about circular refs & memory leaks - parser refers to grammar refers
to parser... Any word on whether it's ok to use Scalar::Util's weaken()
?</font></tt>
<br>
<br><tt><font size=2>-Steve</font></tt>
<br>
<span style="font-family:sans-serif,helvetica; font-size:10pt; color:#000000">---<br>
<br>
This e-mail may contain confidential and/or privileged information. If you are not the intended recipient (or have received this e-mail in error) please notify the sender immediately and delete this e-mail. Any unauthorized copying, disclosure or distribution of the material in this e-mail is strictly forbidden.<br>
<br>
Please refer to http://www.db.com/en/content/eu_disclosures.htm for additional EU corporate and regulatory disclosures.</span><br>