TODO tests in t/parser-subclass.t

[Steve Purkis]

tapx-dev-bounces at hexten.net wrote on 10/06/2008 13:44:02:

> I think this one is for Steve. I commented out the TODO tests in t/ 
> parser-subclass.t because our smoke script was seeing them as a failure.
> 
> Steve: in what sense are they "TODO"? Can they be reinstated?

Oh yeah, forgot about that one.

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 :-/


> I'll fix the smoke script to DRT but it'd be nice to resolve the TODOs 
> anyway.

Agreed - I'll have a look at it later on today/this evening.

I also wanted to write some tests to prove the Grammar & Source classes 
are using the parser to make stuff.

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() ?

-Steve

---

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.

Please refer to http://www.db.com/en/content/eu_disclosures.htm for additional EU corporate and regulatory disclosures.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.hexten.net/pipermail/tapx-dev/attachments/20080611/d9b764d1/attachment.htm