[forge-commits] [forge/java-parser] 7ec717: merge bugfix and tests from refactor branch

Wed Aug 28 18:05:19 EDT 2013

  Branch: refs/heads/refactor
  Home:   https://github.com/forge/java-parser
  Commit: 7ec717c5b6bebf7b586446c79e7455175752bfa9
      https://github.com/forge/java-parser/commit/7ec717c5b6bebf7b586446c79e7455175752bfa9
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-01 (Thu, 01 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/JavaEnumImpl.java
    M impl/src/test/java/org/jboss/forge/test/parser/java/JavaEnumTest.java

  Log Message:
  -----------
  merge bugfix and tests from refactor branch

  Commit: 4e581a5171e5329c79da116541f7e82599a74d39
      https://github.com/forge/java-parser/commit/4e581a5171e5329c79da116541f7e82599a74d39
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-01 (Thu, 01 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/JavaEnumImpl.java
    M impl/src/test/java/org/jboss/forge/test/parser/java/JavaEnumTest.java

  Log Message:
  -----------
  fixup merge from branch

  Commit: 3bdd62960ac976d1bcb66df9b28dab2096307512
      https://github.com/forge/java-parser/commit/3bdd62960ac976d1bcb66df9b28dab2096307512
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-02 (Fri, 02 Aug 2013)

  Changed paths:
    M api/src/main/java/org/jboss/forge/parser/java/AnnotationElement.java

  Log Message:
  -----------
  add class comment for AnnotationElement.DefaultValue

  Commit: 13c7ddedae9ddec62a57722774a6416b9f6356cb
      https://github.com/forge/java-parser/commit/13c7ddedae9ddec62a57722774a6416b9f6356cb
  Author: Vineet Reynolds <Vineet.Reynolds at gmail.com>
  Date:   2013-08-05 (Mon, 05 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/FieldImpl.java
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldTypeTest.java

  Log Message:
  -----------
  FORGE-1086 Fixed Field.getQualifiedType()

The fix ensures that ArrayTypes are not converted into plain SimpleTypes when the field type is parsed.

  Commit: e31de8d6d7b79dde4438dba775b9345106ff828c
      https://github.com/forge/java-parser/commit/e31de8d6d7b79dde4438dba775b9345106ff828c
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-05 (Mon, 05 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/FieldImpl.java
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldTypeTest.java

  Log Message:
  -----------
  Merge pull request #7 from VineetReynolds/FORGE-1086

FORGE-1086 Fixed Field.getQualifiedType()

  Commit: 816c028cd5fb0c7411d33beff92a86f2bd7814f1
      https://github.com/forge/java-parser/commit/816c028cd5fb0c7411d33beff92a86f2bd7814f1
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-05 (Mon, 05 Aug 2013)

  Changed paths:
    M api/src/main/java/org/jboss/forge/parser/java/EnumConstant.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/EnumConstantImpl.java
    A impl/src/test/java/org/jboss/forge/test/parser/java/EnumConstantAnnotationTest.java
    A impl/src/test/resources/org/jboss/forge/grammar/java/MockAnnotatedEnumConstant.java

  Log Message:
  -----------
  enumConstants can host annotations

  Commit: 31d9cb3a09bcef680adc4d997689ed140c7daefc
      https://github.com/forge/java-parser/commit/31d9cb3a09bcef680adc4d997689ed140c7daefc
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-05 (Mon, 05 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/EnumConstantImpl.java

  Log Message:
  -----------
  enumConstant+= equals/hashCode

  Commit: 6e6209e70084f2b04c21be3b370605ea98db2c2a
      https://github.com/forge/java-parser/commit/6e6209e70084f2b04c21be3b370605ea98db2c2a
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-05 (Mon, 05 Aug 2013)

  Changed paths:
    M api/src/main/java/org/jboss/forge/parser/java/EnumConstant.java
    M impl/src/main/java/org/jboss/forge/parser/java/ast/MethodFinderVisitor.java
    A impl/src/main/java/org/jboss/forge/parser/java/impl/EnumConstantBodyImpl.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/EnumConstantImpl.java
    A impl/src/test/java/org/jboss/forge/test/parser/java/EnumConstantTest.java

  Log Message:
  -----------
  [FORGE-1090] EnumConstant should support constructor arguments; [FORGE-1091] EnumConstant should support anonymous subclass body

  Commit: 2eee41accaeb1781dc3d9649c57f40a421a87176
      https://github.com/forge/java-parser/commit/2eee41accaeb1781dc3d9649c57f40a421a87176
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-05 (Mon, 05 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/JavaPackageInfoImpl.java

  Log Message:
  -----------
  Collections.emptyList() for empty list that is usually implemented as read-only

  Commit: 8e2b07da5d100d49d030112501eb9936865a60b4
      https://github.com/forge/java-parser/commit/8e2b07da5d100d49d030112501eb9936865a60b4
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-06 (Tue, 06 Aug 2013)

  Changed paths:
    A impl/src/test/java/org/jboss/forge/test/parser/java/JavaMethodBodyTest.java

  Log Message:
  -----------
  Added Test for Generics

  Commit: 531893dca8b455e948c48f15f8f7201086a08304
      https://github.com/forge/java-parser/commit/531893dca8b455e948c48f15f8f7201086a08304
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-06 (Tue, 06 Aug 2013)

  Changed paths:
    R impl/src/test/java/org/jboss/forge/test/parser/java/JavaMethodBodyTest.java
    A impl/src/test/java/org/jboss/forge/test/parser/java/JavaSourceCompatibilityTest.java

  Log Message:
  -----------
  Added test with generics from added method

  Commit: dfced4abf56ee7c5f9abec739cde2e4931b1563c
      https://github.com/forge/java-parser/commit/dfced4abf56ee7c5f9abec739cde2e4931b1563c
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-06 (Tue, 06 Aug 2013)

  Changed paths:
    M impl/src/test/java/org/jboss/forge/test/parser/java/JavaSourceCompatibilityTest.java

  Log Message:
  -----------
  Added generics statements on constructor

  Commit: 31b11ca79635c401e56f4808f5dfd65bb8e3c25b
      https://github.com/forge/java-parser/commit/31b11ca79635c401e56f4808f5dfd65bb8e3c25b
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-06 (Tue, 06 Aug 2013)

  Changed paths:
    M api/src/main/java/org/jboss/forge/parser/java/Field.java
    M api/src/main/java/org/jboss/forge/parser/java/Member.java
    M api/src/main/java/org/jboss/forge/parser/java/Method.java

  Log Message:
  -----------
  use Named interface for Members' set/getName() declarations

  Commit: 7866ab0e0751a8ac17d87cbc94ab7b224575caa6
      https://github.com/forge/java-parser/commit/7866ab0e0751a8ac17d87cbc94ab7b224575caa6
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-06 (Tue, 06 Aug 2013)

  Changed paths:
    A impl/src/test/java/org/jboss/forge/test/parser/java/JavaSourceCompatibilityTest.java

  Log Message:
  -----------
  Merge remote-tracking branch 'upstream/master'

  Commit: 187d9cdacfcca97575d57442d354983dcf34d97d
      https://github.com/forge/java-parser/commit/187d9cdacfcca97575d57442d354983dcf34d97d
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-06 (Tue, 06 Aug 2013)

  Changed paths:
    M api/src/main/java/org/jboss/forge/parser/java/AnnotationElement.java
    M api/src/main/java/org/jboss/forge/parser/java/EnumConstant.java

  Log Message:
  -----------
  use Named interface for enumConstant and AnnotationElement as well

  Commit: 33ef7c63f10782c148dc75475c917a925ae73baa
      https://github.com/forge/java-parser/commit/33ef7c63f10782c148dc75475c917a925ae73baa
  Author: Vineet Reynolds <Vineet.Reynolds at gmail.com>
  Date:   2013-08-07 (Wed, 07 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/AbstractJavaSource.java
    M impl/src/test/java/org/jboss/forge/test/parser/java/JavaSourceCompatibilityTest.java

  Log Message:
  -----------
  FORGE-1096 Fixed spurious ArrayIndexOutOfBoundsException.

The exception was thrown on rewriting the CompilationUnit, and
was most likely caused due to use of a lower source level.

Setting the source level to 1.7 gets rid of the exception.

  Commit: 5fff96e8b8ca9c58c12d42ca3b1ac571da75236e
      https://github.com/forge/java-parser/commit/5fff96e8b8ca9c58c12d42ca3b1ac571da75236e
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-07 (Wed, 07 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/AbstractJavaSource.java
    M impl/src/test/java/org/jboss/forge/test/parser/java/JavaSourceCompatibilityTest.java

  Log Message:
  -----------
  Merge pull request #8 from VineetReynolds/FORGE-1096

FORGE-1096 Fixed spurious ArrayIndexOutOfBoundsException.

  Commit: ef58a409335d1234c5d9324a88322f90f7f5bf99
      https://github.com/forge/java-parser/commit/ef58a409335d1234c5d9324a88322f90f7f5bf99
  Author: Vineet Reynolds <Vineet.Reynolds at gmail.com>
  Date:   2013-08-07 (Wed, 07 Aug 2013)

  Changed paths:
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldTypeTest.java

  Log Message:
  -----------
  FORGE-918 Added test to reproduce failure.

  Commit: 04396e9c97579ea987afb7bf984877a3f53476a4
      https://github.com/forge/java-parser/commit/04396e9c97579ea987afb7bf984877a3f53476a4
  Author: Vineet Reynolds <Vineet.Reynolds at gmail.com>
  Date:   2013-08-07 (Wed, 07 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/FieldImpl.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/TypeImpl.java
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldTypeTest.java

  Log Message:
  -----------
  FORGE-918 Fix for ignoring extra dimensions of arrays

This ensures that underlying variable fragments having additional dimensions are considered when treating a field as an array type or not.

  Commit: dc9b1610da1761845bfff31a2cb157528183ebcc
      https://github.com/forge/java-parser/commit/dc9b1610da1761845bfff31a2cb157528183ebcc
  Author: Vineet Reynolds <Vineet.Reynolds at gmail.com>
  Date:   2013-08-07 (Wed, 07 Aug 2013)

  Changed paths:
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldTypeTest.java

  Log Message:
  -----------
  FORGE-1097 Added a test to reproduce the failure.

  Commit: 82ac7b9e477b7502e63dd2e888bd32fe21defb63
      https://github.com/forge/java-parser/commit/82ac7b9e477b7502e63dd2e888bd32fe21defb63
  Author: Vineet Reynolds <Vineet.Reynolds at gmail.com>
  Date:   2013-08-07 (Wed, 07 Aug 2013)

  Changed paths:
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldTypeTest.java

  Log Message:
  -----------
  Added test for field types having mixed mode array dimensions.

  Commit: 2719ebf6306d6738836cf59f47d9e5fbbc38f748
      https://github.com/forge/java-parser/commit/2719ebf6306d6738836cf59f47d9e5fbbc38f748
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-08 (Thu, 08 Aug 2013)

  Changed paths:
    M api/pom.xml
    M impl/pom.xml
    M pom.xml

  Log Message:
  -----------
  [maven-release-plugin] prepare release 1.1.3.Final

  Commit: d0c783ebef1b0abb6138c91cbff582ac6795b100
      https://github.com/forge/java-parser/commit/d0c783ebef1b0abb6138c91cbff582ac6795b100
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-08 (Thu, 08 Aug 2013)

  Changed paths:
    M api/pom.xml
    M impl/pom.xml
    M pom.xml

  Log Message:
  -----------
  [maven-release-plugin] prepare for next development iteration

  Commit: 414d29f066636c17fbedc595d53d627cec89b747
      https://github.com/forge/java-parser/commit/414d29f066636c17fbedc595d53d627cec89b747
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-08 (Thu, 08 Aug 2013)

  Changed paths:
    M impl/src/main/java/org/jboss/forge/parser/java/impl/AbstractJavaSourceMemberHolder.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/EnumConstantBodyImpl.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/FieldImpl.java
    A impl/src/test/java/org/jboss/forge/test/parser/java/FieldMultipleTest.java
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldTypeTest.java

  Log Message:
  -----------
  FORGE-1097: Multiple fields are split into their own FieldDeclarations

  Commit: 4ad71a99fa86654c06380b6f9343534cb1458096
      https://github.com/forge/java-parser/commit/4ad71a99fa86654c06380b6f9343534cb1458096
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-08 (Thu, 08 Aug 2013)

  Changed paths:
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldMultipleTest.java

  Log Message:
  -----------
  Added test for multiple fields with annotations

  Commit: 748f3e6282b94a0ac0f7807165555eb5db486b87
      https://github.com/forge/java-parser/commit/748f3e6282b94a0ac0f7807165555eb5db486b87
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-08 (Thu, 08 Aug 2013)

  Changed paths:
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldMultipleTest.java

  Log Message:
  -----------
  Organized imports

  Commit: eca1bb227f460ee6f374b6703322e8183e422eb5
      https://github.com/forge/java-parser/commit/eca1bb227f460ee6f374b6703322e8183e422eb5
  Author: Vineet Reynolds <Vineet.Reynolds at gmail.com>
  Date:   2013-08-08 (Thu, 08 Aug 2013)

  Changed paths:
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldMultipleTest.java

  Log Message:
  -----------
  Added test for ensuring modifiers and initializers are propagated

  Commit: d24ac4e86605ec7be82ad1e3a035ff483fdbec65
      https://github.com/forge/java-parser/commit/d24ac4e86605ec7be82ad1e3a035ff483fdbec65
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-15 (Thu, 15 Aug 2013)

  Changed paths:
    M api/pom.xml
    M impl/pom.xml
    M pom.xml

  Log Message:
  -----------
  [maven-release-plugin] prepare release 1.1.4.Final

  Commit: 7456a939db12237680e930736edf03ba61084c81
      https://github.com/forge/java-parser/commit/7456a939db12237680e930736edf03ba61084c81
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-15 (Thu, 15 Aug 2013)

  Changed paths:
    M api/pom.xml
    M impl/pom.xml
    M pom.xml

  Log Message:
  -----------
  [maven-release-plugin] prepare for next development iteration

  Commit: 8d1f88d15a51465abcf4d2ef6e57c733f4a27578
      https://github.com/forge/java-parser/commit/8d1f88d15a51465abcf4d2ef6e57c733f4a27578
  Author: George Gastaldi <gegastaldi at gmail.com>
  Date:   2013-08-28 (Wed, 28 Aug 2013)

  Changed paths:
    M pom.xml

  Log Message:
  -----------
  Updated to jUnit 4.11

  Commit: b1abb73c248a5e0755d8ea700c71a3d9d001b311
      https://github.com/forge/java-parser/commit/b1abb73c248a5e0755d8ea700c71a3d9d001b311
  Author: Matt Benson <gudnabrsam at gmail.com>
  Date:   2013-08-28 (Wed, 28 Aug 2013)

  Changed paths:
    M api/pom.xml
    M api/src/main/java/org/jboss/forge/parser/java/AnnotationElement.java
    M api/src/main/java/org/jboss/forge/parser/java/EnumConstant.java
    M api/src/main/java/org/jboss/forge/parser/java/Field.java
    M api/src/main/java/org/jboss/forge/parser/java/Member.java
    M api/src/main/java/org/jboss/forge/parser/java/Method.java
    M impl/pom.xml
    M impl/src/main/java/org/jboss/forge/parser/java/ast/MethodFinderVisitor.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/AbstractJavaSource.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/AbstractJavaSourceMemberHolder.java
    A impl/src/main/java/org/jboss/forge/parser/java/impl/EnumConstantBodyImpl.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/EnumConstantImpl.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/FieldImpl.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/JavaPackageInfoImpl.java
    M impl/src/main/java/org/jboss/forge/parser/java/impl/TypeImpl.java
    A impl/src/test/java/org/jboss/forge/test/parser/java/EnumConstantAnnotationTest.java
    A impl/src/test/java/org/jboss/forge/test/parser/java/EnumConstantTest.java
    A impl/src/test/java/org/jboss/forge/test/parser/java/FieldMultipleTest.java
    M impl/src/test/java/org/jboss/forge/test/parser/java/FieldTypeTest.java
    A impl/src/test/java/org/jboss/forge/test/parser/java/JavaSourceCompatibilityTest.java
    A impl/src/test/resources/org/jboss/forge/grammar/java/MockAnnotatedEnumConstant.java
    M pom.xml

  Log Message:
  -----------
  Merge branch 'master' into refactor

Conflicts:
	api/src/main/java/org/jboss/forge/parser/java/EnumConstant.java
	impl/src/main/java/org/jboss/forge/parser/java/impl/EnumConstantImpl.java
	impl/src/main/java/org/jboss/forge/parser/java/impl/JavaEnumImpl.java
	impl/src/test/java/org/jboss/forge/test/parser/java/JavaEnumTest.java

Compare: https://github.com/forge/java-parser/compare/a54fdb035695...b1abb73c248a