[erlang-patches] Fix handling of archive (.ez) files in code:clash/0
Björn Gustavsson
bgustavsson@REDACTED
Fri Feb 5 14:45:14 CET 2010
2010/2/5 Tuncer Ayaz <tuncer.ayaz@REDACTED>:
>> diff --git a/lib/kernel/test/code_SUITE.erl b/lib/kernel/test/code_SUITE.erl
>> index c8063ca..7712973 100644
>> --- a/lib/kernel/test/code_SUITE.erl
>> +++ b/lib/kernel/test/code_SUITE.erl
>> @@ -48,7 +48,7 @@ all(suite) ->
>> delete, purge, soft_purge, is_loaded, all_loaded,
>> load_binary, dir_req, object_code, set_path_file,
>> pa_pz_option, add_del_path,
>> - dir_disappeared, ext_mod_dep,
>> + dir_disappeared, ext_mod_dep, clash,
>
> Interesting as this was part of the rebased tree
> to be found as
> http://github.com/tuncer/otp/commit/585f77
> and also in a previous state
> http://github.com/tuncer/otp/commit/5fcd29
>
> Maybe you took an earlier tree.
Yes, this is strange.
I fetched your branch early this morning.
>
> I've pushed a rebased atop of ccase/r13b04_dev tree
> with these changes but have to ask why we remove
> the two clauses. Every other test seems to have
> the suite and doc clause even if they only return []
> in both clauses.
>
Matter of taste.
"suite" is optional, so I see no reason to put it in
in new test cases. On the other hand, I don't find
it important enough to change old test cases.
"doc" is OK if it it actually gives more information
than the name of the test case. Since it didn't,
I ripped it out.
We currently do not have any tool that will
collect only document strings and produce
a nice report. I am not sure that anyone
is interested in such a report. The documentation
strings are usually not detailed enough to
be of any real use out-of-context.
--
Björn Gustavsson, Erlang/OTP, Ericsson AB
More information about the erlang-patches
mailing list