TODO tests in t/parser-subclass.t

[Steve Purkis]

tapx-dev-bounces at hexten.net wrote on 11/06/2008 11:44:03:

> --- Steve Purkis <steve.purkis at db.com> wrote:
> 
> > 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() ?
> 
> We can't use that module because it's not core.  Tough to get
> Test::Harness into core if we're dragging a lot of baggage along with
> it :)

Thought as much.  IIRC, weaken is XS, so not easy to include here.  Sounds 
like Andy's idea of deleting $self->{parser} is the best one we've got so 
far.  I've briefly considered a global list of all parsers and then 
storing indexes of the Grammar/etc's parser, but thought that'd just 
complicate things too much.


> Side note:  your HTML email results in an incredibly tiny font I can't
> read (this is via Yahoo! Mail on Firefox on both Windows and Macs). 
> Not sure if the font size is intentional or some sort of bug.  Thought
> you should know.

Yeah, I did notice this.  lotus_notes--

I've found the 'plain text' checkbox, so hopefully this problem will go 
away.  Thanks for pointing it out.

-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.