Skip to content

added cache w/ configurable max size#62

Merged
loganwright merged 12 commits intomasterfrom
cache
May 2, 2017
Merged

added cache w/ configurable max size#62
loganwright merged 12 commits intomasterfrom
cache

Conversation

@loganwright
Copy link
Copy Markdown
Member

No description provided.

let raw = try stem.spawnLeaf(raw: expectation)
let context = Context([:])
let rendered = try stem.render(raw, with: context).makeString()
XCTAssertEqual(rendered, expectation)
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.

The backslash should be removed by leaf.

\#foobar()

should render as

#foobar()

func cacheSize() -> Size
}

public protocol CacheProtocol {
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.

Tagging this for removal

private let _gigabytes = _megabytes * 1000

extension Int {
var bytes: Int { return self }
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.

Would be nice to have these public in core.

@loganwright loganwright merged commit b9f827d into master May 2, 2017
@loganwright loganwright deleted the cache branch May 2, 2017 15:20
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