[erlang-patches] [jinterface] added OtpMbox.hash()
Björn Gustavsson
bgustavsson@REDACTED
Mon Sep 20 15:51:28 CEST 2010
On Mon, Sep 20, 2010 at 3:00 PM, Vlad Dumitrescu <vladdu55@REDACTED> wrote:
> The OtpMbox class was missing the hash() method while overriding
> equals(). This can cause problems when when using jinterface in a
> larger Java application, for which there are currently no tests in
> OTP.
The motivation given above is much more useful
if it is included in the commit message. See:
http://github.com/erlang/otp/wiki/Writing-good-commit-messages
> The fix is straightforward, if a test is required please let me
> know.
The reviewer will have to decide whether a test case is
needed or not. Including a test with a bug fix will certainly
increase the probability that we will include it.
--
Björn Gustavsson, Erlang/OTP, Ericsson AB
More information about the erlang-patches
mailing list