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

Code cleanup #4

Open
pakoito opened this issue Aug 10, 2016 · 2 comments
Open

Code cleanup #4

pakoito opened this issue Aug 10, 2016 · 2 comments
Assignees

Comments

@pakoito
Copy link

pakoito commented Aug 10, 2016

There are some other parts of the app where static analysis has flagged me, like lack of equals and hashCode on some inner classes inside AbstractSpecialisedImmutableMap. Plenty of redundant casts and extra typing that could be replaced by diamonds. Indentation is also all over the place, with a mix of tabs and two and four spaces.

I could open another PR and help, if contributions are accepted.

@msteindorfer
Copy link
Member

@pakoito, thanks for getting in touch. Which static analysis tools are you using? Are they part of the Android tool suite?

Part of the data structure implementations were generated by a code generator. It might therefore be true, that redundant casts were introduced to cater for generating primitive specialized versions of the collections. But you're right, that those should be cleaned up in one or the other way.

Indentation and formatting for the project should obey to the Google Coding Styleguide, which is setup via the maven-eclipse-plugin integration.

Parts of capsule were extracted from the rascal-value and rascal codebase, before becoming a standalone project. Those three projects are still co-evolving at the moment. Some formatting inconsistencies might still be around therefore. I'll take care to fix those inconsistencies.

@pakoito
Copy link
Author

pakoito commented Aug 11, 2016

I'm currently using SonarLint, which is an aggregation of PMD, FindBugs and others.

The tests and pom file are indentated with tabs.

@msteindorfer msteindorfer self-assigned this Aug 15, 2016
msteindorfer added a commit that referenced this issue Aug 25, 2016
This relates to issue #4.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants