perl-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Philippe M. Chiasson" <>
Subject Re: svn commit: r357236 - /perl/modperl/trunk/src/modules/perl/modperl_env.c
Date Sat, 17 Dec 2005 03:01:22 GMT
Stas Bekman wrote:
> Philippe M. Chiasson wrote:
>> Stas Bekman wrote:
>>> wrote:
>>>> Author: gozer
>>>> Date: Fri Dec 16 13:20:23 2005
>>>> New Revision: 357236
>>>> URL:
>>>> Log:
>>>> "local %ENV;" caused Segmentation fault in modperl_env_request_tie()
>>>> Reported-By:  Peter Walsham <>
>>>> Thread:
>>> a bug fix w/o adding a test?
>> Well, the test would cause a segfault, so figured it wasn't such a
>> good idea
>> to add it. 
> I'm not following you Philippe, you've fixed it. Isn't that how all
> tests are written? You first write a test, you then fix it and then you
> commit the test and the fix, no?

Yup, you are correct. This reasoning was caused by : gozer.brain->off();

>> But now that the fix is in, I guess I could make a test case to make
>> sure it now works. Good point Stas ;-)
>> Expect test case soon.
> gozer++
>> Which reminds me, it would be nice to have a bug/ test categorie for
>> such gems
>> that don't fit very nicely anywhere else.
> This test fits perfectly into the main test suite. But yes, we need to
> have a new directory for other tests. Long ago I've suggested t_extra.

Wasn't t_extra for the kind of developer's tests what would not be executed
unless we were building in maintainer-mode ?

I was more thinking of a category of tests specifically to nail down known

Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

View raw message