Re: [PATCH] Fix for `submodules' in (ice-9 session) (closes #30062)

[Jose A. Ortega Ruiz]

Hi again,

On Tue, Aug 31 2010, Jose A. Ortega Ruiz wrote:

> On Tue, Aug 31 2010, Andy Wingo wrote:
>
>> Hi,
>>
>> Can you submit a test please, also?
>
> I'm not quite sure what you want the test to be: the procedure
> `submodules' is not exported by `(ice-9 session)', and there's no test
> suite for the latter, so testing this change properly would mean
> creating such a test suite, checking all directly affected exported
> procedures. Those are `apropos-fold-exported' and
> `apropos-fold-accessible'.
>
> Testing them is a bit involved, due to the cycles in the module tree

[... blah blah blah ...]

Please, ignore all the above stupid chatter. The fold functions are
already testing for duplicates and there's no infinite loop anywhere.
Sorry about my big mouth. (I'm still curious about the meaning of the
modules with gensyms as names, though.)

So, i'll write indirect tests for `submodules' by testing
`apropos-fold-exported' and `apropos-fold-all' (i didn't even get the
name right!). Would that work for you?

Thanks,

jao
-- 
"I didn't do it, and I'll never do it again."
 -Derrik Weeks