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.

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

DiamondOperator complains when instantiating/assigning inner types #779

Closed
carlosmiranda opened this issue Jun 6, 2016 · 18 comments
Closed
Assignees
Labels

Comments

@carlosmiranda
Copy link
Contributor

carlosmiranda commented Jun 6, 2016

DiamondOperator complains when you instantiate and assign an inner class type. For example, see the following code which is typical when using the Takes framework:

    @Override
    public Response act(final Request req) throws IOException {
        final RqForm.Smart smart = new RqForm.Smart(new RqForm.Base(req));
        // rest of code
    }

Qulice will complain about this:

[INFO] Checkstyle: src/test/java/com/xockets/xjm/http/TkAppTest.java[96]: Use diamond operator (DiamondOperatorCheck)

Note that RqForm.Smart is NOT a generic type, and neither are its parent types RqForm or Request. There is no way that the diamond operator can be applied here.

@krzyk
Copy link
Collaborator

krzyk commented Jun 6, 2016

@davvd valid bug

@krzyk
Copy link
Collaborator

krzyk commented Jun 6, 2016

@carlosmiranda thanks

@davvd
Copy link

davvd commented Jun 8, 2016

@davvd valid bug

@krzyk tag "bug" added

@davvd davvd added the bug label Jun 8, 2016
@davvd
Copy link

davvd commented Jun 8, 2016

@carlosmiranda many thanks for the report, I topped your account for 15 mins, transaction 89309942

@davvd davvd added DEV labels Jun 8, 2016
@krzyk
Copy link
Collaborator

krzyk commented Jun 11, 2016

@davvd this is urgent

@davvd davvd added the urgent label Jun 13, 2016
@davvd
Copy link

davvd commented Jun 13, 2016

@davvd this is urgent

@krzyk sure, thanks, I added "urgent" label to it

@davvd
Copy link

davvd commented Jun 14, 2016

@dskalenko this task is for you, keep these principles in mind, and don't hesitate to ask any technical questions. The budget of this issue is 30 mins, which is exactly how much will be paid when the task is done (see this for explanation)

@dskalenko
Copy link
Contributor

@davvd I need more time to fix this issue, thanks for understanding

dskalenko added a commit to dskalenko/qulice that referenced this issue Jun 23, 2016
@davvd
Copy link

davvd commented Jun 24, 2016

@davvd I need more time to fix this issue, thanks for understanding

@dskalenko sure, take your time

dskalenko added a commit to dskalenko/qulice that referenced this issue Jul 6, 2016
dskalenko added a commit to dskalenko/qulice that referenced this issue Jul 6, 2016
dskalenko added a commit to dskalenko/qulice that referenced this issue Jul 6, 2016
dskalenko added a commit to dskalenko/qulice that referenced this issue Jul 6, 2016
@dskalenko
Copy link
Contributor

@davvd here my PR #789

@davvd
Copy link

davvd commented Jul 7, 2016

@davvd here my PR #789

@dskalenko thanks

dskalenko added a commit to dskalenko/qulice that referenced this issue Jul 27, 2016
dskalenko added a commit to dskalenko/qulice that referenced this issue Jul 29, 2016
@krzyk
Copy link
Collaborator

krzyk commented Jul 31, 2016

@dskalenko now you have to ask the issue creator to close it

@carlosmiranda
Copy link
Contributor Author

@dskalenko thanks!

@krzyk
Copy link
Collaborator

krzyk commented Aug 1, 2016

@davvd this is not urgent

@krzyk krzyk mentioned this issue Aug 1, 2016
@davvd davvd removed the urgent label Aug 1, 2016
@davvd
Copy link

davvd commented Aug 1, 2016

@davvd this is not urgent

@krzyk got it, "urgent" tag removed from here

@davvd
Copy link

davvd commented Aug 2, 2016

@dskalenko thanks, paid, 30 mins to your account, payment ID is 95473345, task took 1148 hours and 22 mins to complete... +30 to your rating, your total score is +60

@caarlos0
Copy link
Contributor

@krzyk can you release it plz?

@krzyk
Copy link
Collaborator

krzyk commented Aug 13, 2016

@caarlos0 it was released on 0.17.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants