Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Moves check for default toString to Reflection util class

+review REVIEW-5501

  1. … 2 more files in changeset.
Make DirectInstantiator a singleton.

  1. … 89 more files in changeset.
Adding generic types to readableField methods

+review REVIEW-5337

Adding support for locating static methods to JavaReflectionUtil

+review REVIEW-5337

  1. … 3 more files in changeset.
Making PropertyAccessor generic

+review REVIEW-5337

  1. … 4 more files in changeset.
Converting play javascript compiler to use JavaReflectionUtil

+review REVIEW-5337

  1. … 1 more file in changeset.
Changed ClassInspector to traverse the graph of interfaces for a class. Use this in ManagedProxyClassGenerator

  1. … 2 more files in changeset.
Extracted ClassInspector from AbstractClassGenerator so that the logic can be reused elsewhere.

    • -0
    • +206
    ./ClassInspector.java
    • -0
    • +28
    ./PropertyDetails.java
  1. … 2 more files in changeset.
Added missing 'char' to 'Character' to JavaReflectionUtil's mapping from primitive to boxed types.

Simplify MethodDescription, move to base reflection services and reuse.

    • -0
    • +104
    ./MethodDescription.java
  1. … 6 more files in changeset.
Attach generated implementation classes (weakly) to schema objects, instead of using a global cache.

This is done to avoid having to go through another cache and acquire a lock every time we need to instantiate a managed type.

This does make unit testing slightly more inconvenient in that a singleton schema store has to be used for all tests, but I deemed this an acceptable trade off.

+review REVIEW-5329

    • -0
    • +48
    ./MethodSignatureEquivalence.java
  1. … 19 more files in changeset.
Added a RuleAction implementation that’s backed by a RuleSource class with a single @Mutate method

  1. … 2 more files in changeset.
Unpack more of the base component rule definition handler to not require abstract methods.

Motivation here is less boilerplate for subclasses, stronger typing, and more clearly defined semantics.

+review REVIEW-5081

  1. … 8 more files in changeset.
Fixed source file name

    • -0
    • +26
    ./NoSuchMethodException.java
Some clean up of JavaReflectionUtil

- Tightened up definition of what method() does when no matching method can be found

- Removed readField() as it is only used by a test

    • -0
    • +26
    ./NoSuchMEthodException.java
  1. … 3 more files in changeset.
When reflectively invoking a method, wrap thrown checked exceptions in UncheckedException instead of GradleException.

This was particularly problematic when invoking task actions, as the real exception was being masked in the output to the user.

  1. … 1 more file in changeset.
When reflectively invoking a method, wrap thrown checked exceptions in UncheckedException instead of GradleException.

This was particularly problematic when invoking task actions, as the real exception was being masked in the output to the user.

  1. … 1 more file in changeset.
More cleanup on Help for Tasks based on reviews

- REVIEW-3460 add support for enums

- REVIEW-3459 move knowledge from CommandLineTaskConfigurer to CommandLineOptionDescriptor

- knowledge bout option parameters

- how to apply option on tasks

- REVIEW-3459 - dont support '--flag true' just use '--flag' instead

  1. … 10 more files in changeset.
Moved some classes out of org.gradle.util.

  1. … 62 more files in changeset.
Some minor tidy-ups.

  1. … 1 more file in changeset.
Reworked the property handling of JavaReflectionUtil

- Replaced readProperty() and writeProperty() with methods to look up properties.

- Tightened up what is considered a getter or setter method (eg don't consider static methods).

    • -0
    • +26
    ./NoSuchPropertyException.java
    • -0
    • +25
    ./PropertyAccessor.java
    • -0
    • +25
    ./PropertyMutator.java
  1. … 5 more files in changeset.
GRADLE-2863 - don't get involved in private method calls in the dynamic object stuff.

I originally thought we needed to get the desired behaviour, but it turns out that's only necessary for static inner classes. Therefore, we are simplifying our stuff by dropping support for decorating static inner classes. This lack of support is enforced in any way yet though.

  1. … 6 more files in changeset.
GRADLEREV-51 - Remove last of Groovy based ReflectionUtil.

  1. … 7 more files in changeset.
GRADLEREV-51 - replace some usages of Groovy based reflection with Java based.

  1. … 1 more file in changeset.
Merge 1.7-rc-2 release.

Conflicts:

subprojects/core-impl/src/main/groovy/org/gradle/api/internal/artifacts/repositories/DefaultBaseRepositoryFactory.java

subprojects/core-impl/src/test/groovy/org/gradle/api/internal/artifacts/repositories/DefaultBaseRepositoryFactoryTest.groovy

subprojects/core/src/main/groovy/org/gradle/process/internal/ExecAction.java

subprojects/core/src/main/groovy/org/gradle/process/internal/JavaExecAction.java

subprojects/docs/src/docs/release/notes.md

  1. … 10 more files in changeset.
Find annotations implemented by super classes.

  1. … 1 more file in changeset.
Merge remote-tracking branch 'remotes/origin/release'

  1. … 11 more files in changeset.
Make copy specs and file copy details not extensible (in DSL terms).

This is not the most elegant change. The changes to the class generator were made in such a way to change as little is possible due to the timing of this change in the release cycle.

The way we are achieving this functionality (i.e. disabling generating extensibility support) will change in the next release, therefore I am not going to document it in the release notes.

  1. … 7 more files in changeset.
Replace unnecessarily Groovy enabled method dispatch with plain Java.

  1. … 1 more file in changeset.
Fix method searching to always ignore methods overridden by subclasses.

Previously, they were only being ignored if they were 'found' as part of the search.