[erlang-patches] [erlang-bugs] Unused variable causing v3_kernel crash
Anthony Ramine
n.oxyde@REDACTED
Mon Feb 4 15:50:10 CET 2013
I squashed your commit with mine; please refetch.
--
Anthony Ramine
Le 4 févr. 2013 à 14:43, Björn Gustavsson a écrit :
> Thanks for test suite.
>
> There are a few issues, though.
>
> First, the new test cases are not correctly listed
> in the lists in the groups/0 function, so they will
> not actually be run.
>
> Second, being very paranoid when writing test
> cases, I would make sure that the compiled
> code atually works (in this case, that the function
> with side effects is actually called).
>
> Here is my suggestion how it can be done. If
> you are fine with my changes, we can simply
> squash my commit into yours.
>
> https://github.com/bjorng/otp/commit/4e21e16ebc1c012ac52b89ba1b38d6f98cfbac2c
>
> Note that since there is no good reason for
> keeping the test case case in a separate file,
> I have put all code for unused_multiple_values
> into core_fold_SUITE itself.
>
>
>
> On Mon, Feb 4, 2013 at 12:16 PM, Anthony Ramine <n.oxyde@REDACTED> wrote:
> I added a commit with two test cases, please refetch.
>
> --
> Anthony Ramine
>
> Le 4 févr. 2013 à 07:17, Björn Gustavsson a écrit :
>
> >
> > On Sun, Feb 3, 2013 at 12:32 PM, Anthony Ramine <n.oxyde@REDACTED> wrote:
> > Hi,
> >
> > I've added some details and squashed the two commits together,
> > please refetch.
> >
> > In which suite should I put such a test case?
> >
> >
> > I suggest core_fold_SUITE.
> >
> > --
> > Björn Gustavsson, Erlang/OTP, Ericsson AB
>
>
>
>
> --
> Björn Gustavsson, Erlang/OTP, Ericsson AB
More information about the erlang-patches
mailing list