[prev in list] [next in list] [prev in thread] [next in thread]
List: jakarta-commons-dev
Subject: [jira] Closed: (LANG-628) Javadoc for
From: "Henri Yandell (JIRA)" <jira () apache ! org>
Date: 2010-06-30 3:20:49
Message-ID: 6085481.126591277868049924.JavaMail.jira () thor
[Download RAW message or body]
[ https://issues.apache.org/jira/browse/LANG-628?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]
Henri Yandell closed LANG-628.
------------------------------
Resolution: Fixed
I've dropped the 'iConstant *' part. None of the other appends cover that. I've noted \
that you have to check the java.lang.Boolean javadoc for the 1231/1237 part - I'm not \
sure what other detail could be offered; feel free to propose anything if you can \
think of what you'd like to see there.
I've added the following to the class javadoc:
+ * The following is the approach taken. When appending a data field, the current \
total is multiplied by the + * multiplier then a relevant value
+ * for that data type is added. For example, if the current hashCode is 17, and the \
multiplier is 37, then + * appending the integer 45 will create a hashcode of 674, \
namely 17 * 37 + 45.
Please reopen if you think more should be done.
> Javadoc for HashCodeBuilder.append(boolean) does not match implementation.
> --------------------------------------------------------------------------
>
> Key: LANG-628
> URL: https://issues.apache.org/jira/browse/LANG-628
> Project: Commons Lang
> Issue Type: Bug
> Components: lang.*
> Reporter: Sean Mickey
> Priority: Minor
> Fix For: 3.0
>
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> The Javadoc description for HashCodeBuilder.append(boolean) says that it "adds \
> iConstant * 1 to the hashCode," however, the code is actually: iTotal = iTotal * \
> iConstant + (value ? 0 : 1). It is probably just a typo, but it easy to correct. \
> And the reference to hashCode handling in java.lang.Boolean: "not a 1231 or 1237 \
> as done in java.lang.Boolean" could be clarified with more detail. Otherwise, it \
> requires going to the java.lang Javadoc and finding the java.lang.Boolean.hashCode \
> Javadoc description.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic