Skip to content
This repository was archived by the owner on Feb 26, 2023. It is now read-only.
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Using @TextChange and @Trace causes compile error. #1891

Closed

Description

@ytakahashi1981

Hello.
I found a very very small problem on AA 4.1.0.

@Trace
@TextChange(R.id.text)
void onTextChange(int start) {
   // do something
}

This code causes compile error because AA makes this code, the variable named 'start' already declared.

@Override
void onTextChange(final int start) {
  :
    long start = System.currentTimeMillis();
  :
}

And I couldn't change the argument name because AA processor says

Unrecognized parameter name. You can only have start, before, or count parameter name. Try to pick a parameter from the android.text.TextWatcher.onTextChanged() method.

So I can't use @TextChange and @Trace on the same method.

Thank you.

Activity

  1. dodgex commented on Nov 8, 2016

    @dodgex
    Member

    Thank you for the report. This one should be easy to fix on our side! :)

  2. self-assigned this
    on Nov 8, 2016
  3. dodgex commented on Nov 12, 2016

    @dodgex
    Member

    I just openend PR #1893 to fix this issue.

  4. WonderCsabo commented on Nov 24, 2016

    @WonderCsabo
    Member

    Fixed. Thanks @dodgex !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions