Update the Cache Tab ready for release #675

Merged
merged 2 commits into from Dec 13, 2013

Conversation

Projects
None yet
3 participants
@bryanjhogan
Contributor

bryanjhogan commented Dec 13, 2013

Changes to CacheTab.cs and associated files.
Added tests to CacheShould.cs

bryanjhogan added some commits Dec 9, 2013

Changed CacheItemModel properties to nullable.
Corresponding change in Cache.cs

Moving assignment of EffectivePercentagePhysicalMemoryLimit and EffectivePrivateBytesLimit outside loop in Cache.cs
Expanded try/catch in TryGetCacheItemModel.

Added tests to CacheShould.cs.
@CGijbels

This comment has been minimized.

Show comment
Hide comment
@CGijbels

CGijbels Dec 13, 2013

Collaborator

Info: Cleaned up version of #672

Collaborator

CGijbels commented Dec 13, 2013

Info: Cleaned up version of #672

@CGijbels

This comment has been minimized.

Show comment
Hide comment
@CGijbels

CGijbels Dec 13, 2013

Collaborator

Thanks @bryanjhogan ! Looking good!

Collaborator

CGijbels commented Dec 13, 2013

Thanks @bryanjhogan ! Looking good!

CGijbels added a commit that referenced this pull request Dec 13, 2013

Merge pull request #675 from bryanjhogan/cacheTab
Improvements to the Cache tab

@CGijbels CGijbels merged commit 2182cd3 into Glimpse:master Dec 13, 2013

@ghost ghost assigned CGijbels Dec 13, 2013

@avanderhoorn

This comment has been minimized.

Show comment
Hide comment
@avanderhoorn

avanderhoorn Dec 13, 2013

Member

Great work guys!

Member

avanderhoorn commented Dec 13, 2013

Great work guys!

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