Re: [Iverilog-devel] do_include bug: FIXED

[Michael W. <mfw...@MI...>]

On 5 Mar 2008, at 12:52 PM, Cary R. wrote:

> --- Stephen Williams <st...@ic...> wrote:
>
>> I think this is a lot of hoopla for the testing of such a tiny  
>> feature.
>> I would rather not pollute the main test suite script for such rare
>> cases.
>
> Like I said in an earlier post, my preference is for you to have a  
> local
> test for this that you run along with the normal test suite. I have a
> bunch of circuits I run locally to check for problem. It is these  
> local
> tests that found pr1903343. As long as someone in the development  
> process
> is testing this I think we can omit it from the test suite.

Frankly, I think that all useful testing should be made part of one  
testing
framework.

After all, the painfully obvious/embarrassing bug that I introduced  
passed
the official testing suite with flying colors, and the latest patch that
I submitted was accepted as follows:

	"The regression test still passes, and I trust that it solves the
	problem it's supposed to solve."

While I'm flattered, it seems that the trust in the test suite is a  
bit undue.

Michael Witten