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

Improve energy efficiency by applying Cache Energy Pattern #138

Merged
merged 2 commits into from
Jul 27, 2020

Conversation

anasofiagribeiro
Copy link
Contributor

This improves the energy efficiency of SecondScreen by applying the Cache Energy Pattern for mobile applications.

The energy pattern was applied in TaskerConditionReceiver.java. The general idea is to avoid performing unnecessary operations by using cache mechanisms. In particular, the operations in method onReceive will only execute if the bundles changes.

@codecov
Copy link

codecov bot commented Jul 24, 2020

Codecov Report

Merging #138 into master will increase coverage by 0.00%.
The diff coverage is 11.11%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #138   +/-   ##
=========================================
  Coverage     12.57%   12.58%           
  Complexity      375      375           
=========================================
  Files           123      123           
  Lines          8221     8226    +5     
  Branches       1384     1385    +1     
=========================================
+ Hits           1034     1035    +1     
- Misses         7055     7059    +4     
  Partials        132      132           
Impacted Files Coverage Δ Complexity Δ
...erbb/taskbar/receiver/TaskerConditionReceiver.java 9.52% <11.11%> (+3.27%) 1.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e1ed27...e315bfc. Read the comment docs.

@farmerbb farmerbb merged commit 603e5e1 into farmerbb:master Jul 27, 2020
@farmerbb
Copy link
Owner

Thanks!

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