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.

Sign up for GitHub

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

Jump to bottom

Support memoization #219

Merged
merged 27 commits into from Feb 14, 2024
Merged

Support memoization #219

merged 27 commits into from Feb 14, 2024

Conversation

nedtwigg
Copy link
Member

@nedtwigg nedtwigg commented Feb 13, 2024 •
edited

In a snapshot test, the code being tested always runs. In memoization, the code being tested only runs the first time.

@nedtwigg
A provisional API to implement memoizing of regular and suspending fu…
15de0ee 
…nctions.
@nedtwigg
Merge branch 'main' into feat/memo
5884367
@nedtwigg
Create the suspend fun memoize functions.
5483878
@nedtwigg
Add compileOnly dependency for selfie-lib on `kotlinx-serialization…
4b3da7e 
…-json`.
@nedtwigg
Try adding kotlinx-serialization-json to the test implementation to…
8dbeb11 
… fix CI.
@nedtwigg
Add memoizeAsJson to support any Kotlin @Serializable type.
756d8ad
@nedtwigg
Minor refactoring.
886f283
@nedtwigg
Add binary I/O into our filesystem abstraction.
e80ace9
@nedtwigg
Better formatting for dense inline functions.
197741e
@nedtwigg
Better implementation of toBe and toBe_TODO.
7e2649b
@nedtwigg
Minor improvement.
8d4c4ff
@nedtwigg
Add binary API.
b727fd9
@nedtwigg
Add UT_Memoize.
aac2426
@nedtwigg
Add a memoizeBinarySerializable method for java.io.Serializable.
68efa25
@nedtwigg
Add nanoTime.
e60eb98
@nedtwigg
Fix commented-out enabled.
c5b3d7f
@nedtwigg
Make the nanoTimeTest unique.
053da13
@nedtwigg
Add a test for the memoization (string and binary).
dd72619
@nedtwigg
Add the coroutine memo stuff into object Selfie.
7b79a2e
@nedtwigg
Remove all the coroutine stuff with find-replace more-or-less.
6fb35b7
@nedtwigg
Oops, committed a changed UT_Memoize.
aa1bb1c
@nedtwigg
Fix toBeFile_TODO.
bb75cf0
@nedtwigg
Make Selfie's non-suspend memo methods @JvmStatic.
4fa987b
@nedtwigg
Put all the Roundtrip stuff into one file.
2a4e443
@nedtwigg
Move the non-suspend memos into their own files.
bf49d4c
@nedtwigg
Put the rest of the memo classes into their own files.
0f60c30
@nedtwigg
Update changelog.
5c9e578
@nedtwigg nedtwigg marked this pull request as ready for review February 14, 2024 18:56
@nedtwigg nedtwigg merged commit 3e86f3e into main Feb 14, 2024
6 checks passed
@nedtwigg nedtwigg deleted the feat/memo branch February 14, 2024 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@nedtwigg