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

Hotfix/code formatting #10

Merged
merged 3 commits into from
Jun 30, 2015
Merged

Hotfix/code formatting #10

merged 3 commits into from
Jun 30, 2015

Conversation

VenomVendor
Copy link
Contributor

Formatted code as per Android guidelines.
Added Annotations.
Updated view variables with findViewById.
Moved magic number/values to external sources.
Splitted init into multiple methods.
Updated Exceptions.

Refactored ids in Views.
Moved magic number/values to external sources.
Updated `addTextChangedListener` to avoid `NullPointerException`.

Signed-off-by: VenomVendor <info@VenomVendor.com>
- Splitted `init` into multiple methods.
- Updated view variables with `findViewById`.
- throw valid exceptions.
- converted internal methods to private.

Signed-off-by: VenomVendor <info@VenomVendor.com>
-Remove appcompact from sample.

Signed-off-by: VenomVendor <info@VenomVendor.com>
@VenomVendor VenomVendor mentioned this pull request Jun 29, 2015
vinaygaba added a commit that referenced this pull request Jun 30, 2015
@vinaygaba vinaygaba merged commit 5ee4bdc into vinaygaba:master Jun 30, 2015
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

Successfully merging this pull request may close these issues.

2 participants