vitas / beaconloc

Beacon Locator - android application for scan, track and management of beacons
Apache License 2.0
104 stars 43 forks source link

Android automatic refactor - ObsoleteLayoutParam #13

Open luiscruz opened 7 years ago

luiscruz commented 7 years ago

Hi,

I am developing a tool to automatically refactor Android applications with the goal of improving energy efficiency. This pull request has the changes generated while applying the rule "ObsoleteLayoutParam".

While developing your application's views you might be specifying attributes in a view's artefact that are not necessary due to the nature of its parent. In this PR, those attributes were replaced by a comment.

I have made a previous validation of the changes and they seem correct. Unfortunately, this tool is not able keep the original whitespace of the files, so comparison without ignoring whitespace might be confusing. Please consider the changes and let me know if you agree with them.

Best, Luis