java-defects4j-lang-64-valuedenum

A painting is a person's account of this edit as text, recorded independently of the node mapping - a rendering often has several equally defensible answers where the mapping has one. Deletions and insertions carry no position on the opposite side in a painting, so that panel draws no caret for them.

Before…lines 1–176 unchanged (176 lines)
   * @return -ve if this is less than the other object, +ve if greater than,
   *  <code>0</code> of equal
   * @throws ClassCastException if other is not an <code>Enum</code>
   * @throws NullPointerException if other is <code>null</code>
   */
  public int compareTo(Object other) {
      return iValue - ((ValuedEnum) other).iValue;
  }
185
  /**
   * <p>Use reflection to return an objects value.</p>
   *
   * @param other  the object to determine the value for
   * @return the value
   */
          // ignore - should never happen
          // ignore - should never happen
          // ignore - should never happen
195
  /**
   * <p>Human readable description of this <code>Enum</code> item.</p>
   *
   * @return String in the form <code>type[name=value]</code>, for example:
   *  <code>JavaVersion[Java 1.0=100]</code>. Note that the package name is
   *  stripped from the type name.
…lines 202–211 unchanged (10 lines)

After…lines 1–176 unchanged (176 lines)
   * @return -ve if this is less than the other object, +ve if greater than,
   *  <code>0</code> of equal
   * @throws ClassCastException if other is not an <code>Enum</code>
   * @throws NullPointerException if other is <code>null</code>
   */
  public int compareTo(Object other) {
      if (other == this) {
          return 0;
      }
      if (other.getClass() != this.getClass()) {
          if (other.getClass().getName().equals(this.getClass().getName())) {
              return iValue - getValueInOtherClassLoader(other);
          }
          throw new ClassCastException(
                  "Different enum class '" + ClassUtils.getShortClassName(other.getClass()) + "'");
      }
      return iValue - ((ValuedEnum) other).iValue;
  }
195
  /**
   * <p>Use reflection to return an objects value.</p>
   *
   * @param other  the object to determine the value for
   * @return the value
   */
  private int getValueInOtherClassLoader(Object other) {
      try {
          Method mth = other.getClass().getMethod("getValue", null);
          Integer value = (Integer) mth.invoke(other, null);
          return value.intValue();
      } catch (NoSuchMethodException e) {
          // ignore - should never happen
      } catch (IllegalAccessException e) {
          // ignore - should never happen
      } catch (InvocationTargetException e) {
          // ignore - should never happen
      }
      throw new IllegalStateException("This should not happen");
  }
216
  /**
   * <p>Human readable description of this <code>Enum</code> item.</p>
   *
   * @return String in the form <code>type[name=value]</code>, for example:
   *  <code>JavaVersion[Java 1.0=100]</code>. Note that the package name is
…lines 222–232 unchanged (11 lines)

Before…lines 1–176 unchanged (176 lines)
   * @return -ve if this is less than the other object, +ve if greater than,
   *  <code>0</code> of equal
   * @throws ClassCastException if other is not an <code>Enum</code>
   * @throws NullPointerException if other is <code>null</code>
   */
  public int compareTo(Object other) {
      return iValue - ((ValuedEnum) other).iValue;
  }
185
  /**
   * <p>Use reflection to return an objects value.</p>
   *
   * @param other  the object to determine the value for
   * @return the value
   */
          // ignore - should never happen
          // ignore - should never happen
          // ignore - should never happen
195
  /**
   * <p>Human readable description of this <code>Enum</code> item.</p>
   *
   * @return String in the form <code>type[name=value]</code>, for example:
   *  <code>JavaVersion[Java 1.0=100]</code>. Note that the package name is
   *  stripped from the type name.
…lines 202–211 unchanged (10 lines)

After…lines 1–176 unchanged (176 lines)
   * @return -ve if this is less than the other object, +ve if greater than,
   *  <code>0</code> of equal
   * @throws ClassCastException if other is not an <code>Enum</code>
   * @throws NullPointerException if other is <code>null</code>
   */
  public int compareTo(Object other) {
      if (other == this) {
          return 0;
      }
      if (other.getClass() != this.getClass()) {
          if (other.getClass().getName().equals(this.getClass().getName())) {
              return iValue - getValueInOtherClassLoader(other);
          }
          throw new ClassCastException(
                  "Different enum class '" + ClassUtils.getShortClassName(other.getClass()) + "'");
      }
      return iValue - ((ValuedEnum) other).iValue;
  }
195
  /**
   * <p>Use reflection to return an objects value.</p>
   *
   * @param other  the object to determine the value for
   * @return the value
   */
  private int getValueInOtherClassLoader(Object other) {
      try {
          Method mth = other.getClass().getMethod("getValue", null);
          Integer value = (Integer) mth.invoke(other, null);
          return value.intValue();
      } catch (NoSuchMethodException e) {
          // ignore - should never happen
      } catch (IllegalAccessException e) {
          // ignore - should never happen
      } catch (InvocationTargetException e) {
          // ignore - should never happen
      }
      throw new IllegalStateException("This should not happen");
  }
216
  /**
   * <p>Human readable description of this <code>Enum</code> item.</p>
   *
   * @return String in the form <code>type[name=value]</code>, for example:
   *  <code>JavaVersion[Java 1.0=100]</code>. Note that the package name is
…lines 222–232 unchanged (11 lines)

Before…lines 1–176 unchanged (176 lines)
   * @return -ve if this is less than the other object, +ve if greater than,
   *  <code>0</code> of equal
   * @throws ClassCastException if other is not an <code>Enum</code>
   * @throws NullPointerException if other is <code>null</code>
   */
  public int compareTo(Object other) {
      return iValue - ((ValuedEnum) other).iValue;
  }
185
  /**
   * <p>Use reflection to return an objects value.</p>
   *
   * @param other  the object to determine the value for
   * @return the value
   */
          // ignore - should never happen
          // ignore - should never happen
          // ignore - should never happen
195
  /**
   * <p>Human readable description of this <code>Enum</code> item.</p>
   *
   * @return String in the form <code>type[name=value]</code>, for example:
   *  <code>JavaVersion[Java 1.0=100]</code>. Note that the package name is
   *  stripped from the type name.
…lines 202–211 unchanged (10 lines)

After…lines 1–176 unchanged (176 lines)
   * @return -ve if this is less than the other object, +ve if greater than,
   *  <code>0</code> of equal
   * @throws ClassCastException if other is not an <code>Enum</code>
   * @throws NullPointerException if other is <code>null</code>
   */
  public int compareTo(Object other) {
      if (other == this) {
          return 0;
      }
      if (other.getClass() != this.getClass()) {
          if (other.getClass().getName().equals(this.getClass().getName())) {
              return iValue - getValueInOtherClassLoader(other);
          }
          throw new ClassCastException(
                  "Different enum class '" + ClassUtils.getShortClassName(other.getClass()) + "'");
      }
      return iValue - ((ValuedEnum) other).iValue;
  }
195
  /**
   * <p>Use reflection to return an objects value.</p>
   *
   * @param other  the object to determine the value for
   * @return the value
   */
  private int getValueInOtherClassLoader(Object other) {
      try {
          Method mth = other.getClass().getMethod("getValue", null);
          Integer value = (Integer) mth.invoke(other, null);
          return value.intValue();
      } catch (NoSuchMethodException e) {
          // ignore - should never happen
      } catch (IllegalAccessException e) {
          // ignore - should never happen
      } catch (InvocationTargetException e) {
          // ignore - should never happen
      }
      throw new IllegalStateException("This should not happen");
  }
216
  /**
   * <p>Human readable description of this <code>Enum</code> item.</p>
   *
   * @return String in the form <code>type[name=value]</code>, for example:
   *  <code>JavaVersion[Java 1.0=100]</code>. Note that the package name is
…lines 222–232 unchanged (11 lines)

Before

program

package_declaration

package "package"

scoped_identifier

identifier "org"

. "."

identifier "apache"

. "."

identifier "commons"

. "."

identifier "lang"

. "."

identifier "enums"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "java"

. "."

identifier "lang"

. "."

identifier "reflect"

. "."

identifier "InvocationTargetException"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "java"

. "."

identifier "lang"

. "."

identifier "reflect"

. "."

identifier "Method"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "java"

. "."

identifier "util"

. "."

identifier "Iterator"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "java"

. "."

identifier "util"

. "."

identifier "List"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "org"

. "."

identifier "apache"

. "."

identifier "commons"

. "."

identifier "lang"

. "."

identifier "ClassUtils"

; ";"

block_comment "/**\n * Abstract superclass for type-safe enums with integ"...

class_declaration

modifiers

public "public"

abstract "abstract"

class "class"

identifier "ValuedEnum"

superclass

extends "extends"

type_identifier "Enum"

class_body

{ "{"

block_comment "/**\n * Required for serialization support.\n * \n "...

field_declaration

modifiers

private "private"

static "static"

final "final"

integral_type

long "long"

variable_declarator

identifier "serialVersionUID"

= "="

unary_expression

- "-"

decimal_integer_literal "7129650521543789085L"

; ";"

block_comment "/**\n * The value contained in enum.\n */"

field_declaration

modifiers

private "private"

final "final"

integral_type

int "int"

variable_declarator

identifier "iValue"

; ";"

block_comment "/**\n * Constructor for enum item.\n *\n * @param n"...

constructor_declaration (+31 nodes collapsed)

block_comment "/**\n * Gets an <code>Enum</code> object by class and "...

method_declaration (+130 nodes collapsed)

block_comment "/**\n * Get value of enum item.\n *\n * @ret"...

method_declaration

modifiers

public "public"

final "final"

integral_type

int "int"

identifier "getValue"

formal_parameters

( "("

) ")"

block

{ "{"

return_statement

return "return"

identifier "iValue"

; ";"

} "}"

block_comment "/**\n * Tests for order.\n *\n * The defa"...

method_declaration

modifiers

public "public"

integral_type

int "int"

identifier "compareTo"

formal_parameters

( "("

formal_parameter

type_identifier "Object"

identifier "other"

) ")"

block

{ "{"

return_statement

return "return"

binary_expression

identifier "iValue"

- "-"

field_access

parenthesized_expression

( "("

cast_expression

( "("

type_identifier "ValuedEnum"

) ")"

identifier "other"

) ")"

. "."

identifier "iValue"

; ";"

} "}"

block_comment "/**\n * Use reflection to return an objects value."...

line_comment "// ignore - should never happen"

block_comment "/**\n * Human readable description of this <code>Enum<"...

method_declaration (+83 nodes collapsed)

} "}"

After

program

package_declaration

package "package"

scoped_identifier

identifier "org"

. "."

identifier "apache"

. "."

identifier "commons"

. "."

identifier "lang"

. "."

identifier "enums"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "java"

. "."

identifier "lang"

. "."

identifier "reflect"

. "."

identifier "InvocationTargetException"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "java"

. "."

identifier "lang"

. "."

identifier "reflect"

. "."

identifier "Method"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "java"

. "."

identifier "util"

. "."

identifier "Iterator"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "java"

. "."

identifier "util"

. "."

identifier "List"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "org"

. "."

identifier "apache"

. "."

identifier "commons"

. "."

identifier "lang"

. "."

identifier "ClassUtils"

; ";"

block_comment "/**\n * Abstract superclass for type-safe enums with integ"...

class_declaration

modifiers

public "public"

abstract "abstract"

class "class"

identifier "ValuedEnum"

superclass

extends "extends"

type_identifier "Enum"

class_body

{ "{"

block_comment "/**\n * Required for serialization support.\n * \n "...

field_declaration

modifiers

private "private"

static "static"

final "final"

integral_type

long "long"

variable_declarator

identifier "serialVersionUID"

= "="

unary_expression

- "-"

decimal_integer_literal "7129650521543789085L"

; ";"

block_comment "/**\n * The value contained in enum.\n */"

field_declaration

modifiers

private "private"

final "final"

integral_type

int "int"

variable_declarator

identifier "iValue"

; ";"

block_comment "/**\n * Constructor for enum item.\n *\n * @param n"...

constructor_declaration (+31 nodes collapsed)

block_comment "/**\n * Gets an <code>Enum</code> object by class and "...

method_declaration (+130 nodes collapsed)

block_comment "/**\n * Get value of enum item.\n *\n * @ret"...

method_declaration

modifiers

public "public"

final "final"

integral_type

int "int"

identifier "getValue"

formal_parameters

( "("

) ")"

block

{ "{"

return_statement

return "return"

identifier "iValue"

; ";"

} "}"

block_comment "/**\n * Tests for order.\n *\n * The defa"...

method_declaration

modifiers

public "public"

integral_type

int "int"

identifier "compareTo"

formal_parameters

( "("

formal_parameter

type_identifier "Object"

identifier "other"

) ")"

block

{ "{"

if_statement

if "if"

parenthesized_expression

( "("

binary_expression

identifier "other"

== "=="

this "this"

) ")"

block

{ "{"

return_statement

return "return"

decimal_integer_literal "0"

; ";"

} "}"

if_statement

if "if"

parenthesized_expression

( "("

binary_expression

method_invocation

identifier "other"

. "."

identifier "getClass"

argument_list

( "("

) ")"

!= "!="

method_invocation

this "this"

. "."

identifier "getClass"

argument_list

( "("

) ")"

block

{ "{"

if_statement

if "if"

parenthesized_expression

( "("

method_invocation

identifier "other"

. "."

identifier "getClass"

argument_list

( "("

) ")"

. "."

identifier "getName"

argument_list

( "("

) ")"

. "."

identifier "equals"

argument_list

( "("

method_invocation

this "this"

. "."

identifier "getClass"

argument_list

( "("

) ")"

. "."

identifier "getName"

argument_list

( "("

) ")"

block

{ "{"

return_statement

return "return"

binary_expression

identifier "iValue"

- "-"

method_invocation

identifier "getValueInOtherClassLoader"

argument_list

( "("

identifier "other"

) ")"

; ";"

} "}"

throw_statement

throw "throw"

object_creation_expression

new "new"

type_identifier "ClassCastException"

argument_list

( "("

binary_expression

string_literal

" "\""

string_fragment "Different enum class '"

" "\""

+ "+"

method_invocation

identifier "ClassUtils"

. "."

identifier "getShortClassName"

argument_list

( "("

method_invocation

identifier "other"

. "."

identifier "getClass"

argument_list

( "("

) ")"

+ "+"

string_literal

" "\""

string_fragment "'"

" "\""

) ")"

; ";"

} "}"

return_statement

return "return"

binary_expression

identifier "iValue"

- "-"

field_access

parenthesized_expression

( "("

cast_expression

( "("

type_identifier "ValuedEnum"

) ")"

identifier "other"

) ")"

. "."

identifier "iValue"

; ";"

} "}"

block_comment "/**\n * Use reflection to return an objects value."...

method_declaration

modifiers

private "private"

integral_type

int "int"

identifier "getValueInOtherClassLoader"

formal_parameters

( "("

formal_parameter

type_identifier "Object"

identifier "other"

) ")"

block

{ "{"

try_statement

try "try"

block

{ "{"

local_variable_declaration

type_identifier "Method"

variable_declarator

identifier "mth"

= "="

method_invocation

identifier "other"

. "."

identifier "getClass"

argument_list

( "("

) ")"

. "."

identifier "getMethod"

argument_list

( "("

string_literal

" "\""

string_fragment "getValue"

" "\""

, ","

null_literal "null"

) ")"

; ";"

local_variable_declaration

type_identifier "Integer"

variable_declarator

identifier "value"

= "="

cast_expression

( "("

type_identifier "Integer"

) ")"

method_invocation

identifier "mth"

. "."

identifier "invoke"

argument_list

( "("

identifier "other"

, ","

null_literal "null"

) ")"

; ";"

return_statement

return "return"

method_invocation

identifier "value"

. "."

identifier "intValue"

argument_list

( "("

) ")"

; ";"

} "}"

catch_clause

catch "catch"

( "("

catch_formal_parameter

catch_type

type_identifier "NoSuchMethodException"

identifier "e"

) ")"

block

{ "{"

line_comment "// ignore - should never happen"

} "}"

catch_clause

catch "catch"

( "("

catch_formal_parameter

catch_type

type_identifier "IllegalAccessException"

identifier "e"

) ")"

block

{ "{"

line_comment "// ignore - should never happen"

} "}"

catch_clause

catch "catch"

( "("

catch_formal_parameter

catch_type

type_identifier "InvocationTargetException"

identifier "e"

) ")"

block

{ "{"

line_comment "// ignore - should never happen"

} "}"

throw_statement

throw "throw"

object_creation_expression

new "new"

type_identifier "IllegalStateException"

argument_list

( "("

string_literal

" "\""

string_fragment "This should not happen"

" "\""

) ")"

; ";"

} "}"

block_comment "/**\n * Human readable description of this <code>Enum<"...

method_declaration (+83 nodes collapsed)

} "}"