You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "James Sawle (JIRA)" <ji...@apache.org> on 2014/10/10 00:52:34 UTC

[jira] [Updated] (LANG-780) add public org.apache.commons.lang.builder.HashCodeBuilder append(Enum value) to HashCodeBuilder

     [ https://issues.apache.org/jira/browse/LANG-780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

James Sawle updated LANG-780:
-----------------------------
    Attachment: LANG-780.patch

Patch for enum support in HashCodeBuilder.

Using the Enum's class name and value name separated by a '.' to cover the concerns. This should mean that two different Enums that use the same value names do not clash. This will also give consistency across JVMs/different runs, even though the contract does not require it.

> add  public org.apache.commons.lang.builder.HashCodeBuilder append(Enum value) to HashCodeBuilder 
> --------------------------------------------------------------------------------------------------
>
>                 Key: LANG-780
>                 URL: https://issues.apache.org/jira/browse/LANG-780
>             Project: Commons Lang
>          Issue Type: New Feature
>          Components: lang.builder.*
>    Affects Versions: 2.5
>            Reporter: Alex Pokotilo
>             Fix For: Patch Needed
>
>         Attachments: LANG-780.patch
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> Since Enum's hashCode return different values between calls this can cause different values if enum appended to HashCodeBuilder 
> IMHO adding append for enum would be enough to solve this entirely
> public org.apache.commons.lang.builder.HashCodeBuilder append(Enum value)
> {
>  append(value.name().hashCode());
>  return this;
> }



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)