Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

review1: feature: CtMethod#isOverriding(CtMethod) #1220

Merged
merged 8 commits into from
May 4, 2017
Merged

review1: feature: CtMethod#isOverriding(CtMethod) #1220

merged 8 commits into from
May 4, 2017

Commits on May 4, 2017

  1. keep contract: Method overrides itself

    @pvojtechovsky
    pvojtechovsky committed May 4, 2017
    Configuration menu
    Copy the full SHA
    076b8aa View commit details
    Browse the repository at this point in the history

  2. feature: CtMethod#isOverriding(CtMethod)

    @pvojtechovsky
    pvojtechovsky committed May 4, 2017
    Configuration menu
    Copy the full SHA
    9c8b90c View commit details
    Browse the repository at this point in the history

  3. fix NPE when method declaration is not in model

    @pvojtechovsky
    pvojtechovsky committed May 4, 2017
    Configuration menu
    Copy the full SHA
    2160dfc View commit details
    Browse the repository at this point in the history

  4. test 1

    @pvojtechovsky
    pvojtechovsky committed May 4, 2017
    Configuration menu
    Copy the full SHA
    13103db View commit details
    Browse the repository at this point in the history

  5. test method overriding

    @pvojtechovsky
    pvojtechovsky committed May 4, 2017
    Configuration menu
    Copy the full SHA
    9572b7e View commit details
    Browse the repository at this point in the history

  6. improve comment. Describe legacy contract this.isOverriding(this)

    @pvojtechovsky
    pvojtechovsky committed May 4, 2017
    Configuration menu
    Copy the full SHA
    e26e81f View commit details
    Browse the repository at this point in the history

  7. use CtExecutableReference#getExecutableDeclaration

    @pvojtechovsky
    pvojtechovsky committed May 4, 2017
    Configuration menu
    Copy the full SHA
    8781578 View commit details
    Browse the repository at this point in the history

  8. assert that test is checking some methods

    @pvojtechovsky
    pvojtechovsky committed May 4, 2017
    Configuration menu
    Copy the full SHA
    f3a5d55 View commit details
    Browse the repository at this point in the history