Skip to content

delay calculating the shared cache root until it's used - #7385

Merged
MorrisJobke merged 1 commit into
masterfrom
shared-delay-root
Dec 6, 2017
Merged

delay calculating the shared cache root until it's used#7385
MorrisJobke merged 1 commit into
masterfrom
shared-delay-root

Conversation

@icewind1991

Copy link
Copy Markdown
Member

Prevents having to initialize the shared storage needlessly

Signed-off-by: Robin Appelman <robin@icewind.nl>
@codecov

codecovBot commented Dec 4, 2017

Copy link
Copy Markdown

Codecov Report

Merging #7385 into master will increase coverage by <.01%.
The diff coverage is 96.42%.

@@ Coverage Diff @@## master #7385 +/- ##
============================================
+ Coverage 50.91% 50.91% +<.01% - Complexity 24699 24703 +4 
============================================
Files 1586 1586 Lines 94116 94125 +9 Branches 1361 1361 ============================================
+ Hits 47920 47926 +6 - Misses 46196 46199 +3
Impacted FilesCoverage ΔComplexity Δ
lib/private/Files/Cache/Wrapper/CacheJail.php88.37% <100%> (+0.27%)39 <1> (+1)⬆️
apps/files_sharing/lib/Cache.php89.23% <94.11%> (+1.29%)25 <8> (+3)⬆️
core/js/js.js62.98% <0%> (-0.57%)0% <0%> (ø)
apps/files_trashbin/lib/Trashbin.php72.53% <0%> (+0.24%)136% <0%> (ø)⬇️
lib/private/Files/Cache/Propagator.php96.2% <0%> (+1.26%)16% <0%> (ø)⬇️

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Makes sense.
Test seem happy.

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code looks good and it still works 👍

@MorrisJobke
MorrisJobke merged commit c32cb6b into masterDec 6, 2017
@MorrisJobke
MorrisJobke deleted the shared-delay-root branch December 6, 2017 14:14
@MorrisJobke

Copy link
Copy Markdown
Member

stable12 #7323

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@icewind1991@MorrisJobke@rullzer