Skip to content
New issue

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

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

Already on GitHub? # to your account

doc: fix javadoc in TypeFactory.java #2688

Merged
merged 4 commits into from
Oct 20, 2018
Merged

doc: fix javadoc in TypeFactory.java #2688

merged 4 commits into from
Oct 20, 2018

Conversation

zielint0
Copy link
Contributor

@zielint0 zielint0 commented Oct 18, 2018

SonarQube: Remove this unnecessary cast to "CtElement".

@zielint0 zielint0 changed the title refactor: remove redundant cast refactor: remove redundant cast (fix SonarQube) Oct 18, 2018
@monperrus
Copy link
Collaborator

We have two different PRs here (typo and cast). They should be separated in two.

@@ -282,7 +282,7 @@ public TypeFactory() {
}

/**
* Creates a reference to a one-dimension array of given type.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@monperrus
That's right. Thank you for your careful review!

@zielint0
Copy link
Contributor Author

We have two different PRs here (typo and cast). They should be separated in two.

Cast is in #2688

@zielint0 zielint0 changed the title refactor: remove redundant cast (fix SonarQube) doc: fix javadoc in TypeFactory.java Oct 19, 2018
@pvojtechovsky pvojtechovsky merged commit 4f70ab1 into INRIA:master Oct 20, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants